mirror of
https://github.com/mattermost/mattermost-mobile.git
synced 2026-11-12 06:00:14 +00:00
* implements auto cache cleanup when zero persistence mode is enabled (0 value)
* uses memory flag with sqlite instead of loki when running in zero persistence mode
* renames ephemeral mode manager and improves readability
* zero persistence refactors and restore values during transitions
* fixes ephemeral mode tests
* use initializeClient instead of just initialize to wait for doReconnect to finish
that way teams loading is awaited while WS reconnects during ZPM transitions
* adds inital sync as a mount hook to prevent undesired load error states
during app cold start in ZPM mode, we need a way to load team and channels
without showing loading error during the transition. The nre is initial sync
hook allows for it
* reset started sync subject on wipe
* removes extra log lines
* resets tracked server during ephemeral mode transitions
* improvements to apply persistence mode changes when something fails
* address coderabbit PR suggestions
* add batching time logging
* throws ephemeral mode change error as is
* make deleteServerDatabaseFiles private
* inline derive persistence flag function
* return current user id when refetching current user
* support zero persistence mode server for push notifications
* adds support for zero persistence mode when sharing to mattermost
* adds clarifying comment
* catch and log error deleting files
* reorder translations to correct position
* adds some new tests to improve coverage
* ios push notification native fix
* logout when fetchMe fails with 401 status code during reconnection
* properly surface errors
* return proper false value when checking for zero persistence flag
* change comments and properly await on some operations
* prevent not set persistence mode to report as an error
* include servers with no persistence flag set in share list
* disable button for accessibility
* improves when to store and remove signing key from global area
* review push notification reception running in zero persistence mode
* add server to ephemeral mode manager right after login
---------
(cherry picked from commit a5ff8a0d95)
Co-authored-by: Carlos Garcia <carlos.garcia@mattermost.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
152 lines
6.6 KiB
TypeScript
152 lines
6.6 KiB
TypeScript
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
import {wipeServerDatabaseWithRetry} from '@actions/local/ephemeral_mode/wipe';
|
|
import {terminateSession} from '@actions/local/session';
|
|
import {refetchCurrentUser} from '@actions/remote/user';
|
|
import DatabaseManager from '@database/manager';
|
|
import {getServerCredentials} from '@init/credentials';
|
|
import EphemeralModeManager from '@managers/ephemeral_mode_manager';
|
|
import WebsocketManager from '@managers/websocket_manager';
|
|
import {getCurrentChannelId, getCurrentTeamId, getPushVerificationStatus, prepareCommonSystemValues} from '@queries/servers/system';
|
|
|
|
import {applyPersistenceModeChange} from './refresh';
|
|
|
|
jest.mock('@actions/local/ephemeral_mode/wipe', () => ({
|
|
wipeServerFiles: jest.fn(),
|
|
wipeServerDatabaseWithRetry: jest.fn(),
|
|
}));
|
|
jest.mock('@actions/local/session', () => ({
|
|
cancelSessionNotification: jest.fn(),
|
|
terminateSession: jest.fn().mockResolvedValue({}),
|
|
}));
|
|
jest.mock('@actions/remote/user', () => ({
|
|
refetchCurrentUser: jest.fn(),
|
|
}));
|
|
jest.mock('@init/credentials', () => ({
|
|
getServerCredentials: jest.fn(),
|
|
}));
|
|
jest.mock('@managers/ephemeral_mode_manager', () => ({
|
|
__esModule: true,
|
|
default: {removeServer: jest.fn(), addServer: jest.fn()},
|
|
}));
|
|
jest.mock('@managers/websocket_manager', () => ({
|
|
__esModule: true,
|
|
default: {invalidateClient: jest.fn(), createClient: jest.fn(), initializeClient: jest.fn()},
|
|
}));
|
|
jest.mock('@queries/servers/system', () => ({
|
|
getCurrentChannelId: jest.fn(),
|
|
getCurrentTeamId: jest.fn(),
|
|
getPushVerificationStatus: jest.fn(),
|
|
prepareCommonSystemValues: jest.fn(),
|
|
}));
|
|
jest.mock('@utils/log');
|
|
|
|
describe('applyPersistenceModeChange', () => {
|
|
const serverUrl = 'https://server.test';
|
|
let setActiveServerDatabaseSpy: jest.SpyInstance;
|
|
const mockOperator = {batchRecords: jest.fn().mockResolvedValue(undefined)};
|
|
|
|
beforeEach(() => {
|
|
jest.clearAllMocks();
|
|
jest.mocked(wipeServerDatabaseWithRetry).mockResolvedValue({success: true});
|
|
setActiveServerDatabaseSpy = jest.spyOn(DatabaseManager, 'setActiveServerDatabase').mockResolvedValue(undefined);
|
|
jest.spyOn(DatabaseManager, 'getActiveServerUrl').mockResolvedValue(serverUrl);
|
|
jest.spyOn(DatabaseManager, 'getServerDatabaseAndOperator').mockReturnValue({
|
|
database: {} as never,
|
|
operator: mockOperator as never,
|
|
});
|
|
jest.mocked(refetchCurrentUser).mockResolvedValue({currentUserId: 'u1'});
|
|
jest.mocked(getPushVerificationStatus).mockResolvedValue('');
|
|
jest.mocked(getCurrentTeamId).mockResolvedValue('');
|
|
jest.mocked(getCurrentChannelId).mockResolvedValue('');
|
|
jest.mocked(prepareCommonSystemValues).mockResolvedValue([]);
|
|
jest.mocked(getServerCredentials).mockResolvedValue({
|
|
serverUrl,
|
|
userId: 'u1',
|
|
token: 'tok',
|
|
preauthSecret: 'preauth',
|
|
});
|
|
});
|
|
|
|
afterEach(() => {
|
|
jest.restoreAllMocks();
|
|
});
|
|
|
|
it('should wipe the server DB and files, re-seed user state, and create a WS client', async () => {
|
|
const result = await applyPersistenceModeChange(serverUrl);
|
|
|
|
expect(WebsocketManager.invalidateClient).toHaveBeenCalledWith(serverUrl);
|
|
expect(EphemeralModeManager.removeServer).toHaveBeenCalledWith(serverUrl);
|
|
expect(wipeServerDatabaseWithRetry).toHaveBeenCalledWith(serverUrl);
|
|
expect(refetchCurrentUser).toHaveBeenCalledWith(serverUrl, undefined);
|
|
expect(WebsocketManager.createClient).toHaveBeenCalledWith(serverUrl, 'tok', 'preauth');
|
|
expect(WebsocketManager.initializeClient).toHaveBeenCalledWith(serverUrl);
|
|
expect(setActiveServerDatabaseSpy).toHaveBeenCalledWith(serverUrl);
|
|
expect(terminateSession).not.toHaveBeenCalled();
|
|
expect(EphemeralModeManager.addServer).toHaveBeenCalledWith(serverUrl, {cleanFileCache: false});
|
|
expect(result).toEqual({});
|
|
});
|
|
|
|
it('should skip WS client creation when there are no stored credentials', async () => {
|
|
jest.mocked(getServerCredentials).mockResolvedValue(null);
|
|
|
|
await applyPersistenceModeChange(serverUrl);
|
|
|
|
expect(WebsocketManager.createClient).not.toHaveBeenCalled();
|
|
expect(WebsocketManager.initializeClient).not.toHaveBeenCalled();
|
|
expect(terminateSession).not.toHaveBeenCalled();
|
|
expect(EphemeralModeManager.addServer).toHaveBeenCalledWith(serverUrl, {cleanFileCache: false});
|
|
});
|
|
|
|
it('should skip setActiveServerDatabase when the server is not active', async () => {
|
|
jest.spyOn(DatabaseManager, 'getActiveServerUrl').mockResolvedValue('https://other.test');
|
|
|
|
await applyPersistenceModeChange(serverUrl);
|
|
|
|
expect(setActiveServerDatabaseSpy).not.toHaveBeenCalled();
|
|
});
|
|
|
|
it('should terminate the session when the wipe throws', async () => {
|
|
const wipeError = new Error('disk full');
|
|
jest.mocked(wipeServerDatabaseWithRetry).mockRejectedValue(wipeError);
|
|
|
|
const result = await applyPersistenceModeChange(serverUrl);
|
|
|
|
expect(terminateSession).toHaveBeenCalledWith(serverUrl, false);
|
|
expect(EphemeralModeManager.addServer).not.toHaveBeenCalled();
|
|
expect(result).toEqual({error: wipeError});
|
|
});
|
|
|
|
it('should terminate the session when the wipe exhausts all retries', async () => {
|
|
jest.mocked(wipeServerDatabaseWithRetry).mockResolvedValue({success: false});
|
|
|
|
const result = await applyPersistenceModeChange(serverUrl);
|
|
|
|
expect(terminateSession).toHaveBeenCalledWith(serverUrl, false);
|
|
expect(EphemeralModeManager.addServer).not.toHaveBeenCalled();
|
|
expect(result.error).toBeTruthy();
|
|
});
|
|
|
|
it('should terminate the session when restoration fails after a successful wipe', async () => {
|
|
const fetchError = new Error('network error');
|
|
jest.mocked(refetchCurrentUser).mockRejectedValue(fetchError);
|
|
|
|
const result = await applyPersistenceModeChange(serverUrl);
|
|
|
|
expect(terminateSession).toHaveBeenCalledWith(serverUrl, false);
|
|
expect(EphemeralModeManager.addServer).not.toHaveBeenCalled();
|
|
expect(result).toEqual({error: fetchError});
|
|
});
|
|
|
|
it('should terminate the session when refetchCurrentUser resolves with an error', async () => {
|
|
jest.mocked(refetchCurrentUser).mockResolvedValue({error: 'No user fetched'});
|
|
|
|
const result = await applyPersistenceModeChange(serverUrl);
|
|
|
|
expect(terminateSession).toHaveBeenCalledWith(serverUrl, false);
|
|
expect(EphemeralModeManager.addServer).not.toHaveBeenCalled();
|
|
expect(result.error).toBeTruthy();
|
|
});
|
|
});
|