[pr] Fixes for issue #118 #123
[pr] Fixes for issue #118 #123
Conversation
stephen-palmer
commented
May 9, 2019
- Moved whitelist check from _onPut to _onBeginTransaction, Invalidating transactions from IPs that are not whitelisted as soon as they are created
- Have CacheBase process all transactions through the ReliabilityManager when enabled
- Ensure ReliabilityManager does not process invalid transactions
- Moved whitelist check from _onPut to _onBeginTransaction, Invalidating transactions from IPs that are not whitelisted as soon as they are created - Have CacheBase process all transactions through the ReliabilityManager when enabled - Ensure ReliabilityManager does not process invalid transactions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ A review job has been created and sent to the PullRequest network.
Check the status or cancel PullRequest code review here - or - cancel by adding [!pr] to the title of the pull request.
… bunch of test runner process args that were interfering with CLI handling.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
test/command_processor.js
Outdated
beforeEach(() => { | ||
this.cmdProc = new CommandProcessor(new CacheBase()); | ||
}); | ||
|
||
afterEach(() => { | ||
stubs.forEach(s => s.restore()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have you considered using a sandbox to easily restore all the stubs without having to track them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the tip - I upgraded Sinon and refactored to use the default sandbox.
…fying the test code.