exercism / go-test-runner

GNU Affero General Public License v3.0
15 stars 17 forks source link

Fix for non-executed test reporting #100

Closed junedev closed 1 year ago

junedev commented 1 year ago

Adding non-executed tests to the test results caused issues for some practice exercises: https://forum.exercism.org/t/go-track-series-exercise-problem/4503

This changes fixes this by only adding non-exectued tests to the results if the task ids (v3 UI) were enabled.

The PR also contains some function comments that I forgot to commit with my last PR.

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 4398844448


Changes Missing Coverage Covered Lines Changed/Added Lines %
testrunner/execute.go 8 12 66.67%
<!-- Total: 8 12 66.67% -->
Totals Coverage Status
Change from base Build 4364268725: -0.3%
Covered Lines: 404
Relevant Lines: 622

💛 - Coveralls