fix: std/testing/runner.ts and deno test (#4392)

After splitting "failFast" and "exitOnFail" arguments, there was a situation where failing tests did not exit with code 1.

* fixed argument value passed to Deno.runTests() in deno test
* fixed argument value passed to Deno.runTests() in std/testing/runner.ts
* added integration tests for deno test to ensure failFast and exitOnFail work as expected
* don't write test file to file system, but keep it in memory

1 files changed, 2 insertions, 3 deletions

diff --git a/std/testing/runner.ts b/std/testing/runner.ts
index 65b7be66d..9aa31cb4a 100755
--- a/std/testing/runner.ts
+++ b/std/testing/runner.ts
@@ -269,7 +269,6 @@ async function main(): Promise<void> {
   const exclude =
     parsedArgs.exclude != null ? (parsedArgs.exclude as string).split(",") : [];
   const allowNone = parsedArgs["allow-none"];
-  const exitOnFail = parsedArgs.failfast;
   const disableLog = parsedArgs.quiet;
 
   try {
@@ -277,8 +276,8 @@ async function main(): Promise<void> {
       include,
       exclude,
       allowNone,
-      exitOnFail,
-      disableLog
+      disableLog,
+      exitOnFail: true
     });
   } catch (error) {
     if (!disableLog) {