我正在解析一些json以返回基本的字符串令牌或错误消息。
- (void)callBackWithVerifyHttpResponse:(NSData *)response
{
SomeResult *result = [self.parser parseVerifyHttpResponseAndReturnResult:response];
if (result.token) { [self.delegate callBackWithToken:result.token]; }
if (result.error) { [self.delegate callBackWithError:result.error]; }
}
证明这一点的测试
- (void)testVerifyCallbackInvokesErrorCallbackOnDelegateWhenParserReturnsError
{
SomeResult *result = [[SomeResult alloc] init];
result.error = @"fail";
[[self.delegate expect] callBackWithError:@"fail"];
[[self.delegate reject] callBackWithToken:OCMArg.any];
[[[self.parser stub] andReturn:result] parseVerifyHttpResponseAndReturnResult:nil];
[self.sut callBackWithVerifyHttpResponse:nil];
[self.delegate verify];
}
- (void)testVerifyCallbackInvokesTokenCallbackOnDelegateWhenParserReturnsToken
{
SomeResult *result = [[SomeResult alloc] init];
result.token = @"token";
[[self.delegate expect] callBackWithToken:@"token"];
[[self.delegate reject] callBackWithError:OCMArg.any];
[[[self.parser stub] andReturn:result] parseVerifyHttpResponseAndReturnResult:nil];
[self.sut callBackWithVerifyHttpResponse:nil];
[self.delegate verify];
}
一切都很好,直到我将其连接到实际的端点 - 只是发现除非我像下面这样修改回调 - 它正在调用两个回调(不是我希望的)
- (void)callBackWithVerifyHttpResponse:(NSData *)response
{
SomeResult *result = [self.parser parseVerifyHttpResponseAndReturnResult:response];
if (result.token != [NSNull null]) { [self.delegate callBackWithToken:result.token]; }
if (result.error != [NSNull null]) { [self.delegate callBackWithError:result.error]; }
}
所以 2 部分问题
为什么我不能写一个测试来证明这一点?每当我将错误或令牌设置为 NULL 或 NSNull 时,它都可以正常工作(但生产需要此代码才能正常工作)
为什么生产代码只有在我放置
!= [NSNull null]
时才失败条件(但是当我在模拟器中运行 NSLog 值时,我似乎除了<null>
之外什么也得不到?
请记住,令牌/错误属性在 SomeResult 对象上如下所示
@interface SomeResult : NSObject
@property (strong, nonatomic) NSString *token;
@property (strong, nonatomic) NSString *error;
@end
您的原始代码和测试期望nil
标记或错误,而不是[NSNull null]
。大概当您针对生产端点运行它时,您的解析器将值设置为 [NSNull null]
。
这些测试应随修改后的代码一起通过:
- (void)testVerifyCallbackInvokesErrorCallbackOnDelegateWhenParserReturnsError
{
SomeResult *result = [[SomeResult alloc] init];
result.error = @"fail";
result.token = [NSNull null];
[[self.delegate expect] callBackWithError:@"fail"];
[[self.delegate reject] callBackWithToken:OCMArg.any];
[[[self.parser stub] andReturn:result] parseVerifyHttpResponseAndReturnResult:nil];
[self.sut callBackWithVerifyHttpResponse:nil];
[self.delegate verify];
}
- (void)testVerifyCallbackInvokesTokenCallbackOnDelegateWhenParserReturnsToken
{
SomeResult *result = [[SomeResult alloc] init];
result.token = @"token";
result.error = [NSNull null];
[[self.delegate expect] callBackWithToken:@"token"];
[[self.delegate reject] callBackWithError:OCMArg.any];
[[[self.parser stub] andReturn:result] parseVerifyHttpResponseAndReturnResult:nil];
[self.sut callBackWithVerifyHttpResponse:nil];
[self.delegate verify];
}
当您处理可能会巧妙地改变其行为的 Web 服务和/或第三方解析器时,最好进行防御性编码。您可以同时处理nil
和NSNull
:
- (void)callBackWithVerifyHttpResponse:(NSData *)response
{
SomeResult *result = [self.parser parseVerifyHttpResponseAndReturnResult:response];
if (result.token && result.token != [NSNull null]) { [self.delegate callBackWithToken:result.token]; }
if (result.error && result.error != [NSNull null]) { [self.delegate callBackWithError:result.error]; }
}
您可能还希望将其设置为 if/else——要么先测试错误,如果出现错误,则从不设置令牌,或者在获得令牌时忽略错误。如果您不想同时调用这两个回调,请在代码路径中使其无法调用。