1

我正在解析一些 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 部分问题

  1. 为什么我不能写一个测试来证明这一点？每当我将错误或令牌设置为 NULL 或 NSNull 时，它都可以正常工作（但生产工作需要此代码）

  2. 为什么生产代码只会在条件条件下失败!= [NSNull null]（但我似乎什么也得不到，但是<null>当我在模拟器中运行它时 NSLog 值时？

请记住，SomeResult 对象上的令牌/错误属性如下所示

@interface SomeResult : NSObject

@property (strong, nonatomic) NSString *token;
@property (strong, nonatomic) NSString *error;

@end
4

1 回答 1

2

您的原始代码和测试期望 token 或 error 是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——首先测试错误，如果有错误则永远不要设置令牌，或者如果您获得令牌则忽略错误。如果您不想同时调用这两个回调，那么在您的代码路径中使其成为不可能。

于 2012-06-21T16:07:49.150 回答