Fix manifest tests (#58)

* both manifests

* fixed manifest.test

* timeout-minutes: 15

.github/workflows/main.yaml

@@ -13,7 +13,7 @@ concurrency:
 jobs:
   test:
     runs-on: ubuntu-latest
-    timeout-minutes: 9
+    timeout-minutes: 15
     steps:
       - uses: actions/checkout@v4
       - uses: oven-sh/setup-bun@v1

src/config.js

@@ -1,7 +1,7 @@
 const config = {
   manifests: {
     release: 'https://raw.githubusercontent.com/commaai/openpilot/release3/system/hardware/tici/agnos.json',
-    // master: 'https://raw.githubusercontent.com/commaai/openpilot/master/system/hardware/tici/agnos.json',
+    master: 'https://raw.githubusercontent.com/commaai/openpilot/master/system/hardware/tici/agnos.json',
   },
 }
 

src/utils/manifest.test.js

@@ -14,31 +14,14 @@ async function getImageWorker() {
     imageWorker.init()
   })
 
+  vi.resetModules() // this makes the import be reevaluated on each call
   await import('./../workers/image.worker')
 
   return imageWorker
 }
 
 for (const [branch, manifestUrl] of Object.entries(config.manifests)) {
   describe(`${branch} manifest`, async () => {
-    const imageWorkerFileHandler = {
-      getFile: vi.fn(),
-      createWritable: vi.fn().mockImplementation(() => ({
-        write: vi.fn(),
-        close: vi.fn(),
-      })),
-    }
-
-    globalThis.navigator = {
-      storage: {
-        getDirectory: () => ({
-          getFileHandle: () => imageWorkerFileHandler,
-        })
-      }
-    }
-
-    const imageWorker = await getImageWorker()
-
     const images = await getManifest(manifestUrl)
 
     // Check all images are present
@@ -60,13 +43,31 @@ for (const [branch, manifestUrl] of Object.entries(config.manifests)) {
         }
 
         test('image and checksum', async () => {
+          const imageWorkerFileHandler = {
+            getFile: vi.fn(),
+            createWritable: vi.fn().mockImplementation(() => ({
+              write: vi.fn(),
+              close: vi.fn(),
+            })),
+          }
+
+          globalThis.navigator = {
+            storage: {
+              getDirectory: () => ({
+                getFileHandle: () => imageWorkerFileHandler,
+              })
+            }
+          }
+
           imageWorkerFileHandler.getFile.mockImplementation(async () => {
             const response = await fetch(image.archiveUrl)
             expect(response.ok, 'to be uploaded').toBe(true)
 
             return response.blob()
           })
 
+          const imageWorker = await getImageWorker()
+
           await imageWorker.unpackImage(image)
         }, 8 * 60 * 1000)
       })