我如何（单元）测试基于静态方法调用的类行为？

Question

我有这两个课程：

AbstractTaskDispatcher

<?php

declare(strict_types=1);

namespace MyExample;

abstract class AbstractTaskDispatcher
{
    public final function getResult(Task $task) : Result
    {
        if($worker = $this->getWorker($task))
            return $worker->getResult();
        else
            return Result::getUnprocessableTaskResult();
    }

    abstract protected function getWorker(Task $task) : Worker;
}

?>

结果

<?php

declare(strict_types=1);

namespace MyExample;

class Result
{
    private $code;

    public function __construct(int $code = 0)
    {
        $this->code = $code;
    }

    public static function getUnprocessableTaskResult() : Result
    {
        return new Result(1000);
    }

    public function getCode() : int
    {
        return $this->code;
    }
}

?>

我想用 PHPUnit 编写一个单元测试来确保 AbstractTaskDispatcher::getResult() 在没有找到合适的 Worker 来处理任务时返回 Result::getUnprocessableTaskResult()。

我不想这样做：

1. 安排：$expectedResult = Result::getUnprocessableTaskResult();
2. 动作：$result = $dispatcherStub->getResult(New Task());
3. 断言：assertEquals($result, $expectedResult);

因为它依赖于 Result 类的实现，不会是单元测试。

我试着做点什么：

<?php

    use PHPUnit\Framework\TestCase;
    use MyExample as ex;

    class AbstractDispatcherTest extends TestCase
    {
        public function test_getResultSouldReturnUnprocessableTaskResultIfNoWorkerFound()
        {
            $dispatcher = $this->getMockForAbstractClass(ex\AbstractDispatcher::class);
            $arbitraryCode = 6666;
            $expectedResult = new ex\Result($arbitraryCode);
            $resultClass = $this->getMockClass('Result', ['getUnprocessableTaskResult']);
            $resultClass::staticExpects($this->any())
                ->method('getUnprocessableTaskResult')
                ->will($this->returnValue($expectedResult));

            $result = $dispatcher->getResult(new ex\Task([]));

            $this->assertEquals($expectedResult, $result);
        }
    }

?>

但 staticExpects() 方法已被弃用，并且在当前 PHPUnit 版本中不再存在。

我该如何编写这个测试？

Answer 1

你可以简单地测试如下：

public function test_getResultSouldReturnUnprocessableTaskResultIfNoWorkerFound()
{
    $dispatcher = $this->getMockForAbstractClass(ex\AbstractTaskDispatcher::class);
    $dispatcher->expects($this->once())
        ->method('getWorker')
        ->willReturn(false);

    $result = $dispatcher->getResult(new ex\Task([]));

    // Unuseful: this is implicit by the method signature
    $this->assertInstanceOf(ex\Result::class, $result);
    $this->assertEquals(1000, $result->getCode());
}

注意：我将 AbstractTaskDispatcher 类的方法定义更改如下，以便返回 false 值：

/**
 * @param Task $task
 * @return Result|false The Result of the task or false if no suitable Worker is found to process the Task
 */
abstract protected function getWorker(Task $task);

编辑：

正如您评论的那样，您不能检查如下而不是硬编码结果代码：

    $this->assertEquals(ex\Result::getUnprocessableTaskResult(), $result);
    // Or
    $this->assertEquals(ex\Result::getUnprocessableTaskResult()->getCode(), $result->getCode());

希望有帮助