fixup! build: add skip_apidoc_files and include QUIC

Author: RafaelGSS

test/doctool/test-make-doc.mjs

@@ -46,9 +46,11 @@ const expectedJsons = linkedHtmls
                        .map((name) => name.replace('.html', '.json'));
 const expectedDocs = linkedHtmls.concat(expectedJsons);
 const renamedDocs = ['policy.json', 'policy.html'];
+const skipedDocs = ['quic.json', 'quic.html'];
 
 // Test that all the relative links in the TOC match to the actual documents.
 for (const expectedDoc of expectedDocs) {
+  if (skipedDocs.includes(expectedDoc)) continue;
   assert.ok(actualDocs.includes(expectedDoc), `${expectedDoc} does not exist`);
 }
 

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

@@ -25,6 +25,8 @@ const v8OptionsText = cliText.match(/<!-- v8-options start -->(.*)<!-- v8-option
 const manPage = path.join(rootDir, 'doc', 'node.1');
 const manPageText = fs.readFileSync(manPage, { encoding: 'utf8' });
 
+const ignoredDocs = ['--experimental-quic'];
+
 // Documented in /doc/api/deprecations.md
 const deprecated = [
   '--debug',
@@ -80,7 +82,7 @@ for (const [, envVar, config] of nodeOptionsCC.matchAll(addOptionRE)) {
   }
 
   // CLI options
-  if (!isV8Option && !hasTrueAsDefaultValue) {
+  if (!isV8Option && !hasTrueAsDefaultValue && !ignoredDocs.includes(envVar)) {
     if (new RegExp(`###.*\`${envVar}[[=\\s\\b\`]`).test(cliText) === false) {
       assert(false, `Should have option ${envVar} documented`);
     } else {
@@ -101,7 +103,10 @@ for (const [, envVar, config] of nodeOptionsCC.matchAll(addOptionRE)) {
   }
 
   // NODE_OPTIONS
-  if (isInNodeOption && !hasTrueAsDefaultValue && new RegExp(`\`${envVar}\``).test(nodeOptionsText) === false) {
+  if (isInNodeOption && !hasTrueAsDefaultValue &&
+    new RegExp(`\`${envVar}\``).test(nodeOptionsText) === false &&
+    !ignoredDocs.includes(envVar)
+  ) {
     assert(false, `Should have option ${envVar} in NODE_OPTIONS documented`);
   }
 
@@ -121,6 +126,10 @@ for (const [, envVar, config] of nodeOptionsCC.matchAll(addOptionRE)) {
       manPagesOptions.delete(envVar.slice(1));
     }
   }
+
+  if (ignoredDocs.includes(envVar)) {
+    manPagesOptions.delete(envVar.slice(1));
+  }
 }
 
 {

test/parallel/test-process-env-allowed-flags-are-documented.js

@@ -125,6 +125,8 @@ assert(undocumented.delete('--verify-base-objects'));
 assert(undocumented.delete('--no-verify-base-objects'));
 assert(undocumented.delete('--trace-promises'));
 assert(undocumented.delete('--no-trace-promises'));
+assert(undocumented.delete('--experimental-quic'));
+assert(undocumented.delete('--no-experimental-quic'));
 
 // Remove negated versions of the flags.
 for (const flag of undocumented) {