Merge pull request #1555 from thunderstore-io/tests-and-coverage

Tests and coverage

Author: Oksamies

.github/workflows/test.yml

@@ -39,7 +39,7 @@ jobs:
           python-version: "3.8"
       - uses: actions/setup-node@v4
         with:
-          node-version: "20"
+          node-version: "24"
       - name: Get node version
         id: node-version
         run: echo "::set-output name=node-version::$(node --version)"
@@ -88,10 +88,10 @@ jobs:
 #        if: github.event_name != 'pull_request'
 #        run: IMAGE_TAG=${GITHUB_REF##*/} docker compose -f docker-compose.build.yml push
 
-  test:
-    name: Test
+  build:
+    name: Build
     runs-on: ubuntu-latest
-    needs: get-node-version
+    needs: [get-node-version, pre-commit]
     env:
       VITE_SITE_URL: ${{ vars.NEXT_PUBLIC_SITE_URL }}
       VITE_API_URL: ${{ vars.NEXT_PUBLIC_API_URL }}
@@ -116,53 +116,94 @@ jobs:
         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/cyberstorm-remix
+        run: yarn workspace @thunderstore/cyberstorm-remix build
+      - name: Build @thunderstore/storybook
+        run: yarn workspace @thunderstore/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 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/dapper-ts
+        run: yarn workspace @thunderstore/dapper-ts build
+      - name: Build @thunderstore/graph-system
+        run: yarn workspace @thunderstore/graph-system build
       - 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/thunderstore-api
+        run: yarn workspace @thunderstore/thunderstore-api build
       - 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
+        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/typed-event-emitter
+        run: yarn workspace @thunderstore/typed-event-emitter build
       - name: Build @thunderstore/use-promise
         run: yarn workspace @thunderstore/use-promise build
 
+  test:
+    name: Test
+    runs-on: ubuntu-latest
+    needs: [get-node-version, pre-commit, build]
+    env:
+      VITE_SITE_URL: ${{ vars.NEXT_PUBLIC_SITE_URL }}
+      VITE_API_URL: ${{ vars.NEXT_PUBLIC_API_URL }}
+      VITE_AUTH_BASE_URL: ${{ vars.NEXT_PUBLIC_SITE_URL }}
+      VITE_AUTH_RETURN_URL: ${{ vars.NEXT_PUBLIC_SITE_URL }}
+    steps:
+      - name: "Checkout"
+        uses: actions/checkout@v4
+      - name: "Setup Python 3.10"
+        id: setup-python
+        uses: actions/setup-python@v5
+        with:
+          python-version: "3.10"
+      - name: Set up Node 24
+        uses: actions/setup-node@v4
+        with:
+          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 }}
+      - uses: snok/install-poetry@v1
+        with:
+          version: 1.4.1
+          virtualenvs-create: true
+          virtualenvs-in-project: true
+      - name: Cache poetry venv
+        id: cache-poetry
+        uses: actions/cache@v4
+        with:
+          path: tools/test-ci/.venv
+          key: "poetry-${{ runner.os }}-\
+            ${{ steps.setup-python.outputs.python-version }}-\
+            ${{ hashFiles('tools/test-ci/poetry.lock') }}"
+          restore-keys: |
+            poetry-${{ runner.os }}-${{ steps.setup-python.outputs.python-version }}-
+      - name: Install poetry dependencies
+        run: |
+          cd tools/test-ci/
+          poetry install
+      - name: Run testing script
+        run: |
+          cd tools/test-ci/
+          poetry run python run_ci_script.py
+      - name: Upload results to Codecov
+        uses: codecov/codecov-action@v5
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}
+
   chromatic-deployment:
     runs-on: ubuntu-latest
-    needs: ["test", "pre-commit", "get-node-version"]
+    needs: [get-node-version, pre-commit, build, test]
     strategy:
       matrix:
         projects: [

.gitignore

@@ -3,3 +3,4 @@
 **/yarn-error.log
 .npmrc
 /.env.development
+/coverage

README.md

@@ -1,5 +1,7 @@
 # Thunderstore UI
 
+[![codecov](https://codecov.io/gh/thunderstore-io/thunderstore-ui/branch/master/graph/badge.svg)](https://codecov.io/gh/thunderstore-io/thunderstore-ui)
+
 Monorepo containing Remix frontend for [thunderstore.io](https://thunderstore.io)
 and reusable UI components.
 

apps/cyberstorm-remix/app/c/community.tsx

@@ -22,7 +22,7 @@ import { faBook, faDownload } from "@fortawesome/free-solid-svg-icons";
 import { faDiscord } from "@fortawesome/free-brands-svg-icons";
 import { faArrowUpRight } from "@fortawesome/pro-solid-svg-icons";
 import { DapperTs } from "@thunderstore/dapper-ts";
-import { OutletContextShape } from "../root";
+import { type OutletContextShape } from "../root";
 import {
   getPublicEnvVariables,
   getSessionTools,

apps/cyberstorm-remix/app/c/tabs/PackageSearch/PackageSearch.tsx

@@ -6,8 +6,8 @@ import {
   getPublicEnvVariables,
   getSessionTools,
 } from "cyberstorm/security/publicEnvVariables";
-import { OutletContextShape } from "~/root";
-import { Route } from "./+types/PackageSearch";
+import { type OutletContextShape } from "~/root";
+import type { Route } from "./+types/PackageSearch";
 
 export async function loader({ params, request }: Route.LoaderArgs) {
   if (params.communityId) {

apps/cyberstorm-remix/app/commonComponents/Markdown/Sanitize.ts

@@ -1,5 +1,5 @@
-import { Schema, sanitize } from "hast-util-sanitize";
-import { type Nodes } from "hast";
+import { type Schema, sanitize } from "hast-util-sanitize";
+import type { Nodes } from "hast";
 
 /**
  * Based on:

apps/cyberstorm-remix/app/p/dependants/Dependants.tsx

@@ -8,13 +8,13 @@ import "./Dependants.css";
 import { PackageSearch } from "~/commonComponents/PackageSearch/PackageSearch";
 import { DapperTs } from "@thunderstore/dapper-ts";
 import { PackageOrderOptions } from "../../commonComponents/PackageSearch/components/PackageOrder";
-import { OutletContextShape } from "../../root";
+import { type OutletContextShape } from "../../root";
 import { PageHeader } from "~/commonComponents/PageHeader/PageHeader";
 import {
   getPublicEnvVariables,
   getSessionTools,
 } from "cyberstorm/security/publicEnvVariables";
-import { Route } from "./+types/Dependants";
+import type { Route } from "./+types/Dependants";
 import { Suspense } from "react";
 
 export async function loader({ params, request }: Route.LoaderArgs) {

apps/cyberstorm-remix/app/p/packageEdit.tsx

@@ -17,7 +17,7 @@ import {
 } from "@thunderstore/thunderstore-api";
 import { formatToDisplayName } from "@thunderstore/cyberstorm/src/utils/utils";
 import { DapperTs } from "@thunderstore/dapper-ts";
-import { OutletContextShape } from "~/root";
+import { type OutletContextShape } from "~/root";
 import {
   getPublicEnvVariables,
   getSessionTools,

apps/cyberstorm-remix/app/p/packageListing.tsx

@@ -72,7 +72,7 @@ import {
   formatToDisplayName,
 } from "@thunderstore/cyberstorm/src/utils/utils";
 import { DapperTs } from "@thunderstore/dapper-ts";
-import { OutletContextShape } from "~/root";
+import { type OutletContextShape } from "~/root";
 import { CopyButton } from "~/commonComponents/CopyButton/CopyButton";
 import {
   getPublicEnvVariables,

apps/cyberstorm-remix/app/p/tabs/Required/Required.tsx

@@ -4,7 +4,7 @@ import { Await, LoaderFunctionArgs } from "react-router";
 import { useLoaderData, useOutletContext } from "react-router";
 import { ListingDependency } from "~/commonComponents/ListingDependency/ListingDependency";
 import { DapperTs } from "@thunderstore/dapper-ts";
-import { OutletContextShape } from "~/root";
+import { type OutletContextShape } from "~/root";
 import {
   getPublicEnvVariables,
   getSessionTools,