0

我有一个通过命名构造函数创建文件的类，但是当我使用 phpspec 对其进行测试时，它不会创建文件。

我找不到原因，所以我想重新审视我的代码可能会有所帮助。

这是我的文件类：

<?php

namespace Acme;

class File
{
    /**
     * @var Path
     */
    private $path;

    /**
     * @var FileName
     */
    private $fileName;

    private function __construct(Path $path, FileName $fileName)
    {
        $this->path = $path;
        $this->fileName = $fileName;
    }

    public static function create(Path $path, FileName $fileName)
    {
        if (file_exists((string) $path . (string) $fileName)) {
            throw new \DomainException('File already exists');
        }

        if (!touch((string) $path . (string) $fileName)) {
            throw new \DomainException('Cannot create file');
        }

        return new self($path, $fileName);
    }
}

这是我的规格：

<?php

namespace spec\Acme;

use PhpSpec\ObjectBehavior;
use Prophecy\Argument;
use Acme\Path;
use Acme\FileName;

class FileSpec extends ObjectBehavior
{

    private $testPath;
    private $existingFileName = 'existingFile.extension';
    private $nonExistingFileName = 'nonExistingFile.extension';
    private $existingFilePath;
    private $nonExistingFilePath;

    function let()
    {
        $this->testPath = sys_get_temp_dir() . '/';
        $this->existingFilePath = $this->testPath . $this->existingFileName;
        $this->nonExistingFilePath = $this->testPath . $this->nonExistingFileName;

        // Creating existing file
        if (!touch($this->existingFilePath)) {
            throw new \Exception('Cannot create existing file for testing');
        }

        // Removes non existing file
        if (file_exists($this->nonExistingFilePath)) {
            if (!unlink($this->nonExistingFilePath)) {
                throw new \Exception('Cannot remove non existing file for testing');
            }
        }
    }

    function it_does_not_create_a_file_when_the_file_already_exists(Path $path, FileName $fileName)
    {
        $path->__toString()->willReturn($this->testPath);
        $fileName->__toString()->willReturn($this->existingFileName);
        $this->beConstructedThrough('create', [$path, $fileName]);
        $this->shouldThrow(new \DomainException('File already exists'))->duringInstantiation();
    }

    function it_creates_a_new_file_if_file_does_not_exist(Path $path, FileName $fileName)
    {
        $path->__toString()->willReturn($this->testPath);
        $fileName->__toString()->willReturn($this->nonExistingFileName);
        $this->beConstructedThrough('create', [$path, $fileName]);
        assert(file_exists($this->nonExistingFilePath));
    }
}
4

1 回答 1

2

这是因为 phpspec 在需要之前不会实例化该类。只有针对原始类本身的方法调用或期望（即should*）会导致它被实例化，并且beConstructedThrough只是关于它 phpspec 应该如何获得实例的提示。

现在，您可以通过调用某个方法来解决此问题，或者甚至只是调用$this->shouldHaveType(File::class)，但我建议重新考虑该方法。如果您最终要集成到外部的东西中——无论是 SDK、文件系统、数据库等，那么编写集成测试会好得多。无论如何，在这种情况下你已经非常接近了（实际上不需要模拟）。phpspec 更旨在指定类和方法的行为/逻辑.. 描述副作用并不真正适合它的职权范围。这里的使用assert()也暗示了这一点，因为这对于 phpspec 支持的规范肯定不是惯用的。

对于集成测试，PHPUnit 将是一个更好的选择，因为它更通用。所以您可以根据需要灵活地实例化和断言。

于 2016-04-07T21:44:21.177 回答