fix: flush should error on non-existent entries

License: MIT
Signed-off-by: achingbrain <[email protected]>

Author: achingbrain

src/core/flush.js

@@ -1,5 +1,8 @@
 'use strict'
 
+const waterfall = require('async/waterfall')
+const stat = require('./stat')
+
 const {
   FILE_SEPARATOR
 } = require('./utils')
@@ -25,6 +28,9 @@ module.exports = (context) => {
 
     options = Object.assign({}, defaultOptions, options)
 
-    callback()
+    waterfall([
+      (cb) => stat(context)(path, options, cb),
+      (stats, cb) => cb()
+    ], callback)
   }
 }

test/flush.spec.js

@@ -1,6 +1,9 @@
 /* eslint-env mocha */
 'use strict'
 
+const chai = require('chai')
+chai.use(require('dirty-chai'))
+const expect = chai.expect
 const {
   createMfs
 } = require('./helpers')
@@ -18,4 +21,13 @@ describe('flush', function () {
   it('flushes the root node', () => {
     return mfs.flush()
   })
+
+  it('throws a error when trying to flush non-existent directories', async () => {
+    try {
+      await mfs.flush(`/some-dir-${Math.random()}`)
+      throw new Error('No error was thrown')
+    } catch (err) {
+      expect(err.message).to.include('does not exist')
+    }
+  })
 })