tentative: replaced remaining callbacks with promises. check error propagation correctness

This commit is contained in:
michael-overall 2018-09-26 00:21:43 -04:00
parent a1e73febc1
commit 5abefb6c6a
1 changed files with 66 additions and 74 deletions

View File

@ -1,43 +1,43 @@
var util = require('../lib/test-utils.js'); var util = require('../lib/test-utils.js');
var expect = require('chai').expect; var expect = require('chai').expect;
describe('fs.rename', function() { describe('fs.rename', function () {
beforeEach(util.setup); beforeEach(util.setup);
afterEach(util.cleanup); afterEach(util.cleanup);
it('should be a function', function() { it('should be a function', function () {
var fs = util.fs(); var fs = util.fs();
expect(fs.rename).to.be.a('function'); expect(fs.rename).to.be.a('function');
}); });
it('should rename an existing file', function(done) { it('should rename an existing file', function (done) {
var complete1 = false; var complete1 = false;
var complete2 = false; var complete2 = false;
var fs = util.fs(); var fs = util.fs();
function maybeDone() { function maybeDone() {
if(complete1 && complete2) { if (complete1 && complete2) {
done(); done();
} }
} }
fs.open('/myfile', 'w+', function(error, fd) { fs.open('/myfile', 'w+', function (error, fd) {
if(error) throw error; if (error) throw error;
fs.close(fd, function(error) { fs.close(fd, function (error) {
if(error) throw error; if (error) throw error;
fs.rename('/myfile', '/myotherfile', function(error) { fs.rename('/myfile', '/myotherfile', function (error) {
if(error) throw error; if (error) throw error;
fs.stat('/myfile', function(error, result) { fs.stat('/myfile', function (error, result) {
expect(error).to.exist; expect(error).to.exist;
expect(result).not.to.exist; expect(result).not.to.exist;
complete1 = true; complete1 = true;
maybeDone(); maybeDone();
}); });
fs.stat('/myotherfile', function(error, result) { fs.stat('/myotherfile', function (error, result) {
expect(error).not.to.exist; expect(error).not.to.exist;
expect(result.nlinks).to.equal(1); expect(result.nlinks).to.equal(1);
complete2 = true; complete2 = true;
@ -48,19 +48,19 @@ describe('fs.rename', function() {
}); });
}); });
it('should rename an existing directory', function(done) { it('should rename an existing directory', function (done) {
var fs = util.fs(); var fs = util.fs();
fs.mkdir('/mydir', function(error) { fs.mkdir('/mydir', function (error) {
if(error) throw error; if (error) throw error;
fs.rename('/mydir', '/myotherdir', function(error) { fs.rename('/mydir', '/myotherdir', function (error) {
expect(error).not.to.exist; expect(error).not.to.exist;
fs.stat('/mydir', function(error) { fs.stat('/mydir', function (error) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal('ENOENT'); expect(error.code).to.equal('ENOENT');
fs.stat('/myotherdir', function(error, result) { fs.stat('/myotherdir', function (error, result) {
expect(error).not.to.exist; expect(error).not.to.exist;
expect(result.nlinks).to.equal(1); expect(result.nlinks).to.equal(1);
done(); done();
@ -70,22 +70,22 @@ describe('fs.rename', function() {
}); });
}); });
it('should rename an existing directory if the new path points to an existing directory', function(done) { it('should rename an existing directory if the new path points to an existing directory', function (done) {
var fs = util.fs(); var fs = util.fs();
fs.mkdir('/mydir', function(error) { fs.mkdir('/mydir', function (error) {
if(error) throw error; if (error) throw error;
fs.mkdir('/myotherdir', function(error) { fs.mkdir('/myotherdir', function (error) {
if(error) throw error; if (error) throw error;
fs.rename('/mydir', '/myotherdir', function(error) { fs.rename('/mydir', '/myotherdir', function (error) {
expect(error).not.to.exist; expect(error).not.to.exist;
fs.stat('/mydir', function(error) { fs.stat('/mydir', function (error) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal('ENOENT'); expect(error.code).to.equal('ENOENT');
fs.stat('/myotherdir', function(error, result) { fs.stat('/myotherdir', function (error, result) {
expect(error).not.to.exist; expect(error).not.to.exist;
expect(result.nlinks).to.equal(1); expect(result.nlinks).to.equal(1);
done(); done();
@ -96,26 +96,26 @@ describe('fs.rename', function() {
}); });
}); });
it('should fail to rename an existing directory if the new path points to an existing directory that is not empty', function(done) { it('should fail to rename an existing directory if the new path points to an existing directory that is not empty', function (done) {
var fs = util.fs(); var fs = util.fs();
fs.mkdir('/mydir', function(error) { fs.mkdir('/mydir', function (error) {
if(error) throw error; if (error) throw error;
fs.mkdir('/myotherdir', function(error) { fs.mkdir('/myotherdir', function (error) {
if(error) throw error; if (error) throw error;
fs.writeFile('/myotherdir/myfile', 'This is a file', function(error) { fs.writeFile('/myotherdir/myfile', 'This is a file', function (error) {
if(error) throw error; if (error) throw error;
fs.rename('/mydir', '/myotherdir', function(error) { fs.rename('/mydir', '/myotherdir', function (error) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal('ENOTEMPTY'); expect(error.code).to.equal('ENOTEMPTY');
fs.stat('/mydir', function(error) { fs.stat('/mydir', function (error) {
expect(error).not.to.exist; expect(error).not.to.exist;
fs.stat('/myotherdir', function(error) { fs.stat('/myotherdir', function (error) {
expect(error).not.to.exist; expect(error).not.to.exist;
done(); done();
}); });
@ -126,23 +126,23 @@ describe('fs.rename', function() {
}); });
}); });
it('should fail to rename an existing directory if the new path points to an existing file', function(done) { it('should fail to rename an existing directory if the new path points to an existing file', function (done) {
var fs = util.fs(); var fs = util.fs();
fs.mkdir('/mydir', function(error) { fs.mkdir('/mydir', function (error) {
if(error) throw error; if (error) throw error;
fs.writeFile('/myfile', 'This is a file', function(error) { fs.writeFile('/myfile', 'This is a file', function (error) {
if(error) throw error; if (error) throw error;
fs.rename('/mydir', '/myfile', function(error) { fs.rename('/mydir', '/myfile', function (error) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal('ENOTDIR'); expect(error.code).to.equal('ENOTDIR');
fs.stat('/mydir', function(error) { fs.stat('/mydir', function (error) {
expect(error).not.to.exist; expect(error).not.to.exist;
fs.stat('/myfile', function(error) { fs.stat('/myfile', function (error) {
expect(error).not.to.exist; expect(error).not.to.exist;
done(); done();
}); });
@ -152,43 +152,35 @@ describe('fs.rename', function() {
}); });
}); });
it('(promise version) should rename an existing file', function(done) { it('(promise version) should rename an existing file', function (done) {
var complete1 = false; var complete1 = false;
var complete2 = false; var complete2 = false;
var fs = util.fs(); var fs = util.fs();
function maybeDone() { function maybeDone() {
if(complete1 && complete2) { if (complete1 && complete2) {
done(); done();
} }
} }
//TODO: CHECK PROMISE ERRORS ARE PROPAGATED CORRECTLY (re-throw?)
fs.open('/myfile', 'w+', function(error, fd) { Promise.all(
if(error) throw error; fs.promises.open('/myfile', 'w+')
.then((fd)=>fs.promises.close(fd)),
fs.close(fd, function(error) { fs.promises.rename('/myfile', '/myotherfile'),
if(error) throw error; //TODO: for both stat() check expect() vs assert()
fs.promises.stat('/myfile')
fs.promises.rename('/myfile', '/myotherfile').then( .then( (result)=> expect(result).not.to.exist, (error) => expect(error).to.exist)
function(){ .finally(()=>{
fs.stat('/myfile', function(error, result) { complete1 = true;
expect(error).to.exist; maybeDone();
expect(result).not.to.exist; }),
complete1 = true; fs.promises.stat('/myotherfile')
maybeDone(); .then( (result) => expect(result.nlinks).to.equal(1), (error) => expect(error).not.to.exist)
}); .finally(()=>{
complete2 = true;
fs.stat('/myotherfile', function(error, result) { maybeDone();
expect(error).not.to.exist; })
expect(result.nlinks).to.equal(1); );
complete2 = true; //TODO: .catch() probably not necessary--we just want errors to percolate up...
maybeDone();
});
},
function(error){throw error;}
);
});
});
}); });
}); });