exercism / go-test-runner

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

Handle stack overflow correctly #64

Closed junedev closed 2 years ago

junedev commented 2 years ago

Fixes #63

(The code currently includes #65 so I can see the tests are green. #65 should be merged first, I will then merge main here.)

This PR includes a cleanup for some test cases that I overlooked in a previous PR, see https://github.com/exercism/go-test-runner/pull/64/files#r800104193.

coveralls commented 2 years ago

Pull Request Test Coverage Report for Build 1820123410


Changes Missing Coverage Covered Lines Changed/Added Lines %
testrunner/execute.go 8 10 80.0%
<!-- Total: 8 10 80.0% -->
Totals Coverage Status
Change from base Build 1814035975: 0.4%
Covered Lines: 342
Relevant Lines: 455

💛 - Coveralls