Make tests work with repotools

Author: leeyi45

lib/buildtools/src/testing/utils.ts

@@ -109,7 +109,10 @@ export async function getTestConfiguration(directory: string, watch: boolean): P
 
   const packageJsonResult = await findPackageJson(directory);
   if (packageJsonResult === null) {
-    throw new Error(`Could not find a package.json for ${directory}`);
+    return {
+      severity: 'success',
+      config: null
+    };
   }
 
   const [type, jsonDir] = packageJsonResult;

lib/repotools/package.json

@@ -6,7 +6,8 @@
   "description": "Common library for tooling used by the modules repository",
   "devDependencies": {
     "@commander-js/extra-typings": "^13.0.0",
-    "typescript": "^5.8.2"
+    "typescript": "^5.8.2",
+    "vitest": "^3.2.3"
   },
   "dependencies": {
     "chalk": "^5.0.1",
@@ -19,6 +20,9 @@
     "./*": "./dist/*.js"
   },
   "scripts": {
-    "build": "tsc"
+    "build": "tsc --project ./tsconfig.prod.json",
+    "prepare": "yarn build",
+    "tsc": "tsc --project ./tsconfig.json",
+    "test": "vitest"
   }
 }

lib/repotools/src/__tests__/manifest.test.ts

@@ -1,9 +1,14 @@
+import pathlib from 'path';
 import { describe, expect, it, test, vi } from 'vitest';
-import { expectIsSuccess, testMocksDir } from '../../../buildtools/src/__tests__/fixtures.js';
 import { resolveAllTabs, resolveSingleTab } from '../manifest.js';
 
+const testMocksDir = pathlib.resolve(import.meta.dirname, '../../../__test_mocks__');
 vi.mock(import('../getGitRoot.js'));
 
+function expectSuccess(obj: unknown): asserts obj is 'success' {
+  expect(obj).toEqual('success');
+}
+
 describe('Test resolveSingleTab', () => {
   it('properly detects a tab with the src/index.tsx entrypoint', async () => {
     const resolved = await resolveSingleTab(`${testMocksDir}/tabs/tab0`);
@@ -30,8 +35,8 @@ describe('Test resolveSingleTab', () => {
 
 test('resolveAllTabs', async () => {
   const resolved = await resolveAllTabs(`${testMocksDir}/bundles`, `${testMocksDir}/tabs`);
+  expectSuccess(resolved.severity);
 
-  expectIsSuccess(resolved.severity);
   const { tabs } = resolved;
 
   expect(tabs).toHaveProperty('tab0');

lib/repotools/tsconfig.json

@@ -1,11 +1,9 @@
+// Repotools regular tsconfig
 {
   "extends": ["../tsconfig.json"],
   "include": ["./src"],
   "compilerOptions": {
-    "declaration": true,
     "module": "nodenext",
-    "moduleResolution": "nodenext",
-    "noEmit": false,
-    "outDir": "./dist"
+    "moduleResolution": "nodenext"
   }
 }

lib/repotools/tsconfig.prod.json

@@ -0,0 +1,11 @@
+// Repotools production tsconfig
+{
+  "extends": ["./tsconfig.json"],
+  "include": ["./src"],
+  "exclude": ["**/__tests__/**", "**/__mocks__/**"],
+  "compilerOptions": {
+    "declaration": true,
+    "noEmit": false,
+    "outDir": "./dist"
+  }
+}

lib/repotools/vitest.config.js

@@ -0,0 +1,18 @@
+// @ts-check
+import rootConfig from '../../vitest.config.js';
+
+/**
+ * @type {import('vitest/config').ViteUserConfig}
+ */
+const config = {
+  test: {
+    ...rootConfig.test,
+    name: 'Repo Tools',
+    root: import.meta.dirname,
+    include: ['**/__tests__/*.test.ts'],
+    watch: false,
+    projects: []
+  }
+};
+
+export default config;

src/bundles/ar/package.json

@@ -21,4 +21,4 @@
     "tsc": "buildtools tsc .",
     "test": "buildtools test --project ."
   }
-}
+}

src/bundles/arcade_2d/package.json

@@ -21,4 +21,4 @@
     "tsc": "buildtools tsc .",
     "test": "buildtools test --project ."
   }
-}
+}

src/bundles/binary_tree/package.json

@@ -17,4 +17,4 @@
     "lint": "buildtools lint .",
     "test": "buildtools test --project ."
   }
-}
+}

src/bundles/communication/package.json

@@ -22,4 +22,4 @@
     "tsc": "buildtools tsc .",
     "lint": "buildtools lint ."
   }
-}
+}