cleanup

Author: AugustinMauroy

src/__tests__/generators.test.mjs

@@ -1,4 +1,4 @@
-import { ok, strictEqual } from 'node:assert/strict';
+import assert from 'node:assert/strict';
 import { describe, it } from 'node:test';
 
 import createGenerator from '../generators.mjs';
@@ -21,8 +21,8 @@ describe('createGenerator', () => {
   it('should create a generator orchestrator with runGenerators method', () => {
     const { runGenerators } = createGenerator();
 
-    ok(runGenerators);
-    strictEqual(typeof runGenerators, 'function');
+    assert.ok(runGenerators);
+    assert.strictEqual(typeof runGenerators, 'function');
   });
 
   it('should return the ast input directly when generators list is empty', async () => {
@@ -34,9 +34,9 @@ describe('createGenerator', () => {
     });
 
     // Returns array of results, first element is the 'ast' result
-    ok(Array.isArray(results));
-    strictEqual(results.length, 1);
-    ok(results[0]);
+    assert.ok(Array.isArray(results));
+    assert.strictEqual(results.length, 1);
+    assert.ok(results[0]);
   });
 
   it('should run metadata generator', async () => {
@@ -48,9 +48,9 @@ describe('createGenerator', () => {
     });
 
     // Returns array with one element - the collected metadata array
-    ok(Array.isArray(results));
-    strictEqual(results.length, 1);
-    ok(Array.isArray(results[0]));
+    assert.ok(Array.isArray(results));
+    assert.strictEqual(results.length, 1);
+    assert.ok(Array.isArray(results[0]));
   });
 
   it('should handle generator with dependency', async () => {
@@ -63,8 +63,8 @@ describe('createGenerator', () => {
     });
 
     // Should complete without error - returns array of results
-    ok(Array.isArray(results));
-    strictEqual(results.length, 1);
+    assert.ok(Array.isArray(results));
+    assert.strictEqual(results.length, 1);
   });
 
   it('should skip already scheduled generators', async () => {
@@ -77,8 +77,8 @@ describe('createGenerator', () => {
     });
 
     // Returns array with two elements (same result cached for both)
-    ok(Array.isArray(results));
-    strictEqual(results.length, 2);
+    assert.ok(Array.isArray(results));
+    assert.strictEqual(results.length, 2);
   });
 
   it('should handle multiple generators in sequence', async () => {
@@ -91,8 +91,8 @@ describe('createGenerator', () => {
     });
 
     // Returns array of results
-    ok(Array.isArray(results));
-    strictEqual(results.length, 1);
+    assert.ok(Array.isArray(results));
+    assert.strictEqual(results.length, 1);
   });
 
   it('should collect async generator results for dependents', async () => {
@@ -104,8 +104,8 @@ describe('createGenerator', () => {
       generators: ['legacy-json'],
     });
 
-    ok(Array.isArray(results));
-    strictEqual(results.length, 1);
+    assert.ok(Array.isArray(results));
+    assert.strictEqual(results.length, 1);
   });
 
   it('should use multiple threads when specified', async () => {
@@ -118,9 +118,9 @@ describe('createGenerator', () => {
     });
 
     // Returns array of results
-    ok(Array.isArray(results));
-    strictEqual(results.length, 1);
-    ok(Array.isArray(results[0]));
+    assert.ok(Array.isArray(results));
+    assert.strictEqual(results.length, 1);
+    assert.ok(Array.isArray(results[0]));
   });
 
   it('should pass options to generators', async () => {
@@ -135,8 +135,8 @@ describe('createGenerator', () => {
     });
 
     // Returns array of results
-    ok(Array.isArray(results));
-    strictEqual(results.length, 1);
-    ok(Array.isArray(results[0]));
+    assert.ok(Array.isArray(results));
+    assert.strictEqual(results.length, 1);
+    assert.ok(Array.isArray(results[0]));
   });
 });

src/__tests__/metadata.test.mjs

@@ -1,4 +1,4 @@
-import { strictEqual, deepStrictEqual } from 'node:assert/strict';
+import assert from 'node:assert/strict';
 import { describe, it } from 'node:test';
 
 import GitHubSlugger from 'github-slugger';
@@ -21,7 +21,10 @@ describe('createMetadata', () => {
       },
     });
     metadata.setHeading(heading);
-    strictEqual(metadata.create(new VFile(), {}).heading.data, heading.data);
+    assert.strictEqual(
+      metadata.create(new VFile(), {}).heading.data,
+      heading.data
+    );
   });
 
   it('should set the stability correctly', () => {
@@ -34,7 +37,7 @@ describe('createMetadata', () => {
     };
     metadata.addStability(stability);
     const actual = metadata.create(new VFile(), {}).stability;
-    deepStrictEqual(actual, {
+    assert.deepStrictEqual(actual, {
       children: [stability],
       type: 'root',
     });
@@ -83,7 +86,7 @@ describe('createMetadata', () => {
       yaml_position: {},
     };
     const actual = metadata.create(apiDoc, section);
-    deepStrictEqual(actual, expected);
+    assert.deepStrictEqual(actual, expected);
   });
 
   it('should be serializable', () => {
@@ -92,6 +95,6 @@ describe('createMetadata', () => {
       type: 'root',
       children: [],
     });
-    deepStrictEqual(structuredClone(actual), actual);
+    assert.deepStrictEqual(structuredClone(actual), actual);
   });
 });

src/__tests__/streaming.test.mjs

@@ -1,4 +1,4 @@
-import { deepStrictEqual, ok, strictEqual } from 'node:assert/strict';
+import assert from 'node:assert/strict';
 import { describe, it } from 'node:test';
 
 import {
@@ -16,7 +16,7 @@ describe('streaming utilities', () => {
 
       const gen = asyncGen();
 
-      strictEqual(isAsyncGenerator(gen), true);
+      assert.strictEqual(isAsyncGenerator(gen), true);
     });
 
     it('should return false for regular generators', () => {
@@ -26,24 +26,24 @@ describe('streaming utilities', () => {
 
       const gen = syncGen();
 
-      strictEqual(isAsyncGenerator(gen), false);
+      assert.strictEqual(isAsyncGenerator(gen), false);
     });
 
     it('should return false for plain objects', () => {
-      strictEqual(isAsyncGenerator({}), false);
-      strictEqual(isAsyncGenerator([]), false);
-      strictEqual(isAsyncGenerator({ async: true }), false);
+      assert.strictEqual(isAsyncGenerator({}), false);
+      assert.strictEqual(isAsyncGenerator([]), false);
+      assert.strictEqual(isAsyncGenerator({ async: true }), false);
     });
 
     it('should return false for null and undefined', () => {
-      strictEqual(isAsyncGenerator(null), false);
-      strictEqual(isAsyncGenerator(undefined), false);
+      assert.strictEqual(isAsyncGenerator(null), false);
+      assert.strictEqual(isAsyncGenerator(undefined), false);
     });
 
     it('should return false for primitives', () => {
-      strictEqual(isAsyncGenerator(42), false);
-      strictEqual(isAsyncGenerator('string'), false);
-      strictEqual(isAsyncGenerator(true), false);
+      assert.strictEqual(isAsyncGenerator(42), false);
+      assert.strictEqual(isAsyncGenerator('string'), false);
+      assert.strictEqual(isAsyncGenerator(true), false);
     });
 
     it('should return true for objects with Symbol.asyncIterator', () => {
@@ -55,7 +55,7 @@ describe('streaming utilities', () => {
         },
       };
 
-      strictEqual(isAsyncGenerator(asyncIterable), true);
+      assert.strictEqual(isAsyncGenerator(asyncIterable), true);
     });
   });
 
@@ -69,7 +69,7 @@ describe('streaming utilities', () => {
 
       const result = await collectAsyncGenerator(gen());
 
-      deepStrictEqual(result, [1, 2, 3, 4, 5]);
+      assert.deepStrictEqual(result, [1, 2, 3, 4, 5]);
     });
 
     it('should return empty array for empty generator', async () => {
@@ -79,7 +79,7 @@ describe('streaming utilities', () => {
 
       const result = await collectAsyncGenerator(gen());
 
-      deepStrictEqual(result, []);
+      assert.deepStrictEqual(result, []);
     });
 
     it('should handle single chunk', async () => {
@@ -89,7 +89,7 @@ describe('streaming utilities', () => {
 
       const result = await collectAsyncGenerator(gen());
 
-      deepStrictEqual(result, [1, 2, 3]);
+      assert.deepStrictEqual(result, [1, 2, 3]);
     });
 
     it('should handle empty chunks', async () => {
@@ -102,7 +102,7 @@ describe('streaming utilities', () => {
 
       const result = await collectAsyncGenerator(gen());
 
-      deepStrictEqual(result, [1, 2, 3]);
+      assert.deepStrictEqual(result, [1, 2, 3]);
     });
 
     it('should handle objects in chunks', async () => {
@@ -113,16 +113,16 @@ describe('streaming utilities', () => {
 
       const result = await collectAsyncGenerator(gen());
 
-      deepStrictEqual(result, [{ a: 1 }, { b: 2 }, { c: 3 }]);
+      assert.deepStrictEqual(result, [{ a: 1 }, { b: 2 }, { c: 3 }]);
     });
   });
 
   describe('createStreamingCache', () => {
     it('should create a cache with required methods', () => {
       const cache = createStreamingCache();
 
-      ok(cache);
-      strictEqual(typeof cache.getOrCollect, 'function');
+      assert.ok(cache);
+      assert.strictEqual(typeof cache.getOrCollect, 'function');
     });
 
     it('should return same promise for same key', async () => {
@@ -145,8 +145,8 @@ describe('streaming utilities', () => {
       const result1 = await promise1;
       const result2 = await promise2;
 
-      deepStrictEqual(result1, [1, 2, 3]);
-      strictEqual(result1, result2);
+      assert.deepStrictEqual(result1, [1, 2, 3]);
+      assert.strictEqual(result1, result2);
     });
 
     it('should return different results for different keys', async () => {
@@ -163,8 +163,8 @@ describe('streaming utilities', () => {
       const result1 = await cache.getOrCollect('key1', gen1());
       const result2 = await cache.getOrCollect('key2', gen2());
 
-      deepStrictEqual(result1, [1, 2]);
-      deepStrictEqual(result2, [3, 4]);
+      assert.deepStrictEqual(result1, [1, 2]);
+      assert.deepStrictEqual(result2, [3, 4]);
     });
   });
 });

src/generators/man-page/utils/__tests__/converter.test.mjs

@@ -1,4 +1,4 @@
-import { strictEqual } from 'node:assert/strict';
+import assert from 'node:assert/strict';
 import { describe, it } from 'node:test';
 
 import { u } from 'unist-builder';
@@ -20,7 +20,7 @@ const createMockElement = (headingText, description) => ({
 
 const runTests = (cases, conversionFunc) => {
   cases.forEach(({ input, expected }) => {
-    strictEqual(conversionFunc(input), expected);
+    assert.strictEqual(conversionFunc(input), expected);
   });
 };
 
@@ -31,7 +31,7 @@ describe('Mandoc Conversion', () => {
         u('heading', { depth: 1 }, [textNode('Main Title')]),
         u('paragraph', [textNode('Introductory text.')]),
       ]);
-      strictEqual(
+      assert.strictEqual(
         convertNodeToMandoc(node),
         '.Sh Main Title\nIntroductory text.'
       );
@@ -111,7 +111,7 @@ describe('Mandoc Conversion', () => {
         '`-a`, `-b=value`',
         'Description of the options.'
       );
-      strictEqual(
+      assert.strictEqual(
         convertOptionToMandoc(mockElement),
         `.It Fl a , Fl b Ns = Ns Ar value\nDescription of the options.\n.\n`
       );
@@ -122,7 +122,7 @@ describe('Mandoc Conversion', () => {
         '`-a`',
         'Description of the option without a value.'
       );
-      strictEqual(
+      assert.strictEqual(
         convertOptionToMandoc(mockElement),
         `.It Fl a\nDescription of the option without a value.\n.\n`
       );
@@ -133,7 +133,7 @@ describe('Mandoc Conversion', () => {
         '`-x`, `-y`, `-z=value`',
         'Description of multiple options.'
       );
-      strictEqual(
+      assert.strictEqual(
         convertOptionToMandoc(mockElement),
         `.It Fl x , Fl y , Fl z Ns = Ns Ar value\nDescription of multiple options.\n.\n`
       );
@@ -144,7 +144,7 @@ describe('Mandoc Conversion', () => {
         '`-d`, `--option=value with spaces`',
         'Description of special options.'
       );
-      strictEqual(
+      assert.strictEqual(
         convertOptionToMandoc(mockElement),
         `.It Fl d , Fl -option Ns = Ns Ar value with spaces\nDescription of special options.\n.\n`
       );
@@ -157,7 +157,7 @@ describe('Mandoc Conversion', () => {
         '`MY_VAR=some_value`',
         'Description of the environment variable.'
       );
-      strictEqual(
+      assert.strictEqual(
         convertEnvVarToMandoc(mockElement),
         `.It Ev MY_VAR Ar some_value\nDescription of the environment variable.\n.\n`
       );
@@ -168,7 +168,7 @@ describe('Mandoc Conversion', () => {
         '`MY_VAR=`',
         'Description of the environment variable without a value.'
       );
-      strictEqual(
+      assert.strictEqual(
         convertEnvVarToMandoc(mockElement),
         `.It Ev MY_VAR\nDescription of the environment variable without a value.\n.\n`
       );
@@ -179,7 +179,7 @@ describe('Mandoc Conversion', () => {
         '`SPECIAL_VAR=special value!`',
         'Description of special environment variable.'
       );
-      strictEqual(
+      assert.strictEqual(
         convertEnvVarToMandoc(mockElement),
         `.It Ev SPECIAL_VAR Ar special value!\nDescription of special environment variable.\n.\n`
       );

src/generators/metadata/__tests__/index.test.mjs

@@ -1,4 +1,4 @@
-import { deepStrictEqual, strictEqual } from 'node:assert/strict';
+import assert from 'node:assert/strict';
 import { describe, it } from 'node:test';
 
 import generator from '../index.mjs';
@@ -24,8 +24,8 @@ describe('generators/metadata/index', () => {
       results.push(chunk);
     }
 
-    strictEqual(results.length, 2);
-    deepStrictEqual(results[0], [1, 2, 3]);
-    deepStrictEqual(results[1], [4]);
+    assert.strictEqual(results.length, 2);
+    assert.deepStrictEqual(results[0], [1, 2, 3]);
+    assert.deepStrictEqual(results[1], [4]);
   });
 });