chore: warns the user appropriately for non-existent files (#182)

* Warn the user appropriately for non-existent files

start command

* Minor fix

* Tweaks

* refactor

* fix: linting errors

* Minor refactor

remove eslint disable comment

* Minor tweaks

start.js

@@ -71,11 +71,15 @@ function start (args, cb) {
   return runFastify(args, cb)
 }
 
-function stop (error) {
+function stop (error, warn) {
   if (error) {
     console.log(error)
     process.exit(1)
   }
+  if (warn) {
+    console.log(warn)
+    process.exit(1)
+  }
   process.exit()
 }
 
@@ -86,9 +90,16 @@ function runFastify (args, cb) {
 
   loadModules(opts)
 
-  var file = null
+  const filePath = path.resolve(process.cwd(), opts._[0])
+
+  if (!fs.existsSync(filePath)) {
+    return module.exports.stop(null, `${opts._[0]} doesn't exist within ${process.cwd()}`)
+  }
+
+  let file = null
+
   try {
-    file = require(path.resolve(process.cwd(), opts._[0]))
+    file = require(filePath)
   } catch (e) {
     return module.exports.stop(e)
   }

test/start.test.js

@@ -142,38 +142,42 @@ test('should error with a good timeout value', t => {
   start.start(argv)
 })
 
-test('should throw on file not found', t => {
-  t.plan(1)
+test('should warn on file not found', t => {
+  t.plan(2)
 
   const oldStop = start.stop
   t.tearDown(() => { start.stop = oldStop })
-  start.stop = function (err) {
-    t.ok(/Cannot find module.*not-found/.test(err.message), err.message)
+  start.stop = function (err, warn) { // eslint-disable-line
+    // test case changes
+    t.equal(err, null)
+    t.ok(/.*not-found.js doesn't exist within/.test(warn), warn)
   }
 
   const argv = [ '-p', getPort(), './data/not-found.js' ]
   start.start(argv)
 })
 
 test('should throw on package not found', t => {
-  t.plan(1)
+  t.plan(2)
 
   const oldStop = start.stop
   t.tearDown(() => { start.stop = oldStop })
-  start.stop = function (err) {
-    t.ok(/Cannot find module.*unknown-package/.test(err.message), err.message)
+  start.stop = function (err, warn) {
+    t.equal(warn, undefined)
+    t.ok(/Cannot find module 'unknown-package'/.test(err.message), err.message)
   }
 
   const argv = [ '-p', getPort(), './test/data/package-not-found.js' ]
   start.start(argv)
 })
 
 test('should throw on parsing error', t => {
-  t.plan(1)
+  t.plan(2)
 
   const oldStop = start.stop
   t.tearDown(() => { start.stop = oldStop })
-  start.stop = function (err) {
+  start.stop = function (err, warn) { // eslint-disable-line
+    t.equal(warn, undefined)
     t.equal(err.constructor, SyntaxError)
   }