-
Notifications
You must be signed in to change notification settings - Fork 56
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Test fixes #718
Test fixes #718
Conversation
f1f8a0b
to
ec7717d
Compare
|
||
MockRSA.prototype.exportKey = function() {}; | ||
|
||
module.exports = global.IS_TEST_ENV ? MockRSA : require('node-rsa'); |
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.
This allows us to mock RSA key creation in the tests. Previously, new NodeRSA(...)
was the longest running operation in the test suite, and was run several times by the provision and key tests
ec7717d
to
c6c9e45
Compare
Note that the run time I included above is 6838ms. I just ran on master and it took 33224ms. |
- don't use NodeRSA in tests - stub fs operations whenever possible - stub public key operations whenever possible - interval until activeSeeker is available Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
c6c9e45
to
288dd14
Compare
I'm still working on that appveyor nonsense |
Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
33ea67f
to
35146a1
Compare
Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
b2c57a2
to
d762c60
Compare
Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
A complete run should look like:
Specifically, it ends with 1065 assertions
Signed-off-by: Rick Waldron waldron.rick@gmail.com