From a056b15a165be1d6552768f6184edf2c4cb2912a Mon Sep 17 00:00:00 2001 From: RikedyP Date: Thu, 25 Sep 2025 16:58:10 +0100 Subject: [PATCH 1/2] clarify conditions for overall error status --- building/tooling/test-runners/interface.md | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/building/tooling/test-runners/interface.md b/building/tooling/test-runners/interface.md index 04e080a9..e6ea881a 100644 --- a/building/tooling/test-runners/interface.md +++ b/building/tooling/test-runners/interface.md @@ -52,9 +52,7 @@ The following overall statuses are valid: - `fail`: At least one test has the status `fail` or `error` - `error`: No test was executed (this usually means a compile error or a syntax error) -The `error` status should _only_ be used if **none of the tests were run**. -For compiled languages this is generally a result of the code not being able to compile. -For interpreted languages, this is generally the result of a syntax error that stops the file from parsing. +The `error` status should _only_ be used if **all of the tests errored**. For compiled languages this is generally a result of the code not being able to compile. For interpreted languages this is a runtime error, such as a syntax error that stops the file from parsing. #### Message From e3296442f7047678f69e8c810c5485f0e003140d Mon Sep 17 00:00:00 2001 From: Rich Park Date: Mon, 29 Sep 2025 10:49:22 +0100 Subject: [PATCH 2/2] Update building/tooling/test-runners/interface.md Co-authored-by: Glenn Jackman --- building/tooling/test-runners/interface.md | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/building/tooling/test-runners/interface.md b/building/tooling/test-runners/interface.md index e6ea881a..5d8132df 100644 --- a/building/tooling/test-runners/interface.md +++ b/building/tooling/test-runners/interface.md @@ -52,7 +52,9 @@ The following overall statuses are valid: - `fail`: At least one test has the status `fail` or `error` - `error`: No test was executed (this usually means a compile error or a syntax error) -The `error` status should _only_ be used if **all of the tests errored**. For compiled languages this is generally a result of the code not being able to compile. For interpreted languages this is a runtime error, such as a syntax error that stops the file from parsing. +The `error` status should _only_ be used if **all of the tests errored**. +For compiled languages this is generally a result of the code not being able to compile. +For interpreted languages this is a runtime error, such as a syntax error that stops the file from parsing. #### Message