Fix parsing for snapshot generation (#612)

There was a bug where if all tests passed it would obscure the detection
of a mismatch. Mismatched version should never trigger snapshot
auto-generation. This caused this
[PR](#611) to be erroneously
generated.

tests/parse/index.ts

@@ -89,18 +89,18 @@ const mergeTestFailureMetadata = (original: TestResult, incoming: TestResult) =>
     ([testFullName, incomingSuspectedFailureMode]) => {
       const originalSuspectedFailureMode = original.testFailureMetadata.get(testFullName);
       if (originalSuspectedFailureMode) {
-        // If the incoming test result is a pass, invalidate the failure mode
-        if (incoming.testResultStatus === "passed") {
-          original.testFailureMetadata.set(testFullName, "passed");
-          return;
-        }
-
         // If the incoming test result is a version mismatch, invalidate the failure mode
         if (incoming.version && original.version && incoming.version !== original.version) {
           original.testFailureMetadata.set(testFullName, "unknown");
           return;
         }
 
+        // If the incoming test result is a pass, invalidate the failure mode
+        if (incoming.testResultStatus === "passed") {
+          original.testFailureMetadata.set(testFullName, "passed");
+          return;
+        }
+
         // If the incoming test result is a failure and not assertion_failure, **invalidate the failure mode
         if (incomingSuspectedFailureMode !== "assertion_failure") {
           original.testFailureMetadata.set(testFullName, incomingSuspectedFailureMode);