node-red
node-red copied to clipboard
let delay node handle both flush then reset
and add tests
- [x] Bugfix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)
Proposed changes
Now that we support a numeric parameter for flush (rather than flush all) - then it now makes sense that if the users specifies flush and reset that we honour the flush part before the reset. (Current behaviour is to ignore the flush and just reset). This PR fixes that so the flush occurs first. And adds tests.
Checklist
- [x] I have read the contribution guidelines
- [x] For non-bugfix PRs, I have discussed this change on the forum/slack team.
- [x] I have run
gruntto verify the unit tests pass - [x] I have added suitable unit tests to cover the new/changed functionality