bruno icon indicating copy to clipboard operation
bruno copied to clipboard

chore: enhance visibility for unit test results and coverage

Open lanthoor opened this issue 1 year ago • 2 comments

Description

Partially addresses #2901. Steps are required from the maintainers to close the issue (making checks "Required").

  • added junit test result reporter for unit tests
  • added coverage (text + cobertura) reporter for unit tests
  • optimized action to run unit tests in parallel
  • added unit test result reporting
  • added coverage reporting using coveralls (needs to be discussed, as Sonar is already used in closed source parts?)
    • current report here: https://coveralls.io/github/usebruno/bruno
  • synchronizes package.json and package-lock.json

Contribution Checklist:

  • [x] The pull request only addresses one issue or adds one feature.
  • [x] The pull request does not introduce any breaking changes
  • [x] I have added screenshots or gifs to help explain the change if applicable.
  • [x] I have read the contribution guidelines.
  • [x] Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.

Publishing to New Package Managers

Please see here for more information.

lanthoor avatar Aug 23 '24 05:08 lanthoor

@jwetzell @Its-treason Would like your opinions as well.

lanthoor avatar Aug 23 '24 06:08 lanthoor

I don't have anything to input, this would be something the actual Bruno team would likely have an opinion on though.

jwetzell avatar Aug 23 '24 16:08 jwetzell

Hey @lanthoor, thank you for the PR. I will get back to you once we have tested this!

sanjai0py avatar Aug 26 '24 06:08 sanjai0py

PR looks good. We are open to use Coveralls for the OSS repo.

A bit held up this week in cleaning up our PR backlog. I should have this merged next week.

helloanoop avatar Aug 26 '24 06:08 helloanoop

Let me know once you're ready for this. I'll update the branch and resolve conflicts.

lanthoor avatar Aug 26 '24 16:08 lanthoor

@helloanoop @sanjai0py any updates on this?

lanthoor avatar Sep 12 '24 04:09 lanthoor

@helloanoop I've updated the branch. Could you merge this, please? It's been sitting here for over 3 months now.

lanthoor avatar Nov 28 '24 04:11 lanthoor

This has sat here for 7 months. In closing this. If you need this feature, feel free to refer to this PR.

lanthoor avatar Mar 29 '25 16:03 lanthoor