Refactor: executable script consistency

scripts/fix/index.js

@@ -44,19 +44,25 @@ function load(...files) {
   }
 }
 
-if (process.argv[2]) {
-  load(process.argv[2]);
-} else {
-  load(
-    'api',
-    'css',
-    'html',
-    'http',
-    'svg',
-    'javascript',
-    'mathml',
-    'test',
-    'webdriver',
-    'webextensions',
-  );
+if (require.main === module) {
+  if (process.argv[2]) {
+    load(process.argv[2]);
+  } else {
+    load(
+      'api',
+      'css',
+      'html',
+      'http',
+      'svg',
+      'javascript',
+      'mathml',
+      'test',
+      'webdriver',
+      'webextensions',
+    );
+
+    format();
+  }
 }
+
+module.exports = load;

scripts/release/pulls.js

@@ -73,4 +73,6 @@ function main() {
   }
 }
 
-main();
+if (require.main === module) {
+  main();
+}

scripts/release/stats.js

@@ -143,4 +143,6 @@ async function main() {
   );
 }
 
-main();
+if (require.main === module) {
+  main();
+}

scripts/traverse.js

@@ -79,17 +79,32 @@ function traverseFeatures(obj, depth, identifier) {
   }
 }
 
-let features = [];
-const folders =
-  argv.folder == 'all'
-    ? ['api', 'css', 'html', 'http', 'svg', 'javascript', 'mathml', 'webdriver']
-    : argv.folder.split(',');
-const values = Array.isArray(argv.value)
-  ? argv.value
-  : argv.value.toString().split(',');
+const main = (folder = 'all', value = ['null', 'true'], depth = 100) => {
+  let features = [];
+  const folders =
+    folder == 'all'
+      ? [
+          'api',
+          'css',
+          'html',
+          'http',
+          'svg',
+          'javascript',
+          'mathml',
+          'webdriver',
+        ]
+      : folder.split(',');
+  const values = Array.isArray(value) ? value : value.toString().split(',');
 
-for (const folder in folders)
-  traverseFeatures(bcd[folders[folder]], argv.depth, `${folders[folder]}.`);
+  for (const folder in folders)
+    traverseFeatures(bcd[folders[folder]], depth, `${folders[folder]}.`);
 
-console.log(features.join('\n'));
-console.log(features.length);
+  console.log(features.join('\n'));
+  console.log(features.length);
+};
+
+if (require.main === module) {
+  main(argv.folder, argv.value, argv.depth);
+}
+
+module.exports = main;

test/lint.js

@@ -150,49 +150,59 @@ function load(...files) {
   }, false);
 }
 
-/** @type {boolean} */
-var hasErrors = argv.files
-  ? load.apply(undefined, argv.files)
-  : load(
-      'api',
-      'browsers',
-      'css',
-      'html',
-      'http',
-      'svg',
-      'javascript',
-      'mathml',
-      'webdriver',
-      'webextensions',
+const main = (files) => {
+  /** @type {boolean} */
+  var hasErrors = argv.files
+    ? load.apply(undefined, argv.files)
+    : load(
+        'api',
+        'browsers',
+        'css',
+        'html',
+        'http',
+        'svg',
+        'javascript',
+        'mathml',
+        'webdriver',
+        'webextensions',
+      );
+
+  if (hasErrors) {
+    console.warn('');
+    console.warn(
+      chalk`{red Problems in ${pluralize('file', filesWithErrors.size)}:}`,
     );
-
-if (hasErrors) {
-  console.warn('');
-  console.warn(
-    chalk`{red Problems in ${pluralize('file', filesWithErrors.size)}:}`,
-  );
-  for (const [fileName, file] of filesWithErrors) {
-    console.warn(chalk`{red.bold ✖ ${fileName}}`);
-    try {
-      if (file.indexOf('browsers' + path.sep) !== -1) {
-        testSchema(file, './../../schemas/browsers.schema.json');
-        testBrowsersData(file);
-        testLinks(file);
-      } else {
-        testSchema(file);
-        testStyle(file);
-        testLinks(file);
-        testVersions(file);
-        testRealValues(file);
-        testBrowsersPresence(file);
-        testConsistency(file);
-        testPrefix(file);
-        testDescriptions(file);
-        testNotes(file);
+    for (const [fileName, file] of filesWithErrors) {
+      console.warn(chalk`{red.bold ✖ ${fileName}}`);
+      try {
+        if (file.indexOf('browsers' + path.sep) !== -1) {
+          testSchema(file, './../../schemas/browsers.schema.json');
+          testBrowsersData(file);
+          testLinks(file);
+        } else {
+          testSchema(file);
+          testStyle(file);
+          testLinks(file);
+          testVersions(file);
+          testRealValues(file);
+          testBrowsersPresence(file);
+          testConsistency(file);
+          testPrefix(file);
+          testDescriptions(file);
+          testNotes(file);
+        }
+      } catch (e) {
+        console.error(e);
       }
-    } catch (e) {
-      console.error(e);
     }
+    return true;
   }
-  process.exit(1);
+
+  return false;
+};
+
+if (require.main === module) {
+  process.exit(main(argv.files) ? 1 : 0);
 }
+
+module.exports = main;

test/test-regexes.js

@@ -17,6 +17,28 @@ const assert = require('assert').strict;
 /** @type {Identifier} */
 const bcd = require('..');
 
+/** @type {TestCase[]} */
+const tests = [
+  {
+    features: ['css.properties.color.alpha_hexadecimal_notation'],
+    matches: ['#003399ff', '#0af9'],
+    misses: ['#00aaff', '#0af', 'green', '#greenish'],
+  },
+  {
+    features: ['css.properties.transform-origin.three_value_syntax'],
+    matches: [
+      '2px 30% 10px', // length, percentage, length
+      'right bottom -2cm', // two keywords and length
+      'calc(50px - 25%) 2px 1px', // lengths with calc
+    ],
+    misses: [
+      'center', // one value syntax
+      'left 5px', // two value syntax
+      'left calc(10px - 50%)', // two value syntax with calc
+    ],
+  },
+];
+
 /**
  * @param {string} dottedFeature
  */
@@ -45,28 +67,10 @@ function testToken(feature, matches, misses) {
   );
 }
 
-/** @type {TestCase[]} */
-const tests = [
-  {
-    features: ['css.properties.color.alpha_hexadecimal_notation'],
-    matches: ['#003399ff', '#0af9'],
-    misses: ['#00aaff', '#0af', 'green', '#greenish'],
-  },
-  {
-    features: ['css.properties.transform-origin.three_value_syntax'],
-    matches: [
-      '2px 30% 10px', // length, percentage, length
-      'right bottom -2cm', // two keywords and length
-      'calc(50px - 25%) 2px 1px', // lengths with calc
-    ],
-    misses: [
-      'center', // one value syntax
-      'left 5px', // two value syntax
-      'left calc(10px - 50%)', // two value syntax with calc
-    ],
-  },
-];
+const testRegexes = () => {
+  tests.forEach(({ features, matches, misses }) => {
+    features.forEach((feature) => testToken(lookup(feature), matches, misses));
+  });
+};
 
-tests.forEach(({ features, matches, misses }) => {
-  features.forEach((feature) => testToken(lookup(feature), matches, misses));
-});
+module.exports = testRegexes;