Fix tests and lint

.github/workflows/ci.yaml

@@ -14,28 +14,11 @@ jobs:
         with:
           node-version: "lts/*"
 
-      - run: npm install
+      - name: Install Dependencies
+        run: npm install
 
-      - run: npm test
-        working-directory: npm/cli-darwin-arm64
-        env: { NODE_AUTH_TOKEN: "${{ secrets.NPM_TOKEN }}" }
+      - name: Lint
+        run: npm run lint
 
-      - name: Publish @yaakapp/cli-darwin-x64
-        run: npm publish --provenance --access public
-        working-directory: npm/cli-darwin-x64
-        env: { NODE_AUTH_TOKEN: "${{ secrets.NPM_TOKEN }}" }
-
-      - name: Publish @yaakapp/cli-linux-x64
-        run: npm publish --provenance --access public
-        working-directory: npm/cli-linux-x64
-        env: { NODE_AUTH_TOKEN: "${{ secrets.NPM_TOKEN }}" }
-
-      - name: Publish @yaakapp/cli-win32-x64
-        run: npm publish --provenance --access public
-        working-directory: npm/cli-win32-x64
-        env: { NODE_AUTH_TOKEN: "${{ secrets.NPM_TOKEN }}" }
-
-      - name: Publish @yaakapp/cli
-        run: npm publish --provenance --access public
-        working-directory: npm/cli
-        env: { NODE_AUTH_TOKEN: "${{ secrets.NPM_TOKEN }}" }
+      - name: Run Tests
+        run: npm test

package.json

@@ -4,10 +4,12 @@
   "private": true,
   "scripts": {
     "build": "node scripts/build-plugins.cjs",
-    "test": "vitest run"
+    "test": "vitest run",
+    "lint": "tsc"
   },
   "devDependencies": {
     "jsonpath": "^1.1.1",
-    "vitest": "^2.0.4"
+    "vitest": "^2.0.4",
+    "typescript": "^5.5.2"
   }
 }

plugins/importer-openapi/tests/index.test.ts

@@ -1,21 +1,24 @@
+import { Context } from '@yaakapp/api';
 import * as fs from 'node:fs';
 import * as path from 'node:path';
 import { describe, expect, test } from 'vitest';
 import { pluginHookImport } from '../src';
 
+const ctx = {} as Context;
+
 describe('importer-openapi', () => {
   const p = path.join(__dirname, 'fixtures');
   const fixtures = fs.readdirSync(p);
 
   test('Skips invalid file', async () => {
-    const imported = await pluginHookImport({}, '{}');
+    const imported = await pluginHookImport(ctx, '{}');
     expect(imported).toBeUndefined();
   })
 
   for (const fixture of fixtures) {
     test('Imports ' + fixture, async () => {
       const contents = fs.readFileSync(path.join(p, fixture), 'utf-8');
-      const imported = await pluginHookImport({}, contents);
+      const imported = await pluginHookImport(ctx, contents);
       expect(imported?.resources.workspaces).toEqual([
         expect.objectContaining({
           name: 'Swagger Petstore - OpenAPI 3.0',

plugins/importer-postman/tests/index.test.ts

@@ -1,29 +1,19 @@
+import { Context, Model } from '@yaakapp/api';
 import * as fs from 'node:fs';
 import * as path from 'node:path';
-import { afterEach, beforeEach, describe, expect, test, vi } from 'vitest';
-import { Model } from '../../../src-web/lib/models';
+import { describe, expect, test } from 'vitest';
 import { pluginHookImport } from '../src';
 
-let originalRandom = Math.random;
+const ctx = {} as Context;
 
 describe('importer-postman', () => {
-  beforeEach(() => {
-    let i = 0;
-    // Psuedo-random number generator to ensure consistent ID generation
-    Math.random = vi.fn(() => ((i++ * 1000) % 133) / 100);
-  });
-
-  afterEach(() => {
-    Math.random = originalRandom;
-  });
-
   const p = path.join(__dirname, 'fixtures');
   const fixtures = fs.readdirSync(p);
 
   for (const fixture of fixtures) {
     test('Imports ' + fixture, () => {
       const contents = fs.readFileSync(path.join(p, fixture), 'utf-8');
-      const imported = pluginHookImport({}, contents);
+      const imported = pluginHookImport(ctx, contents);
       const folder0 = newId('folder');
       const folder1 = newId('folder');
       expect(imported).toEqual({

plugins/importer-yaak/tests/index.test.ts

@@ -1,7 +1,8 @@
+import { Context } from '@yaakapp/api';
 import { describe, expect, test } from 'vitest';
 import { pluginHookImport } from '../src';
 
-const ctx = {};
+const ctx = {} as Context;
 
 describe('importer-yaak', () => {
   test('Skips invalid imports', () => {