Skip to content

Commit 5ab19f8

Browse files
committed
Build now fails if tests fail; also, tests run in correct order
1 parent 7331abe commit 5ab19f8

3 files changed

Lines changed: 7 additions & 5 deletions

File tree

jakefile.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,11 @@
2222
desc("Test everything");
2323
task("test", [], function() {
2424
var reporter = require("nodeunit").reporters["default"];
25-
reporter.run(['src/server/_server_test.js']);
26-
});
25+
reporter.run(['src/server/_server_test.js'], null, function(failures) {
26+
if (failures) fail("Tests failed");
27+
complete();
28+
});
29+
}, {async: true});
2730

2831
desc("Integrate");
2932
task("integrate", ["default"], function() {

src/server/server.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,5 +2,5 @@
22
"use strict";
33

44
exports.number = function() {
5-
return 0;
5+
return 3;
66
};

todo.txt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,7 @@ Engineering Tasks:
1818
- client-side testing
1919

2020
To Do on current task:
21-
21+
- make lint fail when tests fail
2222

2323
Future ideas:
2424
- Abstract test framework? (Swap out a different framework to make sure abstraction works)
25-
- Add LICENSE.TXT

0 commit comments

Comments
 (0)