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

vsemozhetbyt · targos · commit 623cf11ddac0 · 2018-06-28T20:41:11.000+02:00
PR-URL: #21531 Fixes: #21519 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Gus Caplan <me@gus.host>
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');

-Original file line number
+Diff line change
 constallDocs=fs.readdirSync(apiPath);
 assert.ok(allDocs.includes('_toc.html'));
 -constfilter=['assets','_toc.html','.md'];
 constactualDocs=allDocs.filter(
 -(name)=>!filter.some((str)=>name.includes(str))
 +(name)=>{
 +constextension=path.extname(name);
 +return(extension==='.html'||extension==='.json')&&
 +name!=='_toc.html';
 +}
 );
 consttoc=fs.readFileSync(path.resolve(apiPath,'_toc.html'),'utf8');