OCMock асинхронный блок обратного вызова - PullRequest
Новая
       21

OCMock асинхронный блок обратного вызова

4 голосов
/ 01 февраля 2011

Я создаю небольшую библиотеку для обработки операций загрузки и выгрузки файлов и пытаюсь интегрировать в нее набор тестов. Вместо того, чтобы использовать методы обратного вызова делегата, я обрабатываю асинхронные ответы в блоке обработчика завершения следующим образом: 

  SyncKit *engine = [[SyncKit alloc] init];
  NSURL *localFilePath = [NSURL URLWithString:@"/Users/justin/Desktop/FileTest.png"];

  [engine uploadFileWithFilename:@"FileTest.png" toRemotePath:@"/" fromLocalPath:localFilePath withCompletionHandler:^(id response, NSError *error) {
    if (error)
    {
      NSLog(@"error = %@", error);
      return;
    }

    NSLog(@"File uploaded and return JSON response = %@", response);
  }];

Базовый метод uploadFileWithFilename... выглядит так: 

- (void)uploadFileWithFilename:(NSString *)filename toRemotePath:(NSString *)remotePath fromLocalPath:(NSURL *)localPath withCompletionHandler:(SKCompletionHandler)handler
{
  if ((![[NSFileManager defaultManager] fileExistsAtPath:[localPath path]]))
  {
    NSDictionary *userInfo = [NSDictionary dictionaryWithObject:localPath forKey:@"localPath"];
    NSError *error = [NSError errorWithDomain:SKDropboxErrorDomain code:SKDropboxErrorFileNotFound userInfo:userInfo];
    handler(nil, error);
    return;
  }

  // path is the directory the file will be uploaded to, make sure it doesn't have a trailing /
  // (unless it's the root dir) and is properly escaped
  NSString *trimmedPath;
  if (([remotePath length] > 1) && ([remotePath characterAtIndex:[remotePath length] - 1] == '/')) 
  {
    trimmedPath = [remotePath substringToIndex:[remotePath length] - 1];
  } 
  else if ([remotePath isEqualToString:@"/"])
  {
    trimmedPath = @"//";
  }
  else 
  {
    trimmedPath = remotePath;
  }

  NSString *escapedPath = [NSString escapePath:trimmedPath];
  NSString *fullPath = [NSString stringWithFormat:@"/files/dropbox%@", escapedPath];      
  NSString *urlString = [NSString stringWithFormat:@"%@://%@/%@%@", kSKProtocolHTTPS, kSKDropboxAPIContentHost, kSKDropboxAPIVersion, fullPath];

  NSDictionary *params = [NSDictionary dictionaryWithObjectsAndKeys:filename, @"file", nil];
  NSString *body = [params convertToURIParameterString];

  NSURL *url = nil;
  if ([body length] == 0)
  {
    url = [NSURL URLWithString:[NSString stringWithFormat:@"%@", urlString]];
  }
  else
  {
    url = [NSURL URLWithString:[NSString stringWithFormat:@"%@?%@", urlString, body]];
  }

  __block ASIFormDataRequest *request = [ASIFormDataRequest requestWithURL:url];
  request.delegate = self;
  request.requestMethod = kSKMethodPOST;

  [request addFile:[localPath path] forKey:@"file"];

  //
  // Dropbox uses OAuth to handle its authentication, so we need to pass along the requested
  // tokens and secrets so that we get our stuff back.
  //  
  NSUserDefaults *defaults = [NSUserDefaults standardUserDefaults];
  NSString *token = [defaults objectForKey:@"oauth_token"];
  NSString *secret = [defaults objectForKey:@"oauth_secret"];  

  [request buildPostBody]; 
  NSData *authBody = request.postBody; 

  NSString *header = OAuthorizationHeader(url, request.requestMethod, authBody, kOAuthConsumerKey, kOAuthConsumerSecret, token, secret);  
  [request addRequestHeader:@"Authorization" value:header];  

  [request setCompletionBlock:^{
    NSDictionary *result = (NSDictionary *)[[request responseString] JSONValue];    
    [self.activeUploads removeObjectForKey:remotePath];
    handler(result, nil);
  }];

  [request setFailedBlock:^{
    NSError *error = request.error;
    [self.activeUploads removeObjectForKey:remotePath];
    handler(nil, error);
  }];

  [self.activeUploads setObject:request forKey:remotePath];

  [self.queue addOperation:request];
}

Я видел один пример , где парень использовал определение препроцессора и вставлял OCMock в реальную кодовую базу. Это кажется мне неправильным.

Какова была бы лучшая стратегия для тестирования такого кода?

1 Ответ

1 голос
/ 25 марта 2011

Этот ответ не относится конкретно к OCMock, поэтому он может быть не тем, что вы ищете, но ...

Я бы сделал что-то вроде этого: 

__block BOOL testPassed = NO;

[engine uploadFileWithFilename:@"FileTest.png" 
                  toRemotePath:@"/" 
                 fromLocalPath:localFilePath 
         withCompletionHandler:^(id response, NSError *error) {
    if (error)
    {
      return;
    }

    testPassed = YES;
  }];

[[NSRunLoop mainRunLoop] runMode:NSDefaultRunLoopMode 
                      beforeDate:[[NSDate date] dateByAddingTimeInterval:10]];

// make sure that testPassed is YES...

Таким образом, вы будете блокировать, пока один из обратных вызовов не попадет в основной цикл выполнения.

...