Refactor CLI tests to use vitest (#1075)

Author: willosborne

cli/jest.config.js

@@ -14,7 +14,7 @@
     "watch": "node watch.mjs",
     "copy-calm-schema": "copyfiles \"../calm/draft/2024-10/meta/*\" dist/calm/",
     "copy-docify-templates": "copyfiles \"../shared/src/docify/template-bundles/**/*\" dist --up 4",
-    "test": "jest",
+    "test": "vitest run",
     "lint": "eslint src",
     "lint-fix": "eslint src --fix",
     "dependency-check": "dependency-check --project 'calm-cli' --scan . --out ./dependency-check-report --format ALL --suppression ../.github/node-cve-ignore-list.xml"

cli/package.json

@@ -9,7 +9,7 @@ import axios from 'axios';
 
 
 // Mock axios
-jest.mock('axios');
+vi.mock('axios');
 
 const execPromise = util.promisify(exec);
 
@@ -19,7 +19,7 @@ describe('CLI Integration Tests', () => {
     const millisPerSecond = 1000;
     const integrationTestPrefix = 'calm-test';
     const projectRoot = __dirname;
-    jest.setTimeout(30 * millisPerSecond);
+    vi.setConfig({ testTimeout: 30 * millisPerSecond });
 
     beforeAll(async () => {
         tempDir = fs.mkdtempSync(path.join(os.tmpdir(), integrationTestPrefix));
@@ -32,46 +32,42 @@ describe('CLI Integration Tests', () => {
         }
     }, millisPerSecond * 20);
 
-    test('shows help if no arguments provided', (done) => {
+    test('shows help if no arguments provided', async () => {
         const noArgCommand = 'calm';
         exec(noArgCommand, (error, _stdout, stderr) => {
             expect(error).not.toBeNull();
             expect(stderr).toContain('A set of tools for interacting with the Common Architecture Language Model');
             expect(stderr).toContain('Usage:');
-            done();
         });
     });
 
-    test('shows help if -h provided', (done) => {
+    test('shows help if -h provided', async () => {
         const helpShortFlagCommand = 'calm -h';
         exec(helpShortFlagCommand, (_error, stdout, _stderr) => {
             expect(stdout).toContain('A set of tools for interacting with the Common Architecture Language Model');
             expect(stdout).toContain('Usage:');
-            done();
         });
     });
 
-    test('shows help if --help provided', (done) => {
+    test('shows help if --help provided', async () => {
         const helpLongFlagCommand = 'calm --help';
         exec(helpLongFlagCommand, (_error, stdout, _stderr) => {
             expect(stdout).toContain('A set of tools for interacting with the Common Architecture Language Model');
             expect(stdout).toContain('Usage:');
-            done();
         });
     });
 
-    test('example validate command - outputting JSON to stdout', (done) => {
+    test('example validate command - outputting JSON to stdout', async () => {
         const exampleValidateCommand = 'calm validate -p ../calm/pattern/api-gateway.json -a ../calm/samples/api-gateway-architecture.json';
         exec(exampleValidateCommand, (_error, stdout, _stderr) => {
             const parsedOutput = JSON.parse(stdout);
             const expectedFilePath = path.join(__dirname, '../test_fixtures/validate_output.json');
             const expectedJson = JSON.parse(fs.readFileSync(expectedFilePath, 'utf-8'));
             expect(parsedOutput).toEqual(expectedJson);
-            done();
         });
     });
 
-    test('example validate command - outputting JSON to file', (done) => {
+    test('example validate command - outputting JSON to file', async () => {
         const targetOutputFile = path.join(tempDir, 'validate-output.json');
         const exampleValidateCommand = `calm validate -p ../calm/pattern/api-gateway.json -a ../calm/samples/api-gateway-architecture.json -o ${targetOutputFile}`;
         exec(exampleValidateCommand, (_error, _stdout, _stderr) => {
@@ -84,12 +80,10 @@ describe('CLI Integration Tests', () => {
             const expectedJson = JSON.parse(fs.readFileSync(expectedFilePath, 'utf-8'));
 
             expect(parsedOutput).toEqual(expectedJson);
-
-            done();
         });
     });
 
-    test('example validate command - outputting JUNIT to stdout', (done) => {
+    test('example validate command - outputting JUNIT to stdout', async () => {
         const exampleValidateCommand = 'calm validate -p ../calm/pattern/api-gateway.json -a ../calm/samples/api-gateway-architecture.json -f junit';
         exec(exampleValidateCommand, async (_error, stdout, _stderr) => {
             const parsedOutput = await parseStringPromise(stdout);
@@ -99,11 +93,10 @@ describe('CLI Integration Tests', () => {
             const expectedXml = await parseStringPromise(expectedXmlString);
 
             expect(parsedOutput).toEqual(expectedXml);
-            done();
         });
     });
 
-    test('example validate command - outputting JUNIT to file', (done) => {
+    test('example validate command - outputting JUNIT to file', async () => {
         const targetOutputFile = path.join(tempDir, 'validate-output.xml');
         const exampleValidateCommand = `calm validate -p ../calm/pattern/api-gateway.json -a ../calm/samples/api-gateway-architecture.json -f junit -o ${targetOutputFile}`;
         exec(exampleValidateCommand, async (_error, _stdout, _stderr) => {
@@ -117,13 +110,11 @@ describe('CLI Integration Tests', () => {
             const expectedXml = await parseStringPromise(expectedXmlString);
 
             expect(parsedOutput).toEqual(expectedXml);
-
-            done();
         });
     });
 
 
-    test('example generate command - does it give the output we expect', (done) => {
+    test('example generate command - does it give the output we expect', async () => {
         const targetOutputFile = path.join(tempDir, 'generate-output.json');
         const exampleGenerateCommand = `calm generate -p ../calm/pattern/api-gateway.json -o ${targetOutputFile}`;
         exec(exampleGenerateCommand, async (_error, _stdout, _stderr) => {
@@ -136,23 +127,20 @@ describe('CLI Integration Tests', () => {
             const expectedJson = JSON.parse(fs.readFileSync(expectedFilePath, 'utf-8'));
 
             expect(parsedOutput).toEqual(expectedJson);
-
-            done();
         });
     });
 
-    test('example validate command - outputting PRETTY to stdout', (done) => {
+    test('example validate command - outputting PRETTY to stdout', async () => {
         const exampleValidateCommand = 'calm validate -p ../calm/pattern/api-gateway.json -a ../calm/samples/api-gateway-architecture.json -f pretty';
         exec(exampleValidateCommand, (_error, stdout, _stderr) => {
             const expectedFilePath = path.join(__dirname, '../test_fixtures/validate_output_pretty.txt');
             const expectedOutput = fs.readFileSync(expectedFilePath, 'utf-8');
             //Some minor replacement logic to avoid issues with line endings
             expect(stdout.replace(/\r\n/g, '\n')).toEqual(expectedOutput.replace(/\r\n/g, '\n'));
-            done();
         });
     });
 
-    test('example validate command - outputting PRETTY to file', (done) => {
+    test('example validate command - outputting PRETTY to file', async () => {
         const targetOutputFile = path.join(tempDir, 'validate-output-pretty.txt');
         const exampleValidateCommand = `calm validate -p ../calm/pattern/api-gateway.json -a ../calm/samples/api-gateway-architecture.json -f pretty -o ${targetOutputFile}`;
         exec(exampleValidateCommand, (_error, _stdout, _stderr) => {
@@ -164,34 +152,31 @@ describe('CLI Integration Tests', () => {
 
             //Some minor replacement logic to avoid issues with line endings
             expect(outputString.replace(/\r\n/g, '\n')).toEqual(expectedOutput.replace(/\r\n/g, '\n'));
-            done();
         });
     });
 
-    test('example validate command - fails when neither an architecture or a pattern is provided', (done) => {
+    test('example validate command - fails when neither an architecture or a pattern is provided', async () => {
         const calmValidateCommand = 'calm validate';
         exec(calmValidateCommand, (error, _stdout, stderr) => {
             expect(error).not.toBeNull();
             expect(stderr).toContain('error: one of the required options \'-p, --pattern <file>\' or \'-a, --architecture <file>\' was not specified');
-            done();
         });
     });
 
-    test('example validate command - validates an architecture only', (done) => {
+    test('example validate command - validates an architecture only', async () => {
         const calmValidateArchitectureOnlyCommand = 'calm validate -a ../calm/samples/api-gateway-architecture.json';
         exec(calmValidateArchitectureOnlyCommand, (error, stdout, _stderr) => {
             const expectedFilePath = path.join(__dirname, '../test_fixtures/validate_architecture_only_output.json');
             const expectedOutput = fs.readFileSync(expectedFilePath, 'utf-8');
             expect(error).toBeNull();
             expect(stdout).toContain(expectedOutput);
-            done();
         });
     });
 
     test('example server command - starts server and responds to requests', async () => {
         // Mock the axios response
         const mockResponse = { status: 200, data: { status: 'ok' } };
-        (axios.get as jest.Mock).mockResolvedValue(mockResponse);
+        (axios.get as vi.Mock).mockResolvedValue(mockResponse);
 
         const serverCommand = 'calm server -p 3001 --schemaDirectory ../../dist/calm/';
         const serverProcess = exec(serverCommand);