php - 如果测试访问器是浪费时间，那么测试构造函数逻辑呢？

Question

似乎很多人认为对 setter 和 getter 进行单元测试只是浪费时间，只有当访问器包含某种逻辑时才应该对其进行测试。我同意。

但是那些需要一些(少量)逻辑的属性呢？

class DeliveryReportEvent extends Event
{
    private static $reasonMap = array(
        '401' => "Message expired (device off/not reachable)",
        '201' => "Operator network malfunctioning",
        '203' => "Recipient unreachable (in roaming)",
        '301' => "Invalid recipient (nonexistent/on portability/not enabled)",
        '302' => "Wrong number",
        '303' => "SMS service not enabled",
    );

    private $errorCode;

    public function __construct($errorCode)
    {
        $this->errorCode = $errorCode;

        if(array_key_exists($errorCode, self::$reasonMap)) {
            $this->errorReason = self::$reasonMap;
        }
    }

    public function getErrorCode()
    {
        return $this->errorCode;
    }

    public function getErrorReason()
    {
        return $this->errorReason;
    }
}

虽然测试 getErrorCode() 听起来很愚蠢(因为缺少逻辑和 IDE 功能)，但您认为测试 getErrorReason() 有意义吗？

/**
 * @dataProvider getKnownErrorCodesAndReasons
 */
public function testErrorReasonWithKnownErrorCodes($knownErrorCode,
    $expectedErrorReason)
{
    $event = $this->getMockDeliveryReportEvent($knownErrorCode);

    $actualErrorReason = $event->getErrorReason();

    $this->assertNotNull($errorReason);
    $this->assertContains($expectedErrorReason, $actualErrorReason, '', true);
}

public function getKnownErrorCodesAndReasons()
{
    return array(
        array('401', "expired"),
        array('201', "network malfunctioning"),
        array('203', "unreachable"),
        array('301', "invalid recipient"),
        array('302', "wrong number"),
        array('303', "not enabled"),
    );
}

Answer 1

这都是主观的，取决于许多因素，例如:

• 您希望您的测试用例具有什么级别的代码覆盖率？一些发布管理系统定义了此覆盖范围，并需要在发布确认之前将其与其他标准一起满足。
• getErrorReason() 是一个关键函数吗(尽管它的内部逻辑有多小)？换句话说，如果它搞砸了，它会破坏系统吗？

还取决于其他因素，例如:

• 你有多少空闲时间？
• 你有多纯粹？
• 你有 mustache 吗？

等等等等。:)