exercism / go-test-runner

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

change output of test-runner on compile error to adhere to the standard #12

Closed tehsphinx closed 4 years ago

tehsphinx commented 4 years ago

Fixes #11

tehsphinx commented 4 years ago

@ErikSchierboom: This change removes the extra test I created on a compile error to hold the error message and adds the error message on top level. The only output is then status and message. tests is null. Or should I omit that?

Example:

{
    "status": "error",
    "message": "./sample_test.go:37:1: missing return at end of function",
    "tests": null
}
ErikSchierboom commented 4 years ago

@tehsphinx Awesome! Could you make the tests an empty array perhaps?

tehsphinx commented 4 years ago

@ErikSchierboom

{
    "status": "error",
    "message": "./sample_test.go:37:1: missing return at end of function",
    "tests": []
}
ErikSchierboom commented 4 years ago

Thanks!