Skip to content
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

Add unit tests GitHub Action config #109

Merged

Conversation

EdmondChuiHW
Copy link

@EdmondChuiHW EdmondChuiHW commented Aug 29, 2024

Summary

Add GitHub Action config to this repo for running unit tests automatically upon pushing.

The existing test failure was fixed via #104.

Test plan

This PR is the test 🤯

Screenshot showing the successful run of the unit test GitHub Action job for this PR

GitHub Action run result: https://github.com/facebookexperimental/rn-chrome-devtools-frontend/actions/runs/10616954789/job/29428603293?pr=109

Upstreaming plan

  • This commit should be sent as a patch to the upstream devtools-frontend repo. I've reviewed the contribution guide.
  • This commit is React Native-specific and cannot be upstreamed.

- name: Run unit tests
run: |
cd src
npm run unittest -- --expanded-reportin

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

expanded-reporting
G in the end is missing :)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Both works for some reason lol. I guess Karma is resilient to typos haha

screenshot showing the command running successfully with the flag '--expanded-reportin' with a missing 'g'

Added the g back.

image
https://www.dictionary.com/e/slang/they-see-me-rollin/

@EdmondChuiHW EdmondChuiHW merged commit d3a8d98 into facebookexperimental:main Aug 29, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants