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

Add workaround for timers triggering too early in test

Browse Source
This commit is contained in:
Håkon Løvdal 2023-09-24 18:01:27 +02:00
parent 0c2ab13c48
commit 34e8d2b051
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
test/nodes/core/function/10-function_spec.js
View File

@ -1719,6 +1719,9 @@ describe('function node', function() {
it('should delay handling messages until init completes', function(done) {
const timeout_ms = 200;
// Since helper.load uses process.nextTick timers might occasionally finish
// a couple of milliseconds too early, so give some leeway to the check.
const timeout_check_margin = 5;
var flow = [{id:"n1",type:"function",wires:[["n2"]],initialize: `
return new Promise((resolve,reject) => {
setTimeout(resolve, ${timeout_ms});
@ -1735,9 +1738,9 @@ describe('function node', function() {
receivedMsgs.push(msg)
if (receivedMsgs.length === 5) {
let deltas = receivedMsgs.map(msg => msg.delta);
var errors = deltas.filter(delta => delta < timeout_ms)
var errors = deltas.filter(delta => delta < (timeout_ms - timeout_check_margin))
if (errors.length > 0) {
done(new Error(`Message received before init completed - delta values ${JSON.stringify(deltas)} expected to be > ${timeout_ms}`))
done(new Error(`Message received before init completed - delta values ${JSON.stringify(deltas)} expected to be > ${timeout_ms - timeout_check_margin}`))
} else {
done();
}