2

在重构我的代码库时，我发现了一段代码，我想将其提取到一个单独的类中。

它将为用户个人资料图片提供缩略图。该类（我们将其命名为UserImageManager）将包含 的实例UserMapper和 的实例 ImageRenderer。

现在我确实有一种方法可以检索用户个人资料图片的缓存 url。该方法从 中检索一个User对象，并使用存储在该对象中的图像路径UserMapper触发该render方法 。ImageRendererUser

我为这个小函数编写了一个测试，模拟了这个测试用例ImageRenderer，现在我没有一个断言。断言函数的返回值没有任何意义，因为它返回缓存文件路径，该路径由模拟的ImageRenderer.

现在我的问题是：即使没有断言，它仍然是有效的单元测试吗？或者我应该如何重写我的测试，而不仅仅是检查对模拟对象的调用？

// The unit test
public function renderImageWillReturnCacheImagePath() {
  $this->_userImageManager->setImage(
    // Returns the ImageRenderer mock
    $this->_getImageMock(BASE_PATH . 'uploads/2012-06-06-wegmeister-91792.png', false, false)
  );

  // Invokes the function
  $image = $this->_userImageManager->render('wegmeister');
}

// The method to be tested
public function render($username, $width = false, $height = false) {
  $user = $this->_userMapper->getUserByUsername($username);

  if($user !== false && trim($user->getImage()) !== '') {
    $srcImage = $this->_folderUserImages . $user->getImage();
  }
  else {
    $srcImage = $this->_placeholderUserImage;
  }

  $dstImage = $this->_image->render($srcImage, 'jpg', $width, $height);
  if ($dstImage === false || empty($dstImage)) {
    throw new UnexpectedValueException('ImageRenderingFailed');
  }

  return $dstImage;
}
4

1 回答 1

2

没有断言的测试是不完整的测试。

如果您无法想象断言什么以及如何断言，我会说这意味着您要测试的方法（单元）的结果是未定义的。

由于您没有显示任何规范，也没有告诉该函数返回哪种类型，所以我可以想象您可以测试的唯一一件事是是否在需要时抛出异常。

此外，看起来该方法做得太多而且太复杂。所以我不会把时间花在创建测试上，而是把时间花在重构类上。

将来您应该在编写代码之前编写测试，这样您就不会遇到这种情况。

于 2012-06-14T16:28:11.637 回答