Update failedTests.properties on success (#2854)

I got tricked by `failedTests.properties` not updating when tests were
successful.

Author: AndreasArvidsson

packages/test-harness/src/runAllTests.ts

@@ -77,10 +77,11 @@ async function runTestsInDir(
       const failedTests: string[] = [];
 
       const runner = mocha.run((failures) => {
+        if (shouldLogFailedTests()) {
+          logFailedTests(failedTests);
+        }
+
         if (failures > 0) {
-          if (shouldLogFailedTests()) {
-            logFailedTests(failedTests);
-          }
           reject(`${failures} tests failed.`);
         } else {
           resolve();