mirror of
https://github.com/mattermost/mattermost-mobile.git
synced 2026-11-04 18:08:54 +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 Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * include servers with no persistence flag set in share list * disable button for accessibility Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * 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 --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
214 lines
7.4 KiB
TypeScript
214 lines
7.4 KiB
TypeScript
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
import {getHasEverStartedSyncSubject, getLoadingTeamChannelsSubject, setTeamLoading} from './team_load_store';
|
|
|
|
describe('TeamLoadStore', () => {
|
|
// Use unique server URLs for each test to avoid cross-test pollution
|
|
let testCounter = 0;
|
|
const getUniqueServerUrl = () => `https://test-server-${++testCounter}.com`;
|
|
|
|
it('should create a new subject for a server URL', () => {
|
|
const serverUrl = getUniqueServerUrl();
|
|
const subject = getLoadingTeamChannelsSubject(serverUrl);
|
|
|
|
expect(subject).toBeDefined();
|
|
expect(subject.value).toBe(0);
|
|
});
|
|
|
|
it('should return the same subject for the same server URL', () => {
|
|
const serverUrl = getUniqueServerUrl();
|
|
const subject1 = getLoadingTeamChannelsSubject(serverUrl);
|
|
const subject2 = getLoadingTeamChannelsSubject(serverUrl);
|
|
|
|
expect(subject1).toBe(subject2);
|
|
});
|
|
|
|
it('should create different subjects for different server URLs', () => {
|
|
const serverUrl1 = getUniqueServerUrl();
|
|
const serverUrl2 = getUniqueServerUrl();
|
|
const subject1 = getLoadingTeamChannelsSubject(serverUrl1);
|
|
const subject2 = getLoadingTeamChannelsSubject(serverUrl2);
|
|
|
|
expect(subject1).not.toBe(subject2);
|
|
});
|
|
|
|
it('should increment loading count when setTeamLoading is true', () => {
|
|
const serverUrl = getUniqueServerUrl();
|
|
const subject = getLoadingTeamChannelsSubject(serverUrl);
|
|
const initialValue = subject.value;
|
|
|
|
setTeamLoading(serverUrl, true);
|
|
|
|
expect(subject.value).toBe(initialValue + 1);
|
|
});
|
|
|
|
it('should decrement loading count when setTeamLoading is false', () => {
|
|
const serverUrl = getUniqueServerUrl();
|
|
const subject = getLoadingTeamChannelsSubject(serverUrl);
|
|
|
|
setTeamLoading(serverUrl, true);
|
|
setTeamLoading(serverUrl, true);
|
|
const valueAfterIncrement = subject.value;
|
|
|
|
setTeamLoading(serverUrl, false);
|
|
|
|
expect(subject.value).toBe(valueAfterIncrement - 1);
|
|
});
|
|
|
|
it('should handle multiple concurrent loading operations', () => {
|
|
const serverUrl = getUniqueServerUrl();
|
|
const subject = getLoadingTeamChannelsSubject(serverUrl);
|
|
const initialValue = subject.value;
|
|
|
|
setTeamLoading(serverUrl, true);
|
|
setTeamLoading(serverUrl, true);
|
|
setTeamLoading(serverUrl, true);
|
|
|
|
expect(subject.value).toBe(initialValue + 3);
|
|
|
|
setTeamLoading(serverUrl, false);
|
|
setTeamLoading(serverUrl, false);
|
|
|
|
expect(subject.value).toBe(initialValue + 1);
|
|
|
|
setTeamLoading(serverUrl, false);
|
|
|
|
expect(subject.value).toBe(initialValue);
|
|
});
|
|
|
|
it('should manage loading states independently per server', () => {
|
|
const serverUrl1 = getUniqueServerUrl();
|
|
const serverUrl2 = getUniqueServerUrl();
|
|
const subject1 = getLoadingTeamChannelsSubject(serverUrl1);
|
|
const subject2 = getLoadingTeamChannelsSubject(serverUrl2);
|
|
|
|
setTeamLoading(serverUrl1, true);
|
|
setTeamLoading(serverUrl1, true);
|
|
|
|
setTeamLoading(serverUrl2, true);
|
|
|
|
expect(subject1.value).toBe(2);
|
|
expect(subject2.value).toBe(1);
|
|
|
|
setTeamLoading(serverUrl1, false);
|
|
|
|
expect(subject1.value).toBe(1);
|
|
expect(subject2.value).toBe(1);
|
|
});
|
|
|
|
it('should emit values through observable', () => {
|
|
const serverUrl = getUniqueServerUrl();
|
|
const subject = getLoadingTeamChannelsSubject(serverUrl);
|
|
const mockCallback = jest.fn();
|
|
const subscription = subject.subscribe(mockCallback);
|
|
const initialValue = subject.value;
|
|
|
|
// Should immediately get current value
|
|
expect(mockCallback).toHaveBeenCalledWith(initialValue);
|
|
|
|
setTeamLoading(serverUrl, true);
|
|
|
|
// Should get new value
|
|
expect(mockCallback).toHaveBeenCalledWith(initialValue + 1);
|
|
|
|
setTeamLoading(serverUrl, false);
|
|
|
|
// Should get decremented value
|
|
expect(mockCallback).toHaveBeenCalledWith(initialValue);
|
|
|
|
expect(mockCallback).toHaveBeenCalledTimes(3);
|
|
|
|
subscription.unsubscribe();
|
|
|
|
// After unsubscribe, callback should not be called
|
|
setTeamLoading(serverUrl, true);
|
|
expect(mockCallback).toHaveBeenCalledTimes(3);
|
|
});
|
|
|
|
it('should allow negative loading counts', () => {
|
|
const serverUrl = getUniqueServerUrl();
|
|
const subject = getLoadingTeamChannelsSubject(serverUrl);
|
|
const initialValue = subject.value;
|
|
|
|
setTeamLoading(serverUrl, false);
|
|
|
|
expect(subject.value).toBe(initialValue - 1);
|
|
});
|
|
|
|
it('should initialize hasEverStartedSync to false', () => {
|
|
const serverUrl = getUniqueServerUrl();
|
|
const subject = getHasEverStartedSyncSubject(serverUrl);
|
|
|
|
expect(subject.value).toBe(false);
|
|
});
|
|
|
|
it('should return the same hasEverStartedSync subject for the same server URL', () => {
|
|
const serverUrl = getUniqueServerUrl();
|
|
const subject1 = getHasEverStartedSyncSubject(serverUrl);
|
|
const subject2 = getHasEverStartedSyncSubject(serverUrl);
|
|
|
|
expect(subject1).toBe(subject2);
|
|
});
|
|
|
|
it('should create different hasEverStartedSync subjects for different server URLs', () => {
|
|
const serverUrl1 = getUniqueServerUrl();
|
|
const serverUrl2 = getUniqueServerUrl();
|
|
const subject1 = getHasEverStartedSyncSubject(serverUrl1);
|
|
const subject2 = getHasEverStartedSyncSubject(serverUrl2);
|
|
|
|
expect(subject1).not.toBe(subject2);
|
|
});
|
|
|
|
it('should flip hasEverStartedSync to true on first setTeamLoading(true)', () => {
|
|
const serverUrl = getUniqueServerUrl();
|
|
const subject = getHasEverStartedSyncSubject(serverUrl);
|
|
|
|
setTeamLoading(serverUrl, true);
|
|
|
|
expect(subject.value).toBe(true);
|
|
});
|
|
|
|
it('should keep hasEverStartedSync true after setTeamLoading(false) (monotonic latch)', () => {
|
|
const serverUrl = getUniqueServerUrl();
|
|
const subject = getHasEverStartedSyncSubject(serverUrl);
|
|
|
|
setTeamLoading(serverUrl, true);
|
|
setTeamLoading(serverUrl, false);
|
|
|
|
expect(subject.value).toBe(true);
|
|
});
|
|
|
|
it('should manage hasEverStartedSync independently per server', () => {
|
|
const serverUrl1 = getUniqueServerUrl();
|
|
const serverUrl2 = getUniqueServerUrl();
|
|
const subject1 = getHasEverStartedSyncSubject(serverUrl1);
|
|
const subject2 = getHasEverStartedSyncSubject(serverUrl2);
|
|
|
|
setTeamLoading(serverUrl1, true);
|
|
|
|
expect(subject1.value).toBe(true);
|
|
expect(subject2.value).toBe(false);
|
|
});
|
|
|
|
it('should emit hasEverStartedSync only once across multiple setTeamLoading calls', () => {
|
|
const serverUrl = getUniqueServerUrl();
|
|
const subject = getHasEverStartedSyncSubject(serverUrl);
|
|
const mockCallback = jest.fn();
|
|
const subscription = subject.subscribe(mockCallback);
|
|
|
|
// Initial replay of current value
|
|
expect(mockCallback).toHaveBeenNthCalledWith(1, false);
|
|
|
|
setTeamLoading(serverUrl, true);
|
|
setTeamLoading(serverUrl, true);
|
|
setTeamLoading(serverUrl, false);
|
|
setTeamLoading(serverUrl, true);
|
|
|
|
expect(mockCallback).toHaveBeenCalledTimes(2);
|
|
expect(mockCallback).toHaveBeenNthCalledWith(2, true);
|
|
|
|
subscription.unsubscribe();
|
|
});
|
|
});
|