mirror of
https://github.com/mauriceboe/TREK.git
synced 2026-06-21 22:31:46 +00:00
f8fdb14627
navigator.onLine returns false transiently during service worker activation
(skipWaiting + clientsClaim), causing all repo refresh IIFEs to return null
immediately on first page load — leaving the UI with empty data until F5.
Fixes applied across all list repos (trip, day, place, packing, todo, budget,
reservation, accommodation, file):
- Drop navigator.onLine guard; let fetch fail naturally when truly offline
- Await all upsert calls (some were fire-and-forget, risking race conditions
against subsequent reads and silent swallowed failures)
- Return Promise.resolve(null) instead of Promise.resolve(fresh) in the
IDB-empty network path, so loadTrip's background refresh Promise.all
resolves null and skips set({trip}), preventing a spurious reference change
that was resetting the 1500ms splash timer
Tests updated: placeRepo and packingRepo "empty cache" tests now simulate
genuine network failure (HttpResponse.error) instead of relying on the
navigator.onLine guard that no longer exists; DashboardPage tests clear IDB
before each test and use a query-safe assertion after background refresh.
112 lines
3.6 KiB
TypeScript
112 lines
3.6 KiB
TypeScript
/**
|
|
* packingRepo unit tests.
|
|
*
|
|
* Online path: calls REST via MSW, writes result to Dexie.
|
|
* Offline path: returns Dexie cache, skips REST.
|
|
*/
|
|
import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
|
|
import 'fake-indexeddb/auto';
|
|
import { server } from '../../helpers/msw/server';
|
|
import { http, HttpResponse } from 'msw';
|
|
import { packingRepo } from '../../../src/repo/packingRepo';
|
|
import { offlineDb, clearAll } from '../../../src/db/offlineDb';
|
|
import { buildPackingItem } from '../../helpers/factories';
|
|
|
|
beforeEach(async () => {
|
|
await clearAll();
|
|
Object.defineProperty(navigator, 'onLine', { value: true, writable: true, configurable: true });
|
|
});
|
|
|
|
afterEach(() => {
|
|
vi.restoreAllMocks();
|
|
});
|
|
|
|
describe('packingRepo.list', () => {
|
|
it('online — fetches from REST and caches in Dexie', async () => {
|
|
const item = buildPackingItem({ trip_id: 1 });
|
|
server.use(
|
|
http.get('/api/trips/1/packing', () => HttpResponse.json({ items: [item] })),
|
|
);
|
|
|
|
const result = await packingRepo.list(1);
|
|
expect(result.items).toHaveLength(1);
|
|
expect(result.items[0].id).toBe(item.id);
|
|
|
|
await new Promise(r => setTimeout(r, 0));
|
|
const cached = await offlineDb.packingItems.where('trip_id').equals(1).toArray();
|
|
expect(cached).toHaveLength(1);
|
|
expect(cached[0].id).toBe(item.id);
|
|
});
|
|
|
|
it('offline — returns Dexie cache without REST call', async () => {
|
|
Object.defineProperty(navigator, 'onLine', { value: false });
|
|
|
|
const item = buildPackingItem({ trip_id: 1 });
|
|
await offlineDb.packingItems.put(item);
|
|
|
|
let restCalled = false;
|
|
server.use(
|
|
http.get('/api/trips/1/packing', () => {
|
|
restCalled = true;
|
|
return HttpResponse.json({ items: [] });
|
|
}),
|
|
);
|
|
|
|
const result = await packingRepo.list(1);
|
|
expect(result.items).toHaveLength(1);
|
|
expect(result.items[0].id).toBe(item.id);
|
|
expect(restCalled).toBe(false);
|
|
});
|
|
|
|
it('offline — returns empty array when nothing cached and network fails', async () => {
|
|
server.use(
|
|
http.get('/api/trips/99/packing', () => HttpResponse.error()),
|
|
);
|
|
const result = await packingRepo.list(99);
|
|
expect(result.items).toHaveLength(0);
|
|
});
|
|
});
|
|
|
|
describe('packingRepo.create', () => {
|
|
it('writes item optimistically to Dexie immediately', async () => {
|
|
const result = await packingRepo.create(1, { name: 'Sunscreen' });
|
|
expect(result.item.name).toBe('Sunscreen');
|
|
// tempId is negative (-(Date.now()))
|
|
expect(result.item.id).toBeLessThan(0);
|
|
|
|
const cached = await offlineDb.packingItems.where('trip_id').equals(1).toArray();
|
|
expect(cached).toHaveLength(1);
|
|
expect(cached[0].name).toBe('Sunscreen');
|
|
});
|
|
});
|
|
|
|
describe('packingRepo.update', () => {
|
|
it('writes optimistic update to Dexie immediately', async () => {
|
|
const original = buildPackingItem({ trip_id: 1, name: 'Jacket', checked: 0 });
|
|
await offlineDb.packingItems.put(original);
|
|
|
|
const result = await packingRepo.update(1, original.id, { checked: true });
|
|
expect(result.item.checked).toBeTruthy();
|
|
|
|
const cached = await offlineDb.packingItems.get(original.id);
|
|
expect(cached!.checked).toBeTruthy();
|
|
});
|
|
});
|
|
|
|
describe('packingRepo.delete', () => {
|
|
it('calls REST and removes from Dexie', async () => {
|
|
const item = buildPackingItem({ trip_id: 1 });
|
|
await offlineDb.packingItems.put(item);
|
|
|
|
server.use(
|
|
http.delete(`/api/trips/1/packing/${item.id}`, () => HttpResponse.json({ success: true })),
|
|
);
|
|
|
|
await packingRepo.delete(1, item.id);
|
|
|
|
await new Promise(r => setTimeout(r, 0));
|
|
const cached = await offlineDb.packingItems.get(item.id);
|
|
expect(cached).toBeUndefined();
|
|
});
|
|
});
|