phpunit-使用在不同测试中测试的类的测试类

Question

我找不到有关此主题的好信息，所以我正在召唤这个问题。我们正在为应用程序编写测试。现在有使用其他类的嵌套类，依此类推。每个班级都有自己的测试。例如：

// Some class
class SomeCalculator {
    public function add($a, $b): int {
        return $a + $b;
    }
}

// Its trivial test
class SomeCalculatorTest {
    public function testAdding() {
        $calc = new SomeCalculator();
        $this->assertEquals(2, $calc->add(1, 1));
    }
}

现在考虑一下：

// Some other class that uses SomeCalculator()
class SomePerson {
    protected $name;
    protected $age;

    public function __construct($name, $a, $b) {
        $this->name = $name;
        $this->age = (new SomeCalculator())->add($a, $b);
    }
    public function getName(): string {
        return $this->name;
    }
    public function getAge(): int {
        return $this->age;
    }
}

// Trivial test
class SomePersonTest {
    public function testCreating() {
        $person = new SomePerson('Horst', 1, 1);
        $this->assert('Horst', $person->getName());
        $this->assert(2, $person->getAge()); // should I remove this assert?
    }
}

现在的问题是 - 在为someperson编写测试时，我是否还要断言嵌套类somecalculator的计算结果？类somecalculator有自己的测试，因此我相信可以。 我觉得我们正在复制somecalculator在somecalculatortestest和someperStesteanteTEST中的断言。

谢谢。

Answer 1

好的，因此此代码的主要故障是很难测试。假设testCreating是单元测试，则不必检查与正在测试的服务不同的其他组件的行为。因此，让我们以例如测试更容易进行重构来看一下：

        $this->age = (new SomeCalculator())->add($a, $b);

上面的主要问题。由于在构造函数中使用了新的创建对象，因此两种分类都非常耦合。为了解决我们应该使用依赖性反转，我们也应该将唯一责任的类别分开，即包含数据（dto？），例如通过使用例如 factory模式。在这种情况下，重构示例应该看起来像这样：

class SomePerson {
    protected $name;
    protected $age;

    public function __construct(string $name, int $age) {
        $this->name = $name;
        $this->age = $age;
    }
    public function getName(): string {
        return $this->name;
    }
    public function getAge(): int {
        return $this->age;
    }
}

class PersonFactory {

    private SomeCalculator $calculator;

    public function __construct(SomeCalculator $calculator) {
        $this->calculator = $calculator;
    }

    public function create($name, $a, $b): SomePerson 
    {
        return new SomePerson($name, $this->calculator->add($a, $b));
    }
}

然后对其进行测试可能是这样的：

class PersonFactoryTest extends TestCase {

    public function testCreate(): string 
    {
        $a = 1;
        $b = 1;
        $age = 2;
        $name = 'Horst'
        $calculator = $this->createMock(SomeCalculator::class);
        $calculator->method('calculate')->with($a, $b)->willReturn($age);
        $factory = new PersonFactory($calculator);

        $person = $factory->create($name, $a, $b);

        $this->assertEqual($name, $person->getName());
        $this->assertEqual($age, $person->getAge());
    }
} 

使用模拟这里很重要，因为从PersonFactory计算器的观点行为行为't重要，因此在这种情况下，测试应只能检查是否提供了适当的参数来添加甲基，并且结果是否用于创建新对象。检查是否正确添加计算器应在其他测试中进行。

编辑：

由于模拟，它也是有效的测试（模拟的值只要被模拟就无关紧要，但是对于其他程序员来说，这将使更难理解）：

public function testCreate(): string 
{
    $a = 50;
    $b = -50;
    $age = 999;
    $name = 'Horst'
    $calculator = $this->createMock(SomeCalculator::class);
    $calculator->method('calculate')->with($a, $b)->willReturn($age);
    $factory = new PersonFactory($calculator);

    $person = $factory->create($name, $a, $b);

    $this->assertEqual($name, $person->getName());
    $this->assertEqual($age, $person->getAge());
}