方法代码如下:
- (void)downloadSomething:(NSString *)url {
Downloader *downloader = [[Downloader alloc] initWithUrl:url];
NSData *data = [downloader download];
FileCache *cache = [[FileCache alloc] initWithFile:@"download.cache"];
cache.data = data;
[cache save];
}
我想我应该模拟下载器和文件缓存以验证它是否运行良好。我曾考虑过这样的更改签名:downloadSomething:(NSString *)url downloader:(Downloader *)downloader cache:(FileCache *)cache
,但在调用此方法之前似乎必须做很多工作,这不是我想要的。
我正在使用ocmockito。
此外,有没有使编写代码更易于测试的指南?
编辑: 2017-01-16 14:54:23
编写两个方法是个好主意吗:
- (void)updateCacheWithUrl:(NSString *)url
downloader:(Downloader *)downloader
fileCache:(FileCache *)fileCache; // for testing
- (void)updateCacheWithUrl:(NSString *)url; // call above method with (url, nil, nil);
当协作者在方法中实例化时,这将创建紧密耦合。测试这一点的难度导致我们探索其他设计。
一种方法是将它们传入。这是我通常会做的。然后,我将创建一个更简单的版本,为生产代码提供默认对象。
但是在您的示例中,url
被传递给Downloader
,这使得这更加困难。这表明目前的downloadSomething:
设计违反了单一责任原则。它正在做两件事:下载和缓存。
因此,将这些责任分开可能会使事情更容易测试。