Skip to content

Conversation

withinfocus
Copy link
Contributor

@withinfocus withinfocus commented Jan 10, 2023

Type of change

- [ ] Bug fix
- [ ] New feature development
- [ ] Tech debt (refactoring, code cleanup, dependency upgrades, etc)
- [X] Build/deploy pipeline (DevOps)
- [ ] Other

Objective

Utilizes a popular Action to process .NET test results and upload them as a report artifact. Much easier to parse results than working through the commandline output.

Code changes

  • .github/workflows/test.yml: Additional step for report processing
  • jest.config.js: Configuration of new reporter
  • package.json: New package to provide JUnit report results

Before you submit

  • Please add unit tests where it makes sense to do so (encouraged but not required)
  • If this change requires a documentation update - notify the documentation team
  • If this change has particular deployment requirements - notify the DevOps team

@withinfocus withinfocus force-pushed the testreport branch 5 times, most recently from 1c5af63 to 8b6085c Compare January 10, 2023 19:00
@withinfocus withinfocus marked this pull request as ready for review January 10, 2023 19:12
@joseph-flinn
Copy link
Contributor

Pipeline changes look good for me. I'll let @Hinton do the final confirmation on package updates.

Copy link
Member

@Hinton Hinton left a comment

Choose a reason for hiding this comment

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

It would be nice to figure out why the report ends up in weird places. But this is still a large positive so lets get it merged and we can evaluate it in the future.

@withinfocus withinfocus merged commit 4be2989 into master Jan 11, 2023
@withinfocus withinfocus deleted the testreport branch January 11, 2023 14:01
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.

3 participants