-
Notifications
You must be signed in to change notification settings - Fork 1.3k
test: use pin tests #1405
test: use pin tests #1405
Conversation
|
||
require('./pin') | ||
require('./pin-set') | ||
// require('./key-exchange') |
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.
Does anyone know why these tests were never being run? @richardschneider ??
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.
@diasdavid No idea, the tests should be run.
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.
They weren't being picked up anywhere. I'll add them as a separate PR then.
License: MIT Signed-off-by: Alan Shaw <alan@tableflip.io>
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.
I upped the timeout for teardown for the pin set tests and everything is 💚
Thank you @alanshaw ❤️ |
No description provided.