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

Added codecov.io tests via JUnit tests conversion #36

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

nikolaydubina
Copy link

You guys are amazing! I have been long time advocate and implemented myself similar systems for Continuous Tests Monitoring as you guys just recently published. I already migrated all my open source Go projects (20+) to this setup and published how-to article in go-recipes (so expect more people will start using it soon! hehe).

Here I wanted to show how to do it in official example.

On choice of JUnit tests converter

  • gotestsum, it is widely used tool, but it is too big, it does other things too. it also requires non-github path for module, which raises eyebrows a little in open source Go community.
  • go-junit-report is developed by a guy from Google, which is nice. it is also very small and does one thing only for JUnit conversion, which is in spirit of UNIX philosophy. it also does not require any dependencies. so it is winner for me, albeit I mention other tool too in guide.
image

as for requests:

  • I think Timeline trend chart would be nice per test. (or at least histogram distribution of latency by time)

Thanks again! Nice work!

@nikolaydubina
Copy link
Author

cc: @thomasrockhu-codecov @arshamalh

@nikolaydubina
Copy link
Author

cc: @jstemmer

@thomasrockhu-codecov
Copy link
Contributor

@nikolaydubina thanks for this. I don't think we can merge this in as this has more functionality than we are expecting users when starting off using Codecov.

However, there might be something here with test results. I'm going to cc @rohan-at-sentry who might find this interesting

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.

2 participants