test: skip non-doc files in test-make-doc checks

PR-URL: nodejs#21531 Fixes: nodejs#21519 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Gus Caplan <[email protected]>
nxtedition · Jun 27, 2018 · 759809f · 759809f
1 parent 1f6adff
commit 759809f
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/test/doctool/test-make-doc.js b/test/doctool/test-make-doc.js
@@ -15,9 +15,12 @@ const apiPath = path.resolve(__dirname, '..', '..', 'out', 'doc', 'api');
 const allDocs = fs.readdirSync(apiPath);
 assert.ok(allDocs.includes('_toc.html'));
 
-const filter = ['assets', '_toc.html', '.md'];
 const actualDocs = allDocs.filter(
-  (name) => !filter.some((str) => name.includes(str))
+  (name) => {
+    const extension = path.extname(name);
+    return (extension === '.html' || extension === '.json') &&
+           name !== '_toc.html';
+  }
 );
 
 const toc = fs.readFileSync(path.resolve(apiPath, '_toc.html'), 'utf8');