-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
feat: add api spec test infrastructure #9356
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. |
I have read the CLA Document and I hereby sign the CLA |
25b23ca
to
5f0d013
Compare
👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎ This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. Ignoring: Next stepsTake a deeper look at the dependencyTake a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support [AT] socket [DOT] dev. Remove the packageIf you happen to install a dependency that Socket reports as Known Malware you should immediately remove it and select a different dependency. For other alert types, you may may wish to investigate alternative packages or consider if there are other ways to mitigate the specific risk posed by the dependency. Mark a package as acceptable riskTo ignore an alert, reply with a comment starting with |
3a3fb51
to
96a9a31
Compare
@shanejonas @adonesky1 I take it that we intend on including this as part of our test pipeline? If so, we should add the tests to the smoke pipeline. So that you know, our CI platform (bitrise) operates slightly differently than circleCI. Here is a visual representation of what our smoke pipeline looks like on bitrise: Now here are the steps we can do for now just as a proof of concept and iterate in the future:
Feel free to reach out if anything. |
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.
good stuff @shanejonas I left a few comments. Sharing https://github.com/MetaMask/contributor-docs/blob/main/docs/e2e/mobile-e2e-guidelines.md#step-2-create-the-networks-page-object in case you want a quick run down on how we create page objects within our tests!
49954d7
to
9f23ce6
Compare
@@ -56,6 +56,7 @@ | |||
"test": "yarn test:unit && yarn test:e2e", | |||
"test:unit": "jest ./app/ ./locales/", | |||
"test:unit:update": "time jest -u ./app/", | |||
"test:api-specs": "detox test -c ios.sim.apiSpecs", |
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.
getting this error when running things for the first time using just detox test -c ios.sim.apiSpecs
Failed to find the app binary at:
/Users/jiexi/Projects/metamask-mobile/ios/build/Build/Products/Debug-iphonesimulator/MetaMask.app
@shanejonas nice work on this! Just curious, were you ever able to get the mock server working? |
…lick behind confirmations.
@Cal-L mock server stuff is working now, but an unrelated android e2e suite is failing. |
@SocketSecurity ignore npm/json-schema-ref-parser@6.1.0 |
Bitrise✅✅✅ Commit hash: 576cb10 Note
|
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.
🌮 🌮 🌮 Looks good!
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.
LGTM
Quality Gate passedIssues Measures |
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.
LGTM!
Description
This PR adds tests using api-specs via the @open-rpc/test-coverage tool.
Related issues
Fixes: https://github.com/MetaMask/MetaMask-planning/issues/2238
Manual testing steps
yarn setup
yarn test:e2e:ios:debug:build
yarn test:api-specs
Screenshots/Recordings
Pre-merge author checklist
Pre-merge reviewer checklist