test: skip doctool tests when js-yaml is missing

Skip the doctool tests when js-yaml, which is currently `require()`d from the eslint source tree, is missing. This can happen, for example, because eslint is not included in the release source tarballs. Fixes: nodejs#7201 Ref: nodejs#6495 PR-URL: nodejs#7218 Reviewed-By: Roman Reiss <[email protected]> Reviewed-By: Sakthipriyan Vairamani <[email protected]>
addaleax · Jul 12, 2016 · 383a57a · 383a57a
1 parent 2f7685a
commit 383a57a
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/test/doctool/test-doctool-html.js b/test/doctool/test-doctool-html.js
@@ -5,6 +5,13 @@ const assert = require('assert');
 const fs = require('fs');
 const path = require('path');
 
+// The doctool currently uses js-yaml from the tool/eslint/ tree.
+try {
+  require('../../tools/eslint/node_modules/js-yaml');
+} catch (e) {
+  return common.skip('missing js-yaml (eslint not present)');
+}
+
 const processIncludes = require('../../tools/doc/preprocess.js');
 const html = require('../../tools/doc/html.js');
 

diff --git a/test/doctool/test-doctool-json.js b/test/doctool/test-doctool-json.js
@@ -5,6 +5,13 @@ const assert = require('assert');
 const fs = require('fs');
 const path = require('path');
 
+// The doctool currently uses js-yaml from the tool/eslint/ tree.
+try {
+  require('../../tools/eslint/node_modules/js-yaml');
+} catch (e) {
+  return common.skip('missing js-yaml (eslint not present)');
+}
+
 const json = require('../../tools/doc/json.js');
 
 // Outputs valid json with the expected fields when given simple markdown