From bfcb5a6a94fb119c7e020f34ed117aff8359e2cb Mon Sep 17 00:00:00 2001
From: Volodymyr Klymenko <klymenkodp@gmail.com>
Date: Tue, 16 Oct 2018 19:27:12 -0400
Subject: [PATCH] Added tests for Promises version of mkdir (#415)

---
 tests/spec/fs.mkdir.spec.js | 69 ++++++++++++++++++++++++++++---------
 1 file changed, 53 insertions(+), 16 deletions(-)

diff --git a/tests/spec/fs.mkdir.spec.js b/tests/spec/fs.mkdir.spec.js
index 268cb2b..9d834d3 100644
--- a/tests/spec/fs.mkdir.spec.js
+++ b/tests/spec/fs.mkdir.spec.js
@@ -1,43 +1,43 @@
 var util = require('../lib/test-utils.js');
 var expect = require('chai').expect;
 
-describe('fs.mkdir', function() {
+describe('fs.mkdir', function () {
   beforeEach(util.setup);
   afterEach(util.cleanup);
 
-  it('should be a function', function() {
+  it('should be a function', function () {
     var fs = util.fs();
     expect(fs.mkdir).to.be.a('function');
   });
 
-  it('should return an error if part of the parent path does not exist', function(done) {
+  it('should return an error if part of the parent path does not exist', function (done) {
     var fs = util.fs();
 
-    fs.mkdir('/tmp/mydir', function(error) {
+    fs.mkdir('/tmp/mydir', function (error) {
       expect(error).to.exist;
       expect(error.code).to.equal('ENOENT');
       done();
     });
   });
 
-  it('should return an error if the path already exists', function(done) {
+  it('should return an error if the path already exists', function (done) {
     var fs = util.fs();
 
-    fs.mkdir('/', function(error) {
+    fs.mkdir('/', function (error) {
       expect(error).to.exist;
       expect(error.code).to.equal('EEXIST');
       done();
     });
   });
 
-  it('should make a new directory', function(done) {
+  it('should make a new directory', function (done) {
     var fs = util.fs();
 
-    fs.mkdir('/tmp', function(error) {
+    fs.mkdir('/tmp', function (error) {
       expect(error).not.to.exist;
-      if(error) throw error;
+      if (error) throw error;
 
-      fs.stat('/tmp', function(error, stats) {
+      fs.stat('/tmp', function (error, stats) {
         expect(error).not.to.exist;
         expect(stats).to.exist;
         expect(stats.type).to.equal('DIRECTORY');
@@ -45,13 +45,50 @@ describe('fs.mkdir', function() {
       });
     });
   });
+});
 
-  it('should return an error if the path already exists (using promises)', () => {
-    var fsPromises = util.fs().promises;
-    return fsPromises.mkdir('/').catch(error => {
-      expect(error).to.exist;
-      expect(error.code).to.equal('EEXIST');
-    });
+describe('fs.promises.mkdir', function () {
+  beforeEach(util.setup);
+  afterEach(util.cleanup);
+
+  it('should be a function', function () {
+    var fs = util.fs();
+    expect(fs.promises.mkdir).to.be.a('function');
   });
 
+  it('should return an error if part of the parent path does not exist', function () {
+    var fs = util.fs();
+
+    return fs.promises.mkdir('/tmp/mydir')
+      .catch(error => {
+        expect(error).to.exist;
+        expect(error.code).to.equal('ENOENT');
+      });
+  });
+
+  it('should return an error if the path already exists', function () {
+    var fs = util.fs();
+
+    return fs.promises.mkdir('/')
+      .catch(error =>{
+        expect(error).to.exist;
+        expect(error.code).to.equal('EEXIST');
+      });
+  });
+
+  it('should make a new directory', function () {
+    var fs = util.fs();
+
+    return fs.promises.mkdir('/tmp')
+      .then(() => fs.promises.stat('/tmp'))
+      .then(stats => {
+        expect(stats).to.exist;
+        expect(stats.type).to.equal('DIRECTORY');
+      });
+  });
+
+  it('should return a promise', function () {
+    var fs = util.fs();
+    expect(fs.promises.mkdir('/tmp')).to.be.a('promise');
+  });
 });
\ No newline at end of file