Merge pull request #25 from shiftonetothree/CI-test

Improve test case stability in GitHub action

Author: RaSan147

test/features/stability.test.ts

@@ -2,15 +2,17 @@ import { testEachModel } from "../env";
 import { createModel } from "../utils";
 import { makeTestSound } from "../make-test-sound";
 
+const times = 500;
+
 testEachModel(
-    "speak 1000 times",
+    `speak ${times} times`,
     async ({ app, model: { modelJsonWithUrl, hitTests } }) => {
         const model = await createModel(modelJsonWithUrl);
         model.update(100);
         app.stage.addChild(model);
         app.renderer.resize(model.width, model.height);
         app.render();
-        for (let i = 0; i < 1000; i++) {
+        for (let i = 0; i < times; i++) {
             await new Promise<void>((resolve, reject) => {
                 const duration = Math.random() * 0.01;
                 model.speak(makeTestSound(undefined, 0.1, duration), {

vite.config.ts

@@ -100,6 +100,7 @@ export default defineConfig(({ command, mode }) => {
         ],
         test: {
             include: ["**/*.test.ts", "**/*.test.js"],
+            testTimeout: 10 * 1000,
             browser: {
                 enabled: true,
                 name: "chrome",
@@ -120,6 +121,18 @@ export default defineConfig(({ command, mode }) => {
                     override async sort(files: Parameters<BaseSequencer["sort"]>[0]) {
                         files = await super.sort(files);
 
+                        // stability tests are slow, it will drag others leg behind
+                        // so we put them at the last
+                        const stabilityTestFiles: typeof files = [];
+
+                        files = files.filter(([project, file]) => {
+                            if (file.includes("stability")) {
+                                stabilityTestFiles.push([project, file]);
+                                return false;
+                            }
+                            return true;
+                        });
+
                         const bundleTestFiles: typeof files = [];
 
                         files = files.filter(([project, file]) => {
@@ -130,7 +143,7 @@ export default defineConfig(({ command, mode }) => {
                             return true;
                         });
 
-                        return [...files, ...bundleTestFiles];
+                        return [...files, ...stabilityTestFiles];
                     }
                 },
             },