lib,test: enforce use of assert.fail via a lint rule

Author: aduh95

doc/api/assert.md

@@ -1682,6 +1682,8 @@ If no arguments are passed in at all `message` will be set to the string:
 Be aware that in the `repl` the error message will be different to the one
 thrown in a file! See below for further details.
 
+<!-- eslint-skip -->
+
 ```mjs
 import assert from 'node:assert/strict';
 
@@ -1717,6 +1719,8 @@ assert.ok(0);
 //   assert.ok(0)
 ```
 
+<!-- eslint-skip -->
+
 ```cjs
 const assert = require('node:assert/strict');
 
@@ -1756,20 +1760,20 @@ assert.ok(0);
 import assert from 'node:assert/strict';
 
 // Using `assert()` works the same:
-assert(0);
+assert(2 + 2 > 5);
 // AssertionError: The expression evaluated to a falsy value:
 //
-//   assert(0)
+//   assert(2 + 2 > 5)
 ```
 
 ```cjs
 const assert = require('node:assert');
 
 // Using `assert()` works the same:
-assert(0);
+assert(2 + 2 > 5);
 // AssertionError: The expression evaluated to a falsy value:
 //
-//   assert(0)
+//   assert(2 + 2 > 5)
 ```
 
 ## `assert.rejects(asyncFn[, error][, message])`

doc/api/test.md

@@ -1786,7 +1786,7 @@ This function creates a hook that runs before executing a suite.
 describe('tests', async () => {
   before(() => console.log('about to run some test'));
   it('is a subtest', () => {
-    assert.ok('some relevant assertion here');
+    // Some relevant assertions here
   });
 });
 ```
@@ -1816,7 +1816,7 @@ This function creates a hook that runs after executing a suite.
 describe('tests', async () => {
   after(() => console.log('finished running tests'));
   it('is a subtest', () => {
-    assert.ok('some relevant assertion here');
+    // Some relevant assertion here
   });
 });
 ```
@@ -1849,7 +1849,7 @@ This function creates a hook that runs before each test in the current suite.
 describe('tests', async () => {
   beforeEach(() => console.log('about to run a test'));
   it('is a subtest', () => {
-    assert.ok('some relevant assertion here');
+    // Some relevant assertion here
   });
 });
 ```
@@ -1880,7 +1880,7 @@ The `afterEach()` hook is run even if the test fails.
 describe('tests', async () => {
   afterEach(() => console.log('finished running a test'));
   it('is a subtest', () => {
-    assert.ok('some relevant assertion here');
+    // Some relevant assertion here
   });
 });
 ```
@@ -3472,7 +3472,7 @@ test('top level test', async (t) => {
   await t.test(
     'This is a subtest',
     (t) => {
-      assert.ok('some relevant assertion here');
+      // Some relevant assertion here
     },
   );
 });
@@ -3503,7 +3503,7 @@ finishes.
 ```js
 test('top level test', async (t) => {
   t.after((t) => t.diagnostic(`finished running ${t.name}`));
-  assert.ok('some relevant assertion here');
+  // Some relevant assertion here
 });
 ```
 
@@ -3535,7 +3535,7 @@ test('top level test', async (t) => {
   await t.test(
     'This is a subtest',
     (t) => {
-      assert.ok('some relevant assertion here');
+      // Some relevant assertion here
     },
   );
 });

lib/internal/crypto/cipher.js

@@ -97,7 +97,7 @@ function getDecoder(decoder, encoding) {
     if (normalizedEncoding === undefined) {
       throw new ERR_UNKNOWN_ENCODING(encoding);
     }
-    assert(false, 'Cannot change encoding');
+    assert.fail('Cannot change encoding');
   }
   return decoder;
 }

test/abort/test-abort-fatal-error.js

@@ -36,7 +36,7 @@ exec(...cmdline, common.mustCall((err, stdout, stderr) => {
   if (!err) {
     console.log(stdout);
     console.log(stderr);
-    assert(false, 'this test should fail');
+    assert.fail('this test should fail');
   }
 
   assert(common.nodeProcessAborted(err.code, err.signal));

test/message/internal_assert.js

@@ -4,4 +4,4 @@
 require('../common');
 
 const assert = require('internal/assert');
-assert(false);
+assert(2 + 2 > 5);

test/parallel/test-cli-node-options-docs.js

@@ -87,19 +87,19 @@ for (const [, envVar, config] of nodeOptionsCC.matchAll(addOptionRE)) {
   // CLI options
   if (!isV8Option && !hasTrueAsDefaultValue) {
     if (new RegExp(`###.*\`${envVar}[[=\\s\\b\`]`).test(cliText) === false) {
-      assert(false, `Should have option ${envVar} documented`);
+      assert.fail(`Should have option ${envVar} documented`);
     } else {
       manPagesOptions.delete(envVar.slice(1));
     }
   }
 
   if (!hasTrueAsDefaultValue && new RegExp(`###.*\`--no${envVar.slice(1)}[[=\\s\\b\`]`).test(cliText) === true) {
-    assert(false, `Should not have option --no${envVar.slice(1)} documented`);
+    assert.fail(`Should not have option --no${envVar.slice(1)} documented`);
   }
 
   if (!isV8Option && hasTrueAsDefaultValue) {
     if (new RegExp(`###.*\`--no${envVar.slice(1)}[[=\\s\\b\`]`).test(cliText) === false) {
-      assert(false, `Should have option --no${envVar.slice(1)} documented`);
+      assert.fail(`Should have option --no${envVar.slice(1)} documented`);
     } else {
       manPagesOptions.delete(`-no${envVar.slice(1)}`);
     }
@@ -108,21 +108,21 @@ for (const [, envVar, config] of nodeOptionsCC.matchAll(addOptionRE)) {
   // NODE_OPTIONS
   if (isInNodeOption && !hasTrueAsDefaultValue &&
     new RegExp(`\`${envVar}\``).test(nodeOptionsText) === false) {
-    assert(false, `Should have option ${envVar} in NODE_OPTIONS documented`);
+    assert.fail(`Should have option ${envVar} in NODE_OPTIONS documented`);
   }
 
   if (isInNodeOption && hasTrueAsDefaultValue && new RegExp(`\`--no${envVar.slice(1)}\``).test(cliText) === false) {
-    assert(false, `Should have option --no${envVar.slice(1)} in NODE_OPTIONS documented`);
+    assert.fail(`Should have option --no${envVar.slice(1)} in NODE_OPTIONS documented`);
   }
 
   if (!hasTrueAsDefaultValue && new RegExp(`\`--no${envVar.slice(1)}\``).test(cliText) === true) {
-    assert(false, `Should not have option --no${envVar.slice(1)} in NODE_OPTIONS documented`);
+    assert.fail(`Should not have option --no${envVar.slice(1)} in NODE_OPTIONS documented`);
   }
 
   // V8 options
   if (isV8Option) {
     if (new RegExp(`###.*\`${envVar}[[=\\s\\b\`]`).test(v8OptionsText) === false) {
-      assert(false, `Should have option ${envVar} in V8 options documented`);
+      assert.fail(`Should have option ${envVar} in V8 options documented`);
     } else {
       manPagesOptions.delete(envVar.slice(1));
     }

test/parallel/test-cluster-bind-twice.js

@@ -102,7 +102,7 @@ if (!id) {
     }));
   }, 2));
 } else {
-  assert(0); // Bad command line argument
+  assert.fail('Bad command line argument');
 }
 
 function startWorker() {

test/parallel/test-cluster-eaddrinuse.js

@@ -58,5 +58,5 @@ if (id === 'undefined') {
     }));
   }));
 } else {
-  assert(0);  // Bad argument.
+  assert.fail('Bad argument');
 }

test/parallel/test-fastutf8stream-sync.js

@@ -181,10 +181,6 @@ assert.throws(() => {
   assert.ok(stream.write('hello world 👀\n'));
   assert.ok(stream.write('another line 👀\n'));
 
-  // Check internal buffer length (may not be available in Utf8Stream)
-  // This is implementation-specific, so we just verify writes succeeded
-  assert.ok(true, 'writes completed successfully');
-
   stream.end();
 }
 
@@ -200,7 +196,6 @@ assert.throws(() => {
   }
 
   // Check internal buffer length (implementation-specific)
-  assert.ok(true, 'writes completed successfully');
   readFile(dest, 'utf8', common.mustSucceed((data) => {
     assert.strictEqual(data, str);
   }));

test/parallel/test-fs-glob.mjs

@@ -529,7 +529,6 @@ describe('glob - with restricted directory', function() {
       for await (const match of asyncGlob('*', { cwd: restrictedDir })) {
         results.push(match);
       }
-      assert.ok(true, 'glob completed without throwing on readdir error');
     } finally {
       try {
         chmodSync(restrictedDir, 0o755);