code/wrench-js
Archived
1
Fork 0
Code Issues 22 Pull requests 7 Projects Releases Packages Wiki Activity Actions

Using nodeunit's deepEqual instead of manually looping.

Browse source 
This makes for more readable assertion messages (so you can actually see the difference in the lists), and removes the underscore dependency.
This commit is contained in:
domenic 2012-05-14 19:21:37 -04:00
parent 99342e073f
commit 828798c43c
2 changed files with 2 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

3
package.json
View file

@ -21,8 +21,7 @@
},
"devDependencies": {
"nodeunit": ">= 0.6.4",
"underscore": ">= 1.3.1"
"nodeunit": ">= 0.6.4"
},
"main": "./lib/wrench",

7
tests/readdir.js
View file

@ -2,7 +2,6 @@ var testCase = require('nodeunit').testCase;
var fs = require('fs');
var wrench = require('../lib/wrench');
var path = require('path');
var _und = require("underscore");
function checkResult(test, files) {
@ -15,11 +14,7 @@ function checkResult(test, files) {
'foo/bar/ipsum.js'
];
test.equals(files.length, check.length, 'number of paths is correct');
_und.each(check, function(it) {
test.ok(_und.include(files, it), 'path ' + it + ' should be returned');
});
test.deepEqual(files, check);
test.done();
}