Skip to content

Commit

Permalink
Placed our failure handler in the "jqUnit" namespace where it is expe…
Browse files Browse the repository at this point in the history
…cted to be overridden - bump to version 1.1.0
  • Loading branch information
amb26 committed Oct 20, 2015
1 parent 9427d6f commit 534263d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions lib/jqUnit-node.js
Expand Up @@ -29,7 +29,7 @@ Error.stackTraceLimit = 100;
var QUnit = fluid.registerNamespace("QUnit");
var jqUnit = fluid.registerNamespace("jqUnit");

jqUnit.nodeFailureHandler = function (args, activity) { // TODO: More graceful overriding with FLUID-5582
jqUnit.nodeFailureHandler = function (args, activity) {
if (QUnit.config.current) {
QUnit.ok(false, "Assertion failure (see console.log for expanded message): ".concat(args));
} else {
Expand All @@ -39,7 +39,7 @@ jqUnit.nodeFailureHandler = function (args, activity) { // TODO: More graceful o
// Note that since Infusion has overriden node's global uncaught exception handler, the process will not exit here
};

fluid.pushSoftFailure(jqUnit.nodeFailureHandler);
fluid.failureEvent.addListener(jqUnit.nodeFailureHandler, "jqUnit", "before:fail");

QUnit.load();

Expand Down
2 changes: 1 addition & 1 deletion package.json
@@ -1,7 +1,7 @@
{
"name": "node-jqunit",
"description": "An XUnit unit testing framework (in the style of JUnit) to node.js, using QUnit as the underlying engine",
"version": "1.0.0",
"version": "1.1.0",
"author": "Fluid Project",
"homepage": "http://www.fluidproject.org/",
"dependencies": {
Expand Down

0 comments on commit 534263d

Please sign in to comment.