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

replace go-junit-report with gotestsum #1495

Open
electricmonk opened this issue Dec 2, 2019 · 1 comment
Open

replace go-junit-report with gotestsum #1495

electricmonk opened this issue Dec 2, 2019 · 1 comment

Comments

@electricmonk
Copy link
Contributor

We spend 40 seconds in each build job to download go-junit-report, which is used to convert go test output to junit xml format readable bi circleci.

There's another tool, gotestsum which also supports this, which is available on all circleci machines.

Consider making the switch. @amir-arad already did that in https://github.com/orbs-network/boyarin

@amir-arad
Copy link

amir-arad commented Dec 2, 2019

gotestsum is built into circleci go docker image, not the machine.
technically I only use gotestsum in boyar E2E, which uses machine, so I have to install it explicitly as part of setting up the machine (in addition to re-installing go and nodejs)
see here

using it in a docker image should be a breeze, I just didn't get there yet.

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

No branches or pull requests

2 participants