Merge pull request #1554 from thunderstore-io/ci-improvements

Improve the coverage of CI pipeline

Author: Oksamies

.github/workflows/test.yml

@@ -29,15 +29,15 @@ jobs:
     name: Run pre-commit
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
         with:
           submodules: false
       - name: Setup Python 3.8
         id: setup-python
         uses: actions/setup-python@v2
         with:
           python-version: "3.8"
-      - uses: actions/setup-node@v2
+      - uses: actions/setup-node@v4
         with:
           node-version: "20"
       - name: Get node version
@@ -99,35 +99,66 @@ jobs:
       VITE_AUTH_RETURN_URL: ${{ vars.NEXT_PUBLIC_SITE_URL }}
     steps:
       - uses: actions/checkout@v4
-      - name: Set up Node ${{ needs.get-node-version.outputs.node-version }}
-        uses: actions/setup-node@v3
+      - name: Set up Node 24
+        uses: actions/setup-node@v4
         with:
-          node-version: ${{ needs.get-node-version.outputs.node-version }}
+          node-version: "24"
           cache: "yarn"
       - name: Set fontawesome token
         run: |
           npm config set "@fortawesome:registry" https://npm.fontawesome.com/
           npm config set "//npm.fontawesome.com/:_authToken" ${{ secrets.NPM_FA_TOKEN }}
       - name: Install Yarn dependencies
         run: yarn install --frozen-lockfile
+      - name: Build @thunderstore/beta-switch
+        run: yarn workspace @thunderstore/beta-switch build
       - name: Build @thunderstore/cyberstorm
         run: yarn workspace @thunderstore/cyberstorm build
+      - name: Build @thunderstore/cyberstorm-forms
+        run: yarn workspace @thunderstore/cyberstorm-forms build
+      - name: Build and test @thunderstore/cyberstorm-remix
+        run: |
+          yarn workspace @thunderstore/cyberstorm-remix build
+          yarn workspace @thunderstore/cyberstorm-remix test run
+      # Enable once build process has been fixed.
+      # - name: Build @thunderstore/cyberstorm-storybook
+        # run: yarn workspace @thunderstore/cyberstorm-storybook build
       - name: Build @thunderstore/dapper
         run: yarn workspace @thunderstore/dapper build
       - name: Build @thunderstore/dapper-fake
         run: yarn workspace @thunderstore/dapper-fake build
-      - name: Build @thunderstore/thunderstore-api
-        run: yarn workspace @thunderstore/thunderstore-api build
-      - name: Test @thunderstore/thunderstore-api
-        run: yarn workspace @thunderstore/thunderstore-api test
-      - name: Build @thunderstore/dapper-ts
-        run: yarn workspace @thunderstore/dapper-ts build
-      - name: Test @thunderstore/dapper-ts
-        run: yarn workspace @thunderstore/dapper-ts test
+      - name: Build and test @thunderstore/dapper-ts
+        run: |
+          yarn workspace @thunderstore/dapper-ts build
+          yarn workspace @thunderstore/dapper-ts test
+      - name: Build and test @thunderstore/graph-system
+        run: |
+          yarn workspace @thunderstore/graph-system build
+          yarn workspace @thunderstore/graph-system test run
+      - name: Build @thunderstore/react-dnd
+        run: yarn workspace @thunderstore/react-dnd build
+      - name: Build and test @thunderstore/thunderstore-api
+        run: |
+          yarn workspace @thunderstore/thunderstore-api build
+          yarn workspace @thunderstore/thunderstore-api test
+      - name: Build @thunderstore/ts-api-react
+        run: yarn workspace @thunderstore/ts-api-react build
+      - name: Build @thunderstore/ts-api-react-actions
+        run: yarn workspace @thunderstore/ts-api-react-actions build
+      - name: Build @thunderstore/ts-api-react-forms
+        run: yarn workspace @thunderstore/ts-api-react-forms build
+      - name: Build and test @thunderstore/ts-uploader
+        # Add running tests once they've been fixed.
+        run: |
+          yarn workspace @thunderstore/ts-uploader build
+      - name: Build @thunderstore/ts-uploader-react
+        run: yarn workspace @thunderstore/ts-uploader-react build
+      - name: Build and test @thunderstore/typed-event-emitter
+        run: |
+          yarn workspace @thunderstore/typed-event-emitter build
+          yarn workspace @thunderstore/typed-event-emitter test
       - name: Build @thunderstore/use-promise
         run: yarn workspace @thunderstore/use-promise build
-      - name: Build @thunderstore/cyberstorm-remix
-        run: yarn workspace @thunderstore/cyberstorm-remix build
 
   chromatic-deployment:
     runs-on: ubuntu-latest
@@ -142,7 +173,7 @@ jobs:
         with:
           fetch-depth: 0
       - name: Set up Node ${{ needs.get-node-version.outputs.node-version }}
-        uses: actions/setup-node@v3
+        uses: actions/setup-node@v4
         with:
           node-version: ${{ needs.get-node-version.outputs.node-version }}
           cache: "yarn"

packages/ts-api-react-forms/tsconfig.json

@@ -27,8 +27,7 @@
       {
         "name": "typescript-plugin-css-modules"
       }
-    ],
-    "types": ["react/next"]
+    ]
   },
   "include": ["./src/**/*.tsx", "./src/**/*.ts"],
   "exclude": ["node_modules"]

packages/ts-uploader-react/tsconfig.json

@@ -1,7 +1,7 @@
 {
   "compilerOptions": {
-    "target": "es5",
-    "module": "ES2020",
+    "target": "ESNext",
+    "module": "ESNext",
     "skipLibCheck": true,
     "moduleResolution": "node",
     "removeComments": true,

packages/ts-uploader/src/workers/__tests__/MD5WorkerManager.test.ts

@@ -3,12 +3,22 @@ import { MD5WorkerManager } from "../MD5WorkerManager";
 
 describe("MD5WorkerManager", () => {
   let manager: MD5WorkerManager;
-  const mockWorker: Worker = {
-    onmessage: null,
-    // @ts-expect-error Mocking Worker methods
+
+  const mockWorker = {
+    // event target bits that Worker requires
+    addEventListener: vi.fn(),
+    removeEventListener: vi.fn(),
+    dispatchEvent: vi.fn(),
+
+    // handler properties
+    onmessage: null as Worker["onmessage"],
+    onerror: null as Worker["onerror"],
+    onmessageerror: null as Worker["onmessageerror"],
+
+    // the methods you actually use
     postMessage: vi.fn(),
     terminate: vi.fn(),
-  };
+  } satisfies Partial<Worker> as Worker;
 
   beforeEach(() => {
     // Mock Worker