Merge pull request #173 from pbouianov/issue78

Tests that expect errors check error codes. Fixes issue78
This commit is contained in:
Alan K 2014-04-20 11:06:19 -04:00
commit badc309297
17 changed files with 34 additions and 6 deletions

View File

@ -154,7 +154,7 @@ define(function(require) {
callback = callback || function(){}; callback = callback || function(){};
if(!files) { if(!files) {
callback(new Error("Missing files argument")); callback(new Errors.EINVAL("Missing files argument"));
return; return;
} }
@ -208,7 +208,7 @@ define(function(require) {
callback = callback || function(){}; callback = callback || function(){};
if(!dir) { if(!dir) {
callback(new Error("Missing dir argument")); callback(new Errors.EINVAL("Missing dir argument"));
return; return;
} }
@ -280,7 +280,7 @@ define(function(require) {
callback = callback || function(){}; callback = callback || function(){};
if(!path) { if(!path) {
callback(new Error("Missing path argument")); callback(new Errors.EINVAL("Missing path argument"));
return; return;
} }

View File

@ -15,6 +15,7 @@ define(["Filer", "util"], function(Filer, util) {
fs.lstat('/tmp', function(error, result) { fs.lstat('/tmp', function(error, result) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("ENOENT");
expect(result).not.to.exist; expect(result).not.to.exist;
done(); done();
}); });

View File

@ -14,6 +14,7 @@ define(["Filer", "util"], function(Filer, util) {
fs.mkdir('/tmp/mydir', function(error) { fs.mkdir('/tmp/mydir', function(error) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("ENOENT");
done(); done();
}); });
}); });
@ -23,6 +24,7 @@ define(["Filer", "util"], function(Filer, util) {
fs.mkdir('/', function(error) { fs.mkdir('/', function(error) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("EEXIST");
done(); done();
}); });
}); });

View File

@ -14,6 +14,7 @@ define(["Filer", "util"], function(Filer, util) {
fs.open('/tmp/myfile', 'w+', function(error, result) { fs.open('/tmp/myfile', 'w+', function(error, result) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("ENOENT");
expect(result).not.to.exist; expect(result).not.to.exist;
done(); done();
}); });
@ -24,6 +25,7 @@ define(["Filer", "util"], function(Filer, util) {
fs.open('/myfile', 'r+', function(error, result) { fs.open('/myfile', 'r+', function(error, result) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("ENOENT");
expect(result).not.to.exist; expect(result).not.to.exist;
done(); done();
}); });
@ -37,6 +39,7 @@ define(["Filer", "util"], function(Filer, util) {
if(error) throw error; if(error) throw error;
fs.open('/tmp', 'w', function(error, result) { fs.open('/tmp', 'w', function(error, result) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("EISDIR");
expect(result).not.to.exist; expect(result).not.to.exist;
done(); done();
}); });
@ -50,6 +53,7 @@ define(["Filer", "util"], function(Filer, util) {
if(error) throw error; if(error) throw error;
fs.open('/tmp', 'a', function(error, result) { fs.open('/tmp', 'a', function(error, result) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("EISDIR");
expect(result).not.to.exist; expect(result).not.to.exist;
done(); done();
}); });

View File

@ -14,6 +14,7 @@ define(["Filer", "util"], function(Filer, util) {
fs.readdir('/tmp/mydir', function(error, files) { fs.readdir('/tmp/mydir', function(error, files) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("ENOENT");
expect(files).not.to.exist; expect(files).not.to.exist;
done(); done();
}); });

View File

@ -14,6 +14,7 @@ define(["Filer", "util"], function(Filer, util) {
fs.readlink('/tmp/mydir', function(error) { fs.readlink('/tmp/mydir', function(error) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("ENOENT");
done(); done();
}); });
}); });
@ -23,6 +24,7 @@ define(["Filer", "util"], function(Filer, util) {
fs.readlink('/', function(error) { fs.readlink('/', function(error) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("ENOENT");
done(); done();
}); });
}); });

View File

@ -14,6 +14,7 @@ define(["Filer", "util"], function(Filer, util) {
fs.rmdir('/tmp/mydir', function(error) { fs.rmdir('/tmp/mydir', function(error) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("ENOENT");
done(); done();
}); });
}); });
@ -23,6 +24,7 @@ define(["Filer", "util"], function(Filer, util) {
fs.rmdir('/', function(error) { fs.rmdir('/', function(error) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("EBUSY");
done(); done();
}); });
}); });
@ -36,6 +38,7 @@ define(["Filer", "util"], function(Filer, util) {
if(error) throw error; if(error) throw error;
fs.rmdir('/', function(error) { fs.rmdir('/', function(error) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("EBUSY");
done(); done();
}); });
}); });
@ -53,6 +56,7 @@ define(["Filer", "util"], function(Filer, util) {
if(error) throw error; if(error) throw error;
fs.rmdir('/tmp/myfile', function(error) { fs.rmdir('/tmp/myfile', function(error) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("ENOTDIR");
done(); done();
}); });
}); });
@ -69,6 +73,7 @@ define(["Filer", "util"], function(Filer, util) {
if(error) throw error; if(error) throw error;
fs.rmdir('/tmp/myfile', function (error) { fs.rmdir('/tmp/myfile', function (error) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("ENOTDIR");
done(); done();
}); });
}); });

View File

@ -14,6 +14,7 @@ define(["Filer", "util"], function(Filer, util) {
fs.stat('/tmp', function(error, result) { fs.stat('/tmp', function(error, result) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("ENOENT");
expect(result).not.to.exist; expect(result).not.to.exist;
done(); done();
}); });

View File

@ -14,6 +14,7 @@ define(["Filer", "util"], function(Filer, util) {
fs.symlink('/', '/tmp/mydir', function(error) { fs.symlink('/', '/tmp/mydir', function(error) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("ENOENT");
done(); done();
}); });
}); });
@ -23,6 +24,7 @@ define(["Filer", "util"], function(Filer, util) {
fs.symlink('/tmp', '/', function(error) { fs.symlink('/tmp', '/', function(error) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("EEXIST");
done(); done();
}); });
}); });

View File

@ -18,6 +18,7 @@ define(["Filer", "util"], function(Filer, util) {
fs.truncate('/myfile', -1, function(error) { fs.truncate('/myfile', -1, function(error) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("EINVAL");
done(); done();
}); });
}); });
@ -28,6 +29,7 @@ define(["Filer", "util"], function(Filer, util) {
fs.truncate('/', 0, function(error) { fs.truncate('/', 0, function(error) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("EISDIR");
done(); done();
}); });
}); });

View File

@ -16,6 +16,7 @@ define(["Filer", "util"], function(Filer, util) {
fs.readFile('/no-such-file', 'utf8', function(error, data) { fs.readFile('/no-such-file', 'utf8', function(error, data) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("ENOENT");
expect(data).not.to.exist; expect(data).not.to.exist;
done(); done();
}); });

View File

@ -119,6 +119,7 @@ define(["Filer", "util"], function(Filer, util) {
fs.stat('/myfilelink1', function(error, result) { fs.stat('/myfilelink1', function(error, result) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("ENOENT");
expect(result).not.to.exist; expect(result).not.to.exist;
done(); done();
}); });
@ -209,6 +210,7 @@ define(["Filer", "util"], function(Filer, util) {
fs.stat('/mynotdirlink/myfile', function(error, result) { fs.stat('/mynotdirlink/myfile', function(error, result) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("ENOTDIR");
expect(result).not.to.exist; expect(result).not.to.exist;
done(); done();
}); });

View File

@ -15,6 +15,7 @@ define(["Filer", "util"], function(Filer, util) {
shell.cat(null, function(error, data) { shell.cat(null, function(error, data) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("EINVAL");
expect(data).not.to.exist; expect(data).not.to.exist;
done(); done();
}); });
@ -71,6 +72,7 @@ define(["Filer", "util"], function(Filer, util) {
shell.cat(['/file', '/nofile'], function(err, data) { shell.cat(['/file', '/nofile'], function(err, data) {
expect(err).to.exist; expect(err).to.exist;
expect(err.code).to.equal("ENOENT");
expect(data).not.to.exist; expect(data).not.to.exist;
done(); done();
}); });

View File

@ -36,6 +36,7 @@ define(["Filer", "util"], function(Filer, util) {
shell.cat(null, function(error, list) { shell.cat(null, function(error, list) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("EINVAL");
expect(list).not.to.exist; expect(list).not.to.exist;
done(); done();
}); });

View File

@ -15,6 +15,7 @@ define(["Filer", "util"], function(Filer, util) {
shell.cat(null, function(error, list) { shell.cat(null, function(error, list) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("EINVAL");
expect(list).not.to.exist; expect(list).not.to.exist;
done(); done();
}); });

View File

@ -15,6 +15,7 @@ define(["Filer", "util"], function(Filer, util) {
shell.rm(null, function(error, list) { shell.rm(null, function(error, list) {
expect(error).to.exist; expect(error).to.exist;
expect(error.code).to.equal("EINVAL");
expect(list).not.to.exist; expect(list).not.to.exist;
done(); done();
}); });