test: 100% coverage. w00t

Author: jezhiggins

lib/actions/new-blueprint/index.js

@@ -5,12 +5,14 @@ const fs = require('fs')
 const loadProfile = require('../profile')
 const Scaffold = require('@wmfs/tymly-scaffold')
 
-async function newBlueprint (blueprintName, options) {
+async function newBlueprint (name, options) {
+  const workingDirectory = options.path || '.'
+
+  const blueprintName = name && name.trim()
   if (!blueprintName) {
     return
   }
 
-  const workingDirectory = options.path || '.'
   const targetDirectory = path.join(workingDirectory, blueprintName)
 
   console.log(c.bold(`Creating blueprint ${c.cyan(blueprintName)} in ${targetDirectory}`))

test/add-model-test.js

@@ -1,12 +1,5 @@
 /* eslint-env mocha */
 
-const fail = require('assert').fail
-
-const bddStdin = require('bdd-stdin')
-const stdMocks = require('std-mocks')
-const path = require('path')
-const fs = require('fs-extra')
-
 const helpers = require('./test-helpers')
 
 const addModelTest = require('../lib/actions').addModelAction

test/new-blueprint-test.js

@@ -4,29 +4,26 @@ const chai = require('chai')
 chai.use(require('dirty-chai'))
 const expect = chai.expect
 
-const bddStdin = require('bdd-stdin')
-const stdMocks = require('std-mocks')
 const path = require('path')
-const fs = require('fs-extra')
 
 const helpers = require('./test-helpers')
 
 const newBlueprint = require('../lib/actions').newBlueprintAction
 
 describe('tymly new-blueprint', () => {
   const suiteName = 'new-blueprint'
-  const { expectedPath, outputPath } = helpers.fixturePath(suiteName)
+  let outputPath
 
   before(() => {
-    helpers.prepareFixture(suiteName)
+    outputPath = helpers.prepareFixture(suiteName)
   })
 
   it('does nothing if no blueprint name provided', async () => {
     const dirName = path.join(outputPath, 'do-nothing')
 
-    await newBlueprint(null, {
-      path: dirName
-    })
+    await newBlueprint('   ', { })
+    await newBlueprint('', { })
+    await newBlueprint(null, { })
 
     expect(helpers.doesNotExist(dirName)).to.be.true()
   })

test/test-helpers.js

@@ -4,6 +4,7 @@ const fail = chai.expect.fail
 
 const bddStdin = require('bdd-stdin')
 const stdMocks = require('std-mocks')
+const mocha = require('mocha')
 const path = require('path')
 const fs = require('fs-extra')
 const rimraf = require('rimraf')
@@ -27,7 +28,7 @@ function prepareFixture (testSuiteName) {
 }
 
 function runTest (suiteName, testName, inputs, actionFn, ...args) {
-  it(testName, async () => {
+  mocha.it(testName, async () => {
     stdMocks.use()
 
     const { outputPath, profilePath } = fixturePath(suiteName)