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

fix: Integrate front-end jest tests with datadog reporting #286

Merged
merged 5 commits into from
Feb 14, 2022

Conversation

bryphe-coder
Copy link
Contributor

Fixes #244

  • Adds jest-junit
  • Configures jest-junit to output junit.xml in site/test_results
  • Uploads the emitted junit.xml to datadog as part of the test/js workflow

@bryphe-coder bryphe-coder self-assigned this Feb 14, 2022
Copy link
Member

@kylecarbs kylecarbs left a comment

Choose a reason for hiding this comment

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

So cool!

@codecov
Copy link

codecov bot commented Feb 14, 2022

Codecov Report

Merging #286 (c59a8cc) into main (c5eed90) will increase coverage by 3.08%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #286      +/-   ##
==========================================
+ Coverage   65.24%   68.32%   +3.08%     
==========================================
  Files          62      126      +64     
  Lines         679     6898    +6219     
  Branches       69       69              
==========================================
+ Hits          443     4713    +4270     
- Misses        222     1707    +1485     
- Partials       14      478     +464     
Flag Coverage Δ
unittest-go-macos-latest 66.62% <ø> (?)
unittest-go-ubuntu-latest 67.77% <ø> (?)
unittest-go-windows-latest 66.41% <ø> (?)
unittest-js 65.24% <ø> (ø)
Impacted Files Coverage Δ
cryptorand/numbers.go 46.90% <0.00%> (ø)
cli/projects.go 76.27% <0.00%> (ø)
cli/projectcreate.go 52.40% <0.00%> (ø)
httpapi/httpapi.go 68.11% <0.00%> (ø)
httpmw/provisionerjobparam.go 78.37% <0.00%> (ø)
database/postgres/postgres.go 70.73% <0.00%> (ø)
coderd/files.go 74.46% <0.00%> (ø)
coderd/provisionerdaemons.go 57.23% <0.00%> (ø)
coderd/projectimport.go 59.39% <0.00%> (ø)
database/time.go 0.00% <0.00%> (ø)
... and 54 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c5eed90...c59a8cc. Read the comment docs.

@bryphe-coder
Copy link
Contributor Author

I can see the test results make it into datadog now:
image

So seems like this is working 🎉

@bryphe-coder bryphe-coder merged commit 64c14de into main Feb 14, 2022
@bryphe-coder bryphe-coder deleted the bryphe/fix/244/jest-datadog branch February 14, 2022 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement build tracing for JavaScript tests
2 participants