exercism / go-test-runner

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

Switch to "integration" tests #96

Closed junedev closed 1 year ago

junedev commented 1 year ago

Erik mentioned it was hard to review our test runner changes because we do not have any test cases that show the actual resulting json output.

While the new tests might need some updating when we increase the Go version, they help us to be "more sure" we do not introduce regressions while working on the test runner.

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 4286844697


Files with Coverage Reduction New Missed Lines %
testrunner/ast.go 3 72.13%
testrunner/execute.go 59 52.92%
<!-- Total: 62 -->
Totals Coverage Status
Change from base Build 4186204366: -11.03%
Covered Lines: 354
Relevant Lines: 562

💛 - Coveralls