test(assets): resolve the refresh before the stale loadMore in race tests

Two of the flat-output race tests resolved the loadMore before the refresh.
In that ordering a successful refresh overwrites every piece of state the
stale page touched, so both tests passed with the epoch guard removed and
proved nothing about it.

Flip both to resolve the refresh first, where the stale continuation lands
on an already-reset list, and retarget the first test's assertion to the
offset (observed via the next request's `offset` argument) so it covers the
double-advance axis rather than duplicating the sibling test's list check.

Removing the epoch guard now fails all four guard tests; previously two.

Author: mattmillerai

src/stores/assetsStore.test.ts

@@ -2410,7 +2410,7 @@ describe('assetsStore - Flat Output Assets (cloud-only)', () => {
       expect(store.flatOutputAssets.map((a) => a.id)).toEqual(['fresh-1'])
     })
 
-    it('a stale loadMore that resolves before the refresh does not corrupt seenIds', async () => {
+    it('does not double-advance the offset with a discarded loadMore page', async () => {
       const { store } = await setupStoreWithFirstPage()
       const loadMorePage = deferredPage()
       const refreshPage = deferredPage()
@@ -2422,23 +2422,24 @@ describe('assetsStore - Flat Output Assets (cloud-only)', () => {
       const loadMoreResult = store.loadMoreFlatOutputs()
       const refreshResult = store.updateFlatOutputs()
 
-      // The stale loadMore settles first, before the refresh has replaced the
-      // list. Its page must be discarded rather than folded into seenIds.
-      loadMorePage.resolve(makePage([makeAsset('extra-1', 'extra.png')]))
       refreshPage.resolve(
         makePage([makeAsset('fresh-1', 'fresh.png')], { hasMore: true })
       )
+      loadMorePage.resolve(makePage([makeAsset('extra-1', 'extra.png')]))
       await Promise.all([loadMoreResult, refreshResult])
 
-      expect(store.flatOutputAssets.map((a) => a.id)).toEqual(['fresh-1'])
-
-      // If the discarded loadMore had leaked 'extra-1' into seenIds, this
-      // legitimate next page would be filtered out and never shown.
+      // The refreshed list holds one asset, so the next page starts at 1. A
+      // discarded page that still advanced the offset would skip a row here.
       vi.mocked(assetService.getAssetsPageByTag).mockResolvedValueOnce(
         makePage([makeAsset('extra-1', 'extra.png')])
       )
       await store.loadMoreFlatOutputs()
 
+      expect(assetService.getAssetsPageByTag).toHaveBeenLastCalledWith(
+        'output',
+        true,
+        { limit: FLAT_OUTPUT_PAGE_SIZE, offset: 1 }
+      )
       expect(store.flatOutputAssets.map((a) => a.id)).toEqual([
         'fresh-1',
         'extra-1'
@@ -2457,12 +2458,12 @@ describe('assetsStore - Flat Output Assets (cloud-only)', () => {
       const loadMoreResult = store.loadMoreFlatOutputs()
       const refreshResult = store.updateFlatOutputs()
 
-      loadMorePage.resolve(
-        makePage([makeAsset('extra-1', 'extra.png')], { hasMore: true })
-      )
       refreshPage.resolve(
         makePage([makeAsset('fresh-1', 'fresh.png')], { hasMore: false })
       )
+      loadMorePage.resolve(
+        makePage([makeAsset('extra-1', 'extra.png')], { hasMore: true })
+      )
       await Promise.all([loadMoreResult, refreshResult])
 
       expect(store.flatOutputAssets.map((a) => a.id)).toEqual(['fresh-1'])