feat: allow arbitrary commands, close #165

.travis.yml

@@ -17,6 +17,7 @@ script:
   - npm run demo7
   - START_SERVER_AND_TEST_INSECURE=1 npm run demo9
   - npm run demo-cross-env
+  - npm run demo-commands
 after_success:
   - npm run travis-deploy-once "npm run semantic-release"
 branches:

__snapshots__/utils-spec.js

@@ -1,65 +1,73 @@
 exports['utils getArguments handles 3 arguments with http-get url 1'] = {
-  "start": "start",
+  "start": "npm run start",
   "url": [
     "http-get://localhost:8080"
   ],
-  "test": "test"
+  "test": "npm run test"
 }
 
 exports['utils getArguments returns 3 arguments 1'] = {
-  "start": "start",
+  "start": "npm run start",
   "url": [
     "http://localhost:8080"
   ],
-  "test": "test"
+  "test": "npm run test"
 }
 
 exports['utils getArguments returns 3 arguments with url 1'] = {
-  "start": "start",
+  "start": "npm run start",
   "url": [
     "http://localhost:8080"
   ],
-  "test": "test"
+  "test": "npm run test"
+}
+
+exports['utils getArguments understands custom commands 1'] = {
+  "start": "custom-command --with argument",
+  "url": [
+    "http://localhost:3000"
+  ],
+  "test": "test-command --x=1"
 }
 
 exports['utils getArguments understands several ports 1'] = {
-  "start": "start",
+  "start": "npm run start",
   "url": [
     "http://localhost:3000",
     "http://localhost:4000",
     "http://localhost:5000"
   ],
-  "test": "test"
+  "test": "npm run test"
 }
 
 exports['utils getArguments understands single :port 1'] = {
-  "start": "start",
+  "start": "npm run start",
   "url": [
     "http://localhost:3000"
   ],
-  "test": "test"
+  "test": "npm run test"
 }
 
 exports['utils getArguments understands single port 1'] = {
-  "start": "start",
+  "start": "npm run start",
   "url": [
     "http://localhost:3000"
   ],
-  "test": "test"
+  "test": "npm run test"
 }
 
 exports['utils getArguments understands start plus url 1'] = {
   "start": "start-server",
   "url": [
     "http://localhost:6000"
   ],
-  "test": "test"
+  "test": "npm run test"
 }
 
 exports['utils getArguments understands url plus test 1'] = {
-  "start": "start",
+  "start": "npm run start",
   "url": [
     "http://localhost:6000"
   ],
-  "test": "test"
+  "test": "npm run test"
 }

package.json

@@ -88,6 +88,7 @@
     "demo9": "node src/bin/start.js start-https \"https://127.0.0.1:9000\" test4",
     "demo10": "node src/bin/start.js start-fail http://127.0.0.1:9000 test",
     "demo-cross-env": "node src/bin/start.js start-cross-env 9000",
+    "demo-commands": "node src/bin/start.js 'node test/server.js' 9000 'npm run unit'",
     "travis-deploy-once": "travis-deploy-once"
   },
   "devDependencies": {

src/bin/start.js

@@ -6,9 +6,9 @@ const utils = require('../utils')
 
 const { start, test, url } = utils.getArguments(args)
 
-console.log(`starting server using command "npm run ${start}"`)
-console.log(`and when url "${url}" is responding`)
-console.log(`running tests using command "npm run ${test}"`)
+console.log('starting server using command "%s"', start)
+console.log('and when url "%s" is responding', url)
+console.log('running tests using command "%s"', test)
 
 startAndTest({ start, url, test }).catch(e => {
   console.error(e)

src/index.js

@@ -22,10 +22,9 @@ function startAndTest ({ start, url, test }) {
     url
   )
 
-  debug('starting server, verbose mode?', isDebug())
+  debug('starting server with command "%s", verbose mode?', start, isDebug())
 
-  const startCommand = 'npm run start'
-  const server = execa.shell(startCommand, { stdio: 'inherit' })
+  const server = execa.shell(start, { stdio: 'inherit' })
   let serverStopped
 
   function stopServer () {
@@ -87,9 +86,8 @@ function startAndTest ({ start, url, test }) {
   })
 
   function runTests () {
-    const testCommand = `npm run ${test}`
-    debug('running test script command: %s', testCommand)
-    return execa.shell(testCommand, { stdio: 'inherit' })
+    debug('running test script command: %s', test)
+    return execa.shell(test, { stdio: 'inherit' })
   }
 
   return waited

src/utils-spec.js

@@ -45,6 +45,8 @@ describe('utils', () => {
     })
 
     it('understands start plus url', () => {
+      // note that this script "start-server" does not exist
+      // thus it is left as is - without "npm run" part
       snapshot(getArguments(['start-server', '6000']))
     })
 
@@ -59,6 +61,18 @@ describe('utils', () => {
     it('understands several ports', () => {
       snapshot(getArguments(['3000|4000|5000']))
     })
+
+    it('understands custom commands', () => {
+      // these commands are NOT script names in the package.json
+      // thus they will be run as is
+      snapshot(
+        getArguments([
+          'custom-command --with argument',
+          '3000',
+          'test-command --x=1'
+        ])
+      )
+    })
   })
 
   context('isUrlOrPort', () => {

src/utils.js

@@ -43,6 +43,14 @@ const getArguments = cliArgs => {
     test = cliArgs[2]
   }
 
+  if (isPackageScriptName(start)) {
+    start = `npm run ${start}`
+  }
+
+  if (isPackageScriptName(test)) {
+    test = `npm run ${test}`
+  }
+
   return {
     start,
     url,