NotificationsYou must be signed in to change notification settings
Fork1k
Star11.2k

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

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

bryphe-coder merged 5 commits intomainfrombryphe/fix/244/jest-datadog

Feb 14, 2022

Merged

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

bryphe-coder merged 5 commits intomainfrombryphe/fix/244/jest-datadog

Feb 14, 2022

Conversation

Copy link

Contributor

bryphe-coder commentedFeb 14, 2022

Fixes#244

Addsjest-junit
Configuresjest-junit to outputjunit.xml insite/test_results
Uploads the emittedjunit.xml to datadog as part of thetest/js workflow

bryphe-coder added3 commits

February 14, 2022 17:49

Add jest-junit dependency

c80ce77

Generate junit.xml; ignore in git

4a51c05

Add datadog reporting for front-end tests

096edd2

bryphe-coder self-assigned this

Feb 14, 2022

Formatting

9a71e8e

kylecarbs approved these changes

Feb 14, 2022

View reviewed changes

Copy link

Member

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!

Copy link

codecovbot commentedFeb 14, 2022•
edited
Loading

Codecov Report

Merging#286 (c59a8cc) intomain (c5eed90) willincrease coverage by3.08%.
The diff coverage isn/a.

@@            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%> (ø)`
... and54 more

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last updatec5eed90...c59a8cc. Read thecomment docs.

Install go, so the datadog helper script can run

c59a8cc

Copy link

ContributorAuthor