Conversation
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
|
🚀 🎸 🤘🏻 |
brad-decker
left a comment
There was a problem hiding this comment.
I'm going to do a more thorough review soon, but wanted to say that from a surface level this looks great. Number of tests match up good so I don't think we're missing anything. long live the reign of jest.
|
There are still some inconsistencies such as, we are still using enzyme as out component wrappers and sinon for some stubbing. Here I used |
brad-decker
left a comment
There was a problem hiding this comment.
LGTM, excited to get coverage minimums and path for ward for UI test coverage
Large PR to change the ui/'s tests runner to jest.