Skip to content

Commit b6d36fa

Browse files
committed
Simplified throwable function evaluation and success case requirements.
1 parent 13094af commit b6d36fa

File tree

1 file changed

+3
-15
lines changed

1 file changed

+3
-15
lines changed

stdlib/public/SDK/XCTest/XCTest.swift

Lines changed: 3 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1030,28 +1030,17 @@ public func XCTAssertThrowsError<T>(_ expression: @autoclosure () throws -> T, _
10301030
}
10311031
}
10321032

1033-
public func XCTAssertNoThrow<T>(_ expression: @autoclosure () throws -> T, _ message: @autoclosure () -> String = "", file: StaticString = #file, line: UInt = #line, _ errorHandler: (_ error: Error) -> Void = { _ in }) {
1033+
public func XCTAssertNoThrow<T>(_ expression: @autoclosure () throws -> T, _ message: @autoclosure () -> String = "", file: StaticString = #file, line: UInt = #line) {
10341034
let assertionType = _XCTAssertionType.assertion_NoThrow
10351035

1036-
// evaluate expression exactly once
1037-
var caughtErrorOptional: Error?
1038-
1039-
let result = _XCTRunThrowableBlock {
1040-
do {
1041-
_ = try expression()
1042-
} catch {
1043-
caughtErrorOptional = error
1044-
}
1045-
}
1036+
let result = _XCTRunThrowableBlock { _ = try expression() }
10461037

10471038
switch result {
10481039
case .success:
1049-
guard let caughtError = caughtErrorOptional else {
1040+
guard let error = error else {
10501041
return
10511042
}
10521043

1053-
_XCTRegisterFailure(true, "XCTAssertNoThrow failed: threw error \"\(caughtError)\"", message, file, line)
1054-
10551044
case .failedWithError(let error):
10561045
_XCTRegisterFailure(true, "XCTAssertNoThrow failed: threw error \"\(error)\"", message, file, line)
10571046

@@ -1061,7 +1050,6 @@ public func XCTAssertNoThrow<T>(_ expression: @autoclosure () throws -> T, _ mes
10611050
case .failedWithUnknownException:
10621051
_XCTRegisterFailure(true, _XCTFailureDescription(assertionType, 2), message, file, line)
10631052
}
1064-
10651053
}
10661054

10671055
#if XCTEST_ENABLE_EXCEPTION_ASSERTIONS

0 commit comments

Comments
 (0)