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

maint: upload test results to CircleCI & add Go 1.22 to test matrix #108

Merged
merged 2 commits into from
Feb 14, 2024

Conversation

robbkidd
Copy link
Member

@robbkidd robbkidd commented Feb 14, 2024

We upload test reports for smoke tests for friendlier error reporting and tracking flaky tests. This lets us do that for unit tests, too.

Which problem is this PR solving?

  • Friendlier error reporting in CI.
  • Test coverage for recently release Go 1.22.

Short description of the changes

  • gotestsum is on Circle's current Go runners, so we can use it to produce xUnit XML test run reports to upload to Circle.
  • New Go is out. Let's test against it.

How to verify that this has the expected result

A unit test build's "TESTS" tab at Circle will have data instead of reporting "0 test results uploaded".

gotestsum is on Circle's current Go runners, so we can use it to
produce xUnit XML test run reports for friendlier error reporting.
@robbkidd robbkidd requested a review from a team as a code owner February 14, 2024 17:22
@robbkidd robbkidd self-assigned this Feb 14, 2024
@robbkidd robbkidd added version: no bump A PR with maintenance or doc changes that aren't included in a release. no-changelog Omit this PR from changelog/release notes. type: maintenance The necessary chores to keep the dust off. labels Feb 14, 2024
@robbkidd robbkidd changed the title maint: upload test results to CircleCI maint: upload test results to CircleCI & add Go 1.22 to test matrix Feb 14, 2024
@robbkidd robbkidd merged commit 9a50544 into main Feb 14, 2024
9 checks passed
@robbkidd robbkidd deleted the robb.test-results-in-ci branch February 14, 2024 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog Omit this PR from changelog/release notes. type: maintenance The necessary chores to keep the dust off. version: no bump A PR with maintenance or doc changes that aren't included in a release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants