No error when unlinking nonexistent file (#34)

* No error when unlinking nonexistent file

* Style + comment

Author: HDegroote

index.js

@@ -226,6 +226,9 @@ module.exports = class RandomAccessFile extends RandomAccessStorage {
     fs.unlink(this.filename, onunlink)
 
     function onunlink (err) {
+      // if the file isn't there, its already unlinked, ignore
+      if (err && err.code === 'ENOENT') err = null
+
       if (!self._rmdir || !root || dir === root) return req.callback(err)
       fs.rmdir(dir, onrmdir)
     }

test/basic.js

@@ -439,6 +439,20 @@ test('unlink', async function (t) {
   }
 })
 
+test('unlink on uncreated file does not reject', async function (t) {
+  t.plan(2)
+  const name = gen()
+
+  const file = new RAF(name)
+  t.is(fs.existsSync(file.filename), false) // Not yet created, since no write
+
+  file.unlink(onunlink)
+
+  function onunlink (err) {
+    t.absent(err, 'no error')
+  }
+})
+
 test('pool', function (t) {
   t.plan(8)