frodovdr/node-red
1
0
Fork 0
mirror of https://github.com/node-red/node-red.git synced 2023-10-10 13:36:53 +02:00
Code Issues Releases Wiki Activity

Fix regression in Join node when manual joining array with msg.parts present

Browse Source 
Fixes #3096
This commit is contained in:
Nick O'Leary 2021-07-30 13:17:05 +01:00
parent 27ed81614b
commit 40233c7702
No known key found for this signature in database
GPG Key ID: 4F2157149161A6C9
2 changed files with 23 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
packages/node_modules/@node-red/nodes/core/sequence/17-split.js vendored
View File

@ -629,9 +629,6 @@ module.exports = function(RED) {
if (node.build === 'object') {
propertyKey = RED.util.getMessageProperty(msg,node.key);
}
if (msg.hasOwnProperty("parts")) {
propertyIndex = msg.parts.index;
}
}
if (msg.hasOwnProperty("reset")) {

24
test/nodes/core/sequence/17-split_spec.js
View File

@ -516,6 +516,28 @@ describe('JOIN node', function() {
n1.receive({payload:{a:1}});
});
});
it('should join things into an array ignoring msg.parts.index in manual mode', function(done) {
var flow = [{id:"n1", type:"join", wires:[["n2"]], count:3, joiner:",",mode:"custom"},
{id:"n2", type:"helper"}];
helper.load(joinNode, flow, function() {
var n1 = helper.getNode("n1");
var n2 = helper.getNode("n2");
n2.on("input", function(msg) {
try {
msg.should.have.property("payload");
msg.payload.should.be.an.Array();
msg.payload[0].should.equal(1);
msg.payload[1].should.equal(true);
//msg.payload[2].a.should.equal(1);
done();
}
catch(e) {done(e);}
});
n1.receive({payload:1, parts: {index: 3}});
n1.receive({payload:true, parts: {index: 0}});
n1.receive({payload:{a:1}, parts: {index: 9}});
});
});
it('should join things into an array after a count with a buffer join set', function(done) {
var flow = [{id:"n1", type:"join", wires:[["n2"]], count:3, joinerType:"bin", joiner:"" ,mode:"custom"},
@ -1646,7 +1668,7 @@ describe('JOIN node', function() {
});
});
it('should handle join an array when using msg.parts and duplicate indexed parts arrive', function (done) {
it('should handle join an array when mode is auto and duplicate indexed parts arrive', function (done) {
var flow = [{ id: "n1", type: "join", wires: [["n2"]], joiner: "[44]", joinerType: "bin", build: "array", mode: "auto" },
{ id: "n2", type: "helper" }];
helper.load(joinNode, flow, function () {