frodovdr
/
node-red
mirror of https://github.com/node-red/node-red.git
1
0
Fork 0
Code Issues Releases Wiki Activity

Merge pull request #1845 from node-red-hitachi/fix-file-node-test-for-ordering-assumption-of-output-message 

Update file node test to cope with occasional failure
This commit is contained in:
Nick O'Leary 2018-07-27 13:20:27 +01:00 committed by GitHub
parent 4e549dd426 81efce03ba
commit 3ac8ce03bf
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 88 additions and 60 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
test/nodes/core/storage/50-file_spec.js
View File

@ -46,9 +46,14 @@ describe('file Nodes', function() {
it('should be loaded', function(done) { it('should be loaded', function(done) {
var flow = [{id:"fileNode1", type:"file", name: "fileNode", "filename":fileToTest, "appendNewline":true, "overwriteFile":true}]; var flow = [{id:"fileNode1", type:"file", name: "fileNode", "filename":fileToTest, "appendNewline":true, "overwriteFile":true}];
helper.load(fileNode, flow, function() { helper.load(fileNode, flow, function() {
try {
var fileNode1 = helper.getNode("fileNode1"); var fileNode1 = helper.getNode("fileNode1");
fileNode1.should.have.property('name', 'fileNode'); fileNode1.should.have.property('name', 'fileNode');
done(); done();
}
catch (e) {
done(e);
}
}); });
}); });
@ -59,11 +64,16 @@ describe('file Nodes', function() {
var n1 = helper.getNode("fileNode1"); var n1 = helper.getNode("fileNode1");
var n2 = helper.getNode("helperNode1"); var n2 = helper.getNode("helperNode1");
n2.on("input", function(msg) { n2.on("input", function(msg) {
try {
var f = fs.readFileSync(fileToTest); var f = fs.readFileSync(fileToTest);
f.should.have.length(4); f.should.have.length(4);
fs.unlinkSync(fileToTest); fs.unlinkSync(fileToTest);
msg.should.have.property("payload", "test"); msg.should.have.property("payload", "test");
done(); done();
}
catch (e) {
done(e);
}
}); });
n1.receive({payload:"test"}); n1.receive({payload:"test"});
}); });
@ -83,7 +93,9 @@ describe('file Nodes', function() {
var data = ["test2", true, 999, [2]]; var data = ["test2", true, 999, [2]];
n2.on("input", function (msg) { n2.on("input", function (msg) {
msg.should.have.property("payload", data[count]); try {
msg.should.have.property("payload");
data.should.containDeep([msg.payload]);
if (count === 3) { if (count === 3) {
var f = fs.readFileSync(fileToTest).toString(); var f = fs.readFileSync(fileToTest).toString();
if (os.type() !== "Windows_NT") { if (os.type() !== "Windows_NT") {
@ -97,6 +109,10 @@ describe('file Nodes', function() {
done(); done();
} }
count++; count++;
}
catch (e) {
done(e);
}
}); });
n1.receive({payload:"test2"}); // string n1.receive({payload:"test2"}); // string
@ -126,7 +142,9 @@ describe('file Nodes', function() {
var count = 0; var count = 0;
n2.on("input", function (msg) { n2.on("input", function (msg) {
msg.should.have.property("payload", data[count]); try {
msg.should.have.property("payload");
data.should.containDeep([msg.payload]);
try { try {
if (count === 1) { if (count === 1) {
// Check they got appended as expected // Check they got appended as expected
@ -151,6 +169,10 @@ describe('file Nodes', function() {
done(err); done(err);
} }
count++; count++;
}
catch (e) {
done(e);
}
}); });
// Send two messages to the file // Send two messages to the file
@ -174,7 +196,8 @@ describe('file Nodes', function() {
n2.on("input", function (msg) { n2.on("input", function (msg) {
try { try {
msg.should.have.property("payload", data[count]); msg.should.have.property("payload");
data.should.containDeep([msg.payload]);
if (count == 1) { if (count == 1) {
// Check they got appended as expected // Check they got appended as expected
var f = fs.readFileSync(fileToTest).toString(); var f = fs.readFileSync(fileToTest).toString();
@ -233,6 +256,7 @@ describe('file Nodes', function() {
var n2 = helper.getNode("helperNode1"); var n2 = helper.getNode("helperNode1");
n2.on("input", function (msg) { n2.on("input", function (msg) {
try {
msg.should.have.property("payload", "fine"); msg.should.have.property("payload", "fine");
msg.should.have.property("filename", fileToTest); msg.should.have.property("filename", fileToTest);
@ -246,6 +270,10 @@ describe('file Nodes', function() {
f.should.equal("fine\r\n"); f.should.equal("fine\r\n");
} }
done(); done();
}
catch (e) {
done(e);
}
}); });
n1.receive({payload:"fine", filename:fileToTest}); n1.receive({payload:"fine", filename:fileToTest});