added tests for npm script invocation.

Author: typhonrt

package.json

@@ -5,15 +5,15 @@
   "description": "Provides an NPM module and script which will load a JSON file searching for an Array entry then executes the scripts.",
   "license": "MPL-2.0",
   "author": {
-    "name": "Mike Leahy"
+    "name": "typhonrt"
   },
   "repository": {
     "type": "git",
     "url": "https://github.com/typhonjs-node-npm-scripts/typhonjs-npm-scripts-runner.git"
   },
   "maintainers": [
     {
-      "name": "Mike Leahy",
+      "name": "typhonrt",
       "email": "[email protected]"
     }
   ],
@@ -32,7 +32,11 @@
     "runner"
   ],
   "scripts": {
-    "test": "node ./test/scripts/test.js"
+    "test": "node ./test/scripts/test.js",
+    "test-script": "node ./scripts/runner.js ./test/data/.scriptrc test.data.scripts",
+    "test-script-bad": "node ./scripts/runner.js ./test/data/.badfile test.data.scripts",
+    "test-script-js": "node ./scripts/runner.js ./test/data/.scriptrc.js test.data.scripts",
+    "test-script-message": "node ./scripts/runner.js ./test/data/.scriptrc test.data.scripts 'A custom message -'"
   },
   "main": "src/runner.js",
   "files": [

scripts/runner.js

@@ -5,7 +5,7 @@
  *
  * Two command line arguments must be supplied:
  * ```
- * (string) filePath - The path to a JSON formatted file.
+ * (string|string[]) filePath - The path to a JSON or JS CJS formatted file.
  * (string) scriptEntry - A entry path separated by `.` relative to the root of the JSON file that is an `Array`.
  * ```
  */
@@ -14,12 +14,12 @@ var runner = require('../src/runner');
 
 if (typeof process.argv[2] !== 'string')
 {
-   throw new TypeError('typhonjs-npm-scripts-runner error: argument `file path` is missing or not a `string`.');
+   throw new TypeError('typhonjs-npm-scripts-runner error: argument[1] `file path` is missing or not a `string`.');
 }
 
 if (typeof process.argv[3] !== 'string')
 {
-   throw new TypeError('typhonjs-npm-scripts-runner error: argument `script entry` is missing or not a `string`.');
+   throw new TypeError("typhonjs-npm-scripts-runner error: argument[2] 'script entry' is missing or not a 'string'.");
 }
 
 runner.run(process.argv[2], process.argv[3], process.argv[4]);

test/scripts/test.js

@@ -5,6 +5,7 @@
  */
 
 var fs = require('fs-extra');
+var cp = require('child_process');
 
 fs.emptyDirSync('./test/fixture');
 
@@ -61,24 +62,7 @@ console.log('Testing bare script treated as JSON:');
 runner.run('./test/data/.scriptrc', 'test.data.scripts');
 
 // Verify that two files were copied.
-try
-{
-   // Verify that `./test/fixture/empty2.json` exists.
-   if (!fs.statSync('./test/fixture/empty2.js').isFile())
-   {
-      throw new Error('`./test/fixture/empty2.js` is not a file.');
-   }
-
-   // Verify that `./test/fixture/empty3.json` exists.
-   if (!fs.statSync('./test/fixture/empty2.js').isFile())
-   {
-      throw new Error('`./test/fixture/empty2.js` is not a file.');
-   }
-}
-catch (err)
-{
-   throw new Error('typhonjs-npm-scripts-runner test error: ' + err);
-}
+verifyOutput();
 
 // -------------------------
 
@@ -87,7 +71,8 @@ console.log('\nTesting bare script message prepend:');
 
 runner.run('./test/data/.scriptrc', 'test.data.scripts', 'A custom message -');
 
-fs.emptyDirSync('./test/fixture');
+// Verify that two files were copied.
+verifyOutput();
 
 // -------------------------
 
@@ -98,24 +83,7 @@ console.log('\nTesting bad file array first entry:');
 runner.run(['./test/data/.badfile1', './test/data/.scriptrc', './test/data/.badfile2'], 'test.data.scripts');
 
 // Verify that two files were copied.
-try
-{
-   // Verify that `./test/fixture/empty2.json` exists.
-   if (!fs.statSync('./test/fixture/empty2.js').isFile())
-   {
-      throw new Error('`./test/fixture/empty2.js` is not a file.');
-   }
-
-   // Verify that `./test/fixture/empty3.json` exists.
-   if (!fs.statSync('./test/fixture/empty2.js').isFile())
-   {
-      throw new Error('`./test/fixture/empty2.js` is not a file.');
-   }
-}
-catch (err)
-{
-   throw new Error('typhonjs-npm-scripts-runner test error: ' + err);
-}
+verifyOutput();
 
 // -------------------------
 
@@ -124,7 +92,8 @@ console.log('\nTesting message prepend:');
 
 runner.run('./test/data/.scriptrc', 'test.data.scripts', 'A custom message -');
 
-fs.emptyDirSync('./test/fixture');
+// Verify that two files were copied.
+verifyOutput();
 
 // ------------------------------------------------------------------------------------------------------------------
 
@@ -135,24 +104,7 @@ console.log('\nTesting CJS scriptrc.js require:');
 runner.run('./test/data/.scriptrc.js', 'test.data.scripts');
 
 // Verify that two files were copied.
-try
-{
-   // Verify that `./test/fixture/empty2.json` exists.
-   if (!fs.statSync('./test/fixture/empty2.js').isFile())
-   {
-      throw new Error('`./test/fixture/empty2.js` is not a file.');
-   }
-
-   // Verify that `./test/fixture/empty3.json` exists.
-   if (!fs.statSync('./test/fixture/empty2.js').isFile())
-   {
-      throw new Error('`./test/fixture/empty2.js` is not a file.');
-   }
-}
-catch (err)
-{
-   throw new Error('typhonjs-npm-scripts-runner test error: ' + err);
-}
+verifyOutput();
 
 // -------------------------
 
@@ -161,7 +113,8 @@ console.log('\nTesting CJS scriptrc.js require - message prepend:');
 
 runner.run('./test/data/.scriptrc.js', 'test.data.scripts', 'A custom message -');
 
-fs.emptyDirSync('./test/fixture');
+// Verify that two files were copied.
+verifyOutput();
 
 // -------------------------
 
@@ -172,24 +125,7 @@ console.log('\nTesting CJS bad first entry for file array:');
 runner.run(['./test/data/.badfile1.js', './test/data/.scriptrc.js', './test/data/.badfile2.js'], 'test.data.scripts');
 
 // Verify that two files were copied.
-try
-{
-   // Verify that `./test/fixture/empty2.json` exists.
-   if (!fs.statSync('./test/fixture/empty2.js').isFile())
-   {
-      throw new Error('`./test/fixture/empty2.js` is not a file.');
-   }
-
-   // Verify that `./test/fixture/empty3.json` exists.
-   if (!fs.statSync('./test/fixture/empty2.js').isFile())
-   {
-      throw new Error('`./test/fixture/empty2.js` is not a file.');
-   }
-}
-catch (err)
-{
-   throw new Error('typhonjs-npm-scripts-runner test error: ' + err);
-}
+verifyOutput();
 
 // -------------------------
 
@@ -198,7 +134,8 @@ console.log('\nTesting CJS prepend message:');
 
 runner.run('./test/data/.scriptrc.js', 'test.data.scripts', 'A custom message -');
 
-fs.emptyDirSync('./test/fixture');
+// Verify that two files were copied.
+verifyOutput();
 
 // -------------------------
 
@@ -209,30 +146,7 @@ console.log('\nTesting JSON file array only executes first matching script:');
 runner.run(['./test/data/.scriptrc', './test/data/.scriptrc2'], 'test.data.scripts');
 
 // Verify that two files were copied and that 'scriptrc2' is not executed.
-try
-{
-   // Verify that `./test/fixture/empty2.json` exists.
-   if (!fs.statSync('./test/fixture/empty2.js').isFile())
-   {
-      throw new Error('`./test/fixture/empty2.js` is not a file.');
-   }
-
-   // Verify that `./test/fixture/empty3.json` exists.
-   if (!fs.statSync('./test/fixture/empty2.js').isFile())
-   {
-      throw new Error('`./test/fixture/empty2.js` is not a file.');
-   }
-
-   // Verify that `./test/fixture/empty3.json` exists.
-   if (fs.existsSync('./test/fixture/SHOULD_NOT_EXIST.js'))
-   {
-      throw new Error('`./test/fixture/SHOULD_NOT_EXIST.js` is was copied / scriptrc2 was incorrectly executed.');
-   }
-}
-catch (err)
-{
-   throw new Error('typhonjs-npm-scripts-runner test error: ' + err);
-}
+verifyOutput();
 
 // -------------------------
 
@@ -243,27 +157,88 @@ console.log('\nTesting CJS file array only executes first matching script:');
 runner.run(['./test/data/.scriptrc.js', './test/data/.scriptrc2'], 'test.data.scripts');
 
 // Verify that two files were copied and that 'scriptrc2' is not executed.
-try
+verifyOutput();
+
+// ------------------------------------------------------------------------------------------------------------------
+
+// Test running 'test-script'
+process.stdout.write('\nTesting NPM script execution\n');
+
+// Notify what command is being executed then execute it.
+var exec = 'npm run test-script';
+process.stdout.write('\nexecuting: ' + exec + '\n');
+cp.execSync(exec, { stdio: 'inherit' });
+
+// Verify that two files were copied.
+verifyOutput();
+
+// -------------------------
+
+// Notify what command is being executed then execute it.
+exec = 'npm run test-script-bad';
+process.stdout.write('\nexecuting: ' + exec + '\n');
+
+thrown = false;
+try { cp.execSync(exec, { stdio: 'inherit' }); }
+catch(err) { thrown = true; }
+if (!thrown)
 {
-   // Verify that `./test/fixture/empty2.json` exists.
-   if (!fs.statSync('./test/fixture/empty2.js').isFile())
+   throw new Error("typhonjs-npm-scripts-runner test error: 'npm run test-script-bad' does not throw error.");
+}
+
+// -------------------------
+
+// Notify what command is being executed then execute it.
+exec = 'npm run test-script-js';
+process.stdout.write('\nexecuting: ' + exec + '\n');
+cp.execSync(exec, { stdio: 'inherit' });
+
+// Verify that two files were copied.
+verifyOutput();
+
+// -------------------------
+
+// Notify what command is being executed then execute it.
+exec = 'npm run test-script-message';
+process.stdout.write('\nexecuting: ' + exec + '\n');
+cp.execSync(exec, { stdio: 'inherit' });
+
+// Verify that two files were copied.
+verifyOutput();
+
+// ------------------------------------------------------------------------------------------------------------------
+
+/**
+ * Verifies testing output in './test/fixture'
+ */
+function verifyOutput()
+{
+   try
    {
-      throw new Error('`./test/fixture/empty2.js` is not a file.');
+      // Verify that `./test/fixture/empty2.json` exists.
+      if (!fs.statSync('./test/fixture/empty2.js').isFile())
+      {
+         throw new Error('`./test/fixture/empty2.js` is not a file.');
+      }
+
+      // Verify that `./test/fixture/empty3.json` exists.
+      if (!fs.statSync('./test/fixture/empty2.js').isFile())
+      {
+         throw new Error('`./test/fixture/empty2.js` is not a file.');
+      }
+
+      // Verify that `./test/fixture/empty3.json` exists.
+      if (fs.existsSync('./test/fixture/SHOULD_NOT_EXIST.js'))
+      {
+         throw new Error('`./test/fixture/SHOULD_NOT_EXIST.js` is was copied / scriptrc2 was incorrectly executed.');
+      }
    }
-
-   // Verify that `./test/fixture/empty3.json` exists.
-   if (!fs.statSync('./test/fixture/empty2.js').isFile())
+   catch (err)
    {
-      throw new Error('`./test/fixture/empty2.js` is not a file.');
+      throw new Error('typhonjs-npm-scripts-runner test error: ' + err);
    }
-
-   // Verify that `./test/fixture/empty3.json` exists.
-   if (fs.existsSync('./test/fixture/SHOULD_NOT_EXIST.js'))
+   finally
    {
-      throw new Error('`./test/fixture/SHOULD_NOT_EXIST.js` is was copied / scriptrc2 was incorrectly executed.');
+      fs.emptyDirSync('./test/fixture');
    }
-}
-catch (err)
-{
-   throw new Error('typhonjs-npm-scripts-runner test error: ' + err);
-}
+}