Skip to content

Commit 2960223

Browse files
committed
Update error handling of some tests
1 parent a57b3d3 commit 2960223

1 file changed

Lines changed: 10 additions & 5 deletions

File tree

Tests/OktaDirectAuthTests/FactorStepHandlerTests.swift

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -55,8 +55,10 @@ final class FactorStepHandlerTests: XCTestCase {
5555
let handler = try await factor.stepHandler(flow: flow,
5656
openIdConfiguration: openIdConfiguration,
5757
loginHint: loginHint)
58-
let tokenStepHandler = try XCTUnwrap(handler as? TokenStepHandler)
59-
let request = try XCTUnwrap(tokenStepHandler.request as? TokenRequest)
58+
let tokenStepHandler = try XCTUnwrap(handler as? TokenStepHandler,
59+
"Expected a TokenStepHandler")
60+
let request = try XCTUnwrap(tokenStepHandler.request as? TokenRequest,
61+
"Expected a TokenRequest")
6062
XCTAssertEqual(request.clientConfiguration.clientId, client.configuration.clientId)
6163

6264
if let loginHint = loginHint {
@@ -118,7 +120,8 @@ final class FactorStepHandlerTests: XCTestCase {
118120
let handler = try await factor.stepHandler(flow: flow,
119121
openIdConfiguration: openIdConfiguration,
120122
loginHint: loginHint)
121-
let tokenStepHandler = try XCTUnwrap(handler as? OOBStepHandler<T>)
123+
let tokenStepHandler = try XCTUnwrap(handler as? OOBStepHandler<T>,
124+
"Expected a OOBStepHandler of type \(type(of: factor))")
122125
if let loginHint = loginHint {
123126
XCTAssertEqual(tokenStepHandler.loginHint, loginHint)
124127
} else {
@@ -267,11 +270,13 @@ final class FactorStepHandlerTests: XCTestCase {
267270
return
268271
}
269272

270-
let tokenBody = try XCTUnwrap(urlSession.request(matching: "/token")?.bodyString)
273+
let tokenBody = try XCTUnwrap(urlSession.request(matching: "/token")?.bodyString,
274+
"Cannot get token request body")
271275
let tokenParams = tokenBody.urlFormDecoded()
272276

273277
XCTAssertEqual(tokenParams["grant_type"],
274-
"urn:okta:params:oauth:grant-type:oob")
278+
"urn:okta:params:oauth:grant-type:oob",
279+
"The token grant_type should be oob")
275280
}
276281

277282
func testPrimaryOOBBindingTransferFail() async throws {

0 commit comments

Comments
 (0)