fix: Use lodash has to check config keys presence (#265)

Closes #263

src/runAll.js

@@ -2,19 +2,20 @@
 
 const sgf = require('staged-git-files')
 const Listr = require('listr')
+const has = require('lodash/has')
 const runScript = require('./runScript')
 const generateTasks = require('./generateTasks')
 const resolveGitDir = require('./resolveGitDir')
 
 /**
  * Executes all tasks and either resolves or rejects the promise
  * @param packageJson
- * @param config
+ * @param config {Object}
  * @returns {Promise}
  */
 module.exports = function runAll(packageJson, config) {
   // Config validation
-  if (!config || !config.gitDir || !config.concurrent || !config.renderer) {
+  if (!config || !has(config, 'gitDir') || !has(config, 'concurrent') || !has(config, 'renderer')) {
     throw new Error('Invalid config provided to runAll! Use getConfig instead.')
   }
 

test/runAll.spec.js

@@ -15,6 +15,13 @@ describe('runAll', () => {
     expect(() => runAll(packageJson)).toThrowErrorMatchingSnapshot()
   })
 
+  it('should not throw when a valid config is provided', () => {
+    const config = getConfig({
+      concurrent: false
+    })
+    expect(() => runAll(packageJson, config)).not.toThrow()
+  })
+
   it('should return a promise', () => {
     expect(runAll(packageJson, getConfig({}))).toBeInstanceOf(Promise)
   })