Skip to content

Commit 03d9e28

Browse files
committed
examples: lower-case_spec.js: Allow proper assertion failure reporting (#45)
The exceptions raised by assertion failures get swallowed by the Node-RED internals, so catch them explicitly and pass them to done() as suggested in #45.
1 parent 9adb40d commit 03d9e28

File tree

1 file changed

+18
-6
lines changed

1 file changed

+18
-6
lines changed

examples/lower-case_spec.js

Lines changed: 18 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13,17 +13,25 @@ describe('lower-case Node', function () {
1313
var flow = [{ id: "n1", type: "lower-case", name: "lower-case" }];
1414
helper.load(lowerNode, flow, function () {
1515
var n1 = helper.getNode("n1");
16-
n1.should.have.property('name', 'lower-case');
17-
done();
16+
try {
17+
n1.should.have.property('name', 'lower-case');
18+
done();
19+
} catch(err) {
20+
done(err);
21+
}
1822
});
1923
});
2024

2125
it('should be loaded in exported flow', function (done) {
2226
var flow = [{"id":"3912a37a.c3818c","type":"lower-case","z":"e316ac4b.c85a2","name":"lower-case","x":240,"y":320,"wires":[[]]}];
2327
helper.load(lowerNode, flow, function () {
2428
var n1 = helper.getNode("3912a37a.c3818c");
25-
n1.should.have.property('name', 'lower-case');
26-
done();
29+
try {
30+
n1.should.have.property('name', 'lower-case');
31+
done();
32+
} catch(err) {
33+
done(err);
34+
}
2735
});
2836
});
2937

@@ -36,8 +44,12 @@ describe('lower-case Node', function () {
3644
var n2 = helper.getNode("n2");
3745
var n1 = helper.getNode("n1");
3846
n2.on("input", function (msg) {
39-
msg.should.have.property('payload', 'uppercase');
40-
done();
47+
try {
48+
msg.should.have.property('payload', 'uppercase');
49+
done();
50+
} catch(err) {
51+
done(err);
52+
}
4153
});
4254
n1.receive({ payload: "UpperCase" });
4355
});

0 commit comments

Comments
 (0)