refactor

src/stores/__tests__/vrcStatus.test.js

@@ -0,0 +1,132 @@
+import { beforeEach, describe, expect, test, vi } from 'vitest';
+import { createPinia, setActivePinia } from 'pinia';
+
+const mocks = vi.hoisted(() => ({
+    execute: vi.fn(),
+    formatDateFilter: vi.fn(() => 'formatted-time'),
+    openExternalLink: vi.fn(),
+    toast: {
+        warning: vi.fn(),
+        success: vi.fn(),
+        dismiss: vi.fn()
+    }
+}));
+
+vi.mock('../../service/webapi', () => ({
+    default: {
+        execute: (...args) => mocks.execute(...args)
+    }
+}));
+
+vi.mock('../../shared/utils', () => ({
+    formatDateFilter: (...args) => mocks.formatDateFilter(...args),
+    openExternalLink: (...args) => mocks.openExternalLink(...args)
+}));
+
+vi.mock('vue-sonner', () => ({
+    toast: mocks.toast
+}));
+
+vi.mock('vue-i18n', () => ({
+    useI18n: () => ({
+        t: (key) => key
+    })
+}));
+
+vi.mock('worker-timers', () => ({
+    setInterval: vi.fn(),
+    clearInterval: vi.fn(),
+    setTimeout: vi.fn(),
+    clearTimeout: vi.fn()
+}));
+
+function flushPromises() {
+    return new Promise((resolve) => setTimeout(resolve, 0));
+}
+
+import { useVrcStatusStore } from '../vrcStatus';
+
+describe('useVrcStatusStore.getVrcStatus', () => {
+    beforeEach(async () => {
+        mocks.execute.mockResolvedValue({
+            status: 200,
+            data: JSON.stringify({
+                page: { updated_at: '2026-01-01T00:00:00.000Z' },
+                status: { description: 'All Systems Operational' }
+            })
+        });
+
+        setActivePinia(createPinia());
+        useVrcStatusStore();
+        await flushPromises();
+        vi.clearAllMocks();
+    });
+
+    test('sets failed status when API returns non-200', async () => {
+        const store = useVrcStatusStore();
+        mocks.execute.mockResolvedValueOnce({
+            status: 503,
+            data: 'service unavailable'
+        });
+
+        await store.getVrcStatus();
+
+        expect(mocks.execute).toHaveBeenCalledWith({
+            url: 'https://status.vrchat.com/api/v2/status.json',
+            method: 'GET',
+            headers: { Referer: 'https://vrcx.app' }
+        });
+        expect(store.lastStatus).toBe('Failed to fetch VRC status');
+        expect(mocks.toast.warning).toHaveBeenCalledTimes(1);
+    });
+
+    test('fetches summary for incident status and appends component summary', async () => {
+        const store = useVrcStatusStore();
+        mocks.execute
+            .mockResolvedValueOnce({
+                status: 200,
+                data: JSON.stringify({
+                    page: { updated_at: '2026-01-02T00:00:00.000Z' },
+                    status: { description: 'Partial System Outage' }
+                })
+            })
+            .mockResolvedValueOnce({
+                status: 200,
+                data: JSON.stringify({
+                    components: [
+                        { name: 'API', status: 'major_outage' },
+                        { name: 'Website', status: 'operational' },
+                        { name: 'CDN', status: 'partial_outage' }
+                    ]
+                })
+            });
+
+        await store.getVrcStatus();
+        await flushPromises();
+
+        expect(mocks.execute).toHaveBeenCalledTimes(2);
+        expect(mocks.execute.mock.calls[1][0].url).toBe(
+            'https://status.vrchat.com/api/v2/summary.json'
+        );
+        expect(store.lastStatus).toBe('Partial System Outage');
+        expect(store.statusText).toBe('Partial System Outage: API, CDN');
+        expect(mocks.toast.warning).toHaveBeenCalled();
+    });
+
+    test('clears status when all systems are operational', async () => {
+        const store = useVrcStatusStore();
+        mocks.execute.mockResolvedValueOnce({
+            status: 200,
+            data: JSON.stringify({
+                page: { updated_at: '2026-01-03T00:00:00.000Z' },
+                status: { description: 'All Systems Operational' }
+            })
+        });
+
+        await store.getVrcStatus();
+
+        expect(mocks.execute).toHaveBeenCalledTimes(1);
+        expect(store.lastStatus).toBe('');
+        expect(store.statusText).toBe('');
+    });
+});

src/stores/__tests__/vrcxUpdater.test.js

@@ -0,0 +1,96 @@
+import { beforeEach, describe, expect, test, vi } from 'vitest';
+import { createPinia, setActivePinia } from 'pinia';
+
+const mocks = vi.hoisted(() => ({
+    configRepository: {
+        getString: vi.fn(),
+        setString: vi.fn()
+    },
+    changeLogRemoveLinks: vi.fn((value) => value),
+    toast: {
+        error: vi.fn(),
+        success: vi.fn(),
+        warning: vi.fn()
+    }
+}));
+
+vi.mock('../../service/config', () => ({
+    default: mocks.configRepository
+}));
+
+vi.mock('../../shared/utils', () => ({
+    changeLogRemoveLinks: (...args) => mocks.changeLogRemoveLinks(...args)
+}));
+
+vi.mock('vue-sonner', () => ({
+    toast: mocks.toast
+}));
+
+vi.mock('vue-i18n', () => ({
+    useI18n: () => ({
+        t: (key) => key
+    })
+}));
+
+function flushPromises() {
+    return new Promise((resolve) => setTimeout(resolve, 0));
+}
+
+import { useVRCXUpdaterStore } from '../vrcxUpdater';
+
+describe('useVRCXUpdaterStore.setAutoUpdateVRCX', () => {
+    beforeEach(async () => {
+        mocks.configRepository.getString.mockImplementation(
+            (key, defaultValue) => {
+                if (key === 'VRCX_autoUpdateVRCX') {
+                    return Promise.resolve('Off');
+                }
+                if (key === 'VRCX_id') {
+                    return Promise.resolve('test-vrcx-id');
+                }
+                if (key === 'VRCX_lastVRCXVersion') {
+                    return Promise.resolve('2026.1.0');
+                }
+                return Promise.resolve(defaultValue ?? '');
+            }
+        );
+        mocks.configRepository.setString.mockResolvedValue(undefined);
+
+        globalThis.AppApi = {
+            GetVersion: vi.fn().mockResolvedValue('2026.1.0')
+        };
+
+        setActivePinia(createPinia());
+        useVRCXUpdaterStore();
+        await flushPromises();
+        vi.clearAllMocks();
+    });
+
+    test('sets autoUpdateVRCX to Off, clears pending flag, and persists config', async () => {
+        const store = useVRCXUpdaterStore();
+        store.pendingVRCXUpdate = true;
+
+        await store.setAutoUpdateVRCX('Off');
+
+        expect(store.autoUpdateVRCX).toBe('Off');
+        expect(store.pendingVRCXUpdate).toBe(false);
+        expect(mocks.configRepository.setString).toHaveBeenCalledWith(
+            'VRCX_autoUpdateVRCX',
+            'Off'
+        );
+    });
+
+    test('updates autoUpdateVRCX for non-Off values and keeps pending flag', async () => {
+        const store = useVRCXUpdaterStore();
+        store.pendingVRCXUpdate = true;
+
+        await store.setAutoUpdateVRCX('Notify');
+
+        expect(store.autoUpdateVRCX).toBe('Notify');
+        expect(store.pendingVRCXUpdate).toBe(true);
+        expect(mocks.configRepository.setString).toHaveBeenCalledWith(
+            'VRCX_autoUpdateVRCX',
+            'Notify'
+        );
+    });
+});

src/stores/group.js

@@ -168,14 +168,11 @@ export const useGroupStore = defineStore('Group', () => {
         D.members = [];
         D.memberFilter = groupDialogFilterOptions.everyone;
         D.calendar = [];
-        const loadGroupRequest = forceRefresh
-            ? groupRequest.getGroup({
-                  groupId,
-                  includeRoles: false
-              })
-            : groupRequest.getCachedGroup({
-                  groupId
-              });
+        const loadGroupRequest = groupRequest.getGroup({
+            groupId,
+            includeRoles: false
+        });
+
         loadGroupRequest
             .catch((err) => {
                 D.loading = false;
@@ -962,6 +959,9 @@ export const useGroupStore = defineStore('Group', () => {
         }
     }
 
+    /**
+     *
+     */
     function clearGroupInstances() {
         groupInstances.value = [];
     }