[XCTestCase] Fix total number of tests output

#33 introduced
another regression that was not fixed in 4944003: the total number of test
runs was not being reported correctly--only the total number of
failures.

Fix the misreporting, as well as the incorrect verification in the
`FailingTestSuite` functional test.

Author: modocache

Sources/XCTest/XCTestCase.swift

@@ -63,12 +63,11 @@ extension XCTestCase {
                     if !failure.expected {
                         unexpectedFailures += 1
                     }
-
                     result = failures.count > 0 ? "failed" : "passed"
-                    XCTAllRuns.append(XCTRun(duration: duration, method: method, passed: failures.count == 0, failures: failures))
                 }
 
                 print("Test Case '\(method)' \(result) (\(printableStringForTimeInterval(duration)) seconds).")
+                XCTAllRuns.append(XCTRun(duration: duration, method: method, passed: failures.count == 0, failures: failures))
                 XCTFailureHandler = nil
             }
         }

Tests/Functional/FailingTestSuite/main.swift

@@ -13,7 +13,7 @@
 // CHECK: .*/Tests/Functional/FailingTestSuite/main.swift:54: error: FailingTestCase.test_fails_with_message : XCTAssertTrue failed - Foo bar.
 // CHECK: Test Case 'FailingTestCase.test_fails_with_message' failed \(\d+\.\d+ seconds\).
 // CHECK: Executed 3 tests, with 2 failures \(0 unexpected\) in \d+\.\d+ \(\d+\.\d+\) seconds
-// CHECK: Total executed 2 tests, with 2 failures \(0 unexpected\) in \d+\.\d+ \(\d+\.\d+\) seconds
+// CHECK: Total executed 4 tests, with 2 failures \(0 unexpected\) in \d+\.\d+ \(\d+\.\d+\) seconds
 
 #if os(Linux) || os(FreeBSD)
     import XCTest