CCM-13342: minor changes from 2025-11-17/separate-workflows-for-docs-002

Author: nhsd-angel-pastor

src/cloudevents/Makefile

@@ -147,7 +147,7 @@ deploy-ci:
 	pwd && \
 	ls -la
 	@echo "from manual test tools versions file contains:"
-	cat .tool-versions
+	cat .tool-versions || echo "  -> .tool-versions file not found in this directory"
 	/usr/local/bin/asdf --version
 	@echo "Setting up asdf environment and adding plugins"
 	export ASDF_DATA_DIR=$$HOME/.asdf && \

src/cloudevents/jest.config.cjs

@@ -17,6 +17,9 @@ module.exports = {
         target: 'ES2020',
         moduleResolution: 'node',
         noEmit: true
+      },
+      diagnostics: {
+        ignoreCodes: [1343]  // Ignore TS1343: import.meta errors
       }
     }]
   },

src/cloudevents/package.json

@@ -34,8 +34,8 @@
     "lint": "echo 'no linting configured'",
     "lint:fix": "echo 'no linting configured'",
     "test": "jest",
-    "test:coverage": "jest --coverage; node -e \"const fs = require('fs'); fs.mkdirSync('.reports/unit/coverage', {recursive: true}); fs.copyFileSync('coverage/lcov.info', '.reports/unit/coverage/lcov.info');\"",
-    "test:unit": "jest --coverage; node -e \"const fs = require('fs'); fs.mkdirSync('.reports/unit/coverage', {recursive: true}); fs.copyFileSync('coverage/lcov.info', '.reports/unit/coverage/lcov.info');\"",
+    "test:coverage": "jest --coverage && node -e \"const fs = require('fs'); fs.mkdirSync('.reports/unit/coverage', {recursive: true}); fs.copyFileSync('coverage/lcov.info', '.reports/unit/coverage/lcov.info');\"",
+    "test:unit": "jest --coverage && node -e \"const fs = require('fs'); fs.mkdirSync('.reports/unit/coverage', {recursive: true}); fs.copyFileSync('coverage/lcov.info', '.reports/unit/coverage/lcov.info');\"",
     "test:watch": "jest --watch",
     "typecheck": "echo 'no typechecking configured'",
     "update-readme": "tsx tools/generator/readme-generator/update-readme-cli.ts",

src/cloudevents/tools/cache/__tests__/schema-cache-network.test.ts

@@ -19,9 +19,11 @@ describe('schema-cache network operations', () => {
     }
   });
 
-  beforeAll((done) => {
+  beforeAll(async () => {
+    console.log('[TEST] Starting HTTP server setup...');
     // Create a local HTTP server for testing
     server = http.createServer((req, res) => {
+      console.log(`[TEST] Server received request for: ${req.url}`);
       // Handle different test scenarios based on URL path
       if (req.url === '/schema.json') {
         res.writeHead(200, { 'Content-Type': 'application/json' });
@@ -50,15 +52,28 @@ describe('schema-cache network operations', () => {
       }
     });
 
-    server.listen(0, 'localhost', () => {
-      const address = server.address() as AddressInfo;
-      serverUrl = `http://localhost:${address.port}`;
-      done();
+    await new Promise<void>((resolve, reject) => {
+      server.listen(0, '127.0.0.1', (err?: Error) => {
+        if (err) {
+          console.error('[TEST] Server failed to start:', err);
+          reject(err);
+          return;
+        }
+        const address = server.address() as AddressInfo;
+        serverUrl = `http://127.0.0.1:${address.port}`;
+        console.log(`[TEST] Server started successfully on ${serverUrl}`);
+        resolve();
+      });
     });
   });
 
-  afterAll((done) => {
-    server.close(done);
+  afterAll(async () => {
+    await new Promise<void>((resolve, reject) => {
+      server.close((err) => {
+        if (err) reject(err);
+        else resolve();
+      });
+    });
   });
 
   beforeEach(() => {

src/cloudevents/tools/discovery/discover-schema-dependencies.js.bak

@@ -592,10 +592,19 @@ describe('DocsGenerator', () => {
     });
 
     it('should process multiple schemas', async () => {
-      // Create multiple schema files
-      fs.writeFileSync(path.join(INPUT_DIR, 'schema1.schema.json'), JSON.stringify({ type: 'object' }));
-      fs.writeFileSync(path.join(INPUT_DIR, 'schema2.schema.json'), JSON.stringify({ type: 'string' }));
-      fs.writeFileSync(path.join(INPUT_DIR, 'schema3.schema.yml'), JSON.stringify({ type: 'number' }));
+      // Create multiple schema files with unique IDs
+      fs.writeFileSync(path.join(INPUT_DIR, 'schema1.schema.json'), JSON.stringify({
+        $id: 'schema1.json',
+        type: 'object'
+      }));
+      fs.writeFileSync(path.join(INPUT_DIR, 'schema2.schema.json'), JSON.stringify({
+        $id: 'schema2.json',
+        type: 'string'
+      }));
+      fs.writeFileSync(path.join(INPUT_DIR, 'schema3.schema.yml'), JSON.stringify({
+        $id: 'schema3.json',
+        type: 'number'
+      }));
 
       const config: DocsGeneratorConfig = {
         inputDir: INPUT_DIR,
@@ -606,6 +615,9 @@ describe('DocsGenerator', () => {
       const generator = new DocsGenerator(config);
       const result = await generator.generate();
 
+      if (!result.success) {
+        console.error('Generation failed:', result.error);
+      }
       expect(result.success).toBe(true);
       expect(result.schemasProcessed).toBe(3);
     });

src/cloudevents/tools/generator/__tests__/docs-generator.test.ts

@@ -132,11 +132,23 @@ export async function handleCli(args: string[]): Promise<CliResult> {
 }
 
 // Execute CLI if this module is run directly
-if (import.meta.url === `file://${process.argv[1]}`) {
-  handleCli(process.argv.slice(2)).then((result) => {
-    process.exit(result.exitCode);
-  }).catch((err) => {
-    console.error('Unexpected error:', err);
-    process.exit(1);
-  });
+// Note: This uses eval to prevent Jest/CommonJS from parsing import.meta
+// istanbul ignore next - CLI entry point, difficult to test in Jest
+// @ts-ignore
+try {
+  const importMeta = eval('import.meta');
+  if (importMeta && importMeta.url === `file://${process.argv[1]}`) {
+    handleCli(process.argv.slice(2)).then((result) => {
+      process.exit(result.exitCode);
+    }).catch((err) => {
+      console.error('Unexpected error:', err);
+      process.exit(1);
+    });
+  }
+} catch {
+  // Intentionally ignoring exception: import.meta not available in CommonJS/Jest environments.
+  // This is expected when the module is imported rather than executed directly.
+  if (process.env.DEBUG) {
+    console.debug('Module loaded in CommonJS/Jest environment (import.meta not available)');
+  }
 }