Как правильно писать phpunit тесты?

Question

[kaxa3201]

Учусь писать phpunit тесты , хочу попробовать на имеющемся коде написать тест
мой код

<?php
class ContentService implements ContentServiceInterface
{
    private $client;

    /**
     * ContentServiceInterface constructor.
     * @param Client $client
     */
    public function __construct(Client $client)
    {
        $this->client = $client;
    }

    /**
     * @param $query
     * @return mixed
     */
    public function getContent($query): ?string
    {
        $queryEncode = urlencode($query);

        if ($result = $this->loadFromCache($queryEncode)) {
            return $result['content'];
        }

        return null;
    }

    public function sqrt($x)
    {
        return sqrt($x);
    }

}

вот мой тест пробный

<?php

namespace Tests\Feature;

use App\Services\ContentService;
use GuzzleHttp\Client;
use Tests\TestCase;

class ContentServiceTest extends TestCase
{
    public $client;

    public function __construct()
    {
        parent:: __construct();


    }


    public function testsqrt($client): void
    {
        $o = new ContentService($client);
        $this->assertEquals(4, $o->sqrt(16));
    }
}

Получаю ошибку

ArgumentCountError: Too few arguments to function Tests\Feature\ContentServiceTest::testsqrt(), 0 passed

Comments

[Артём]

Попробуйте использовать depends: https://phpunit.de/manual/6.5/en/writing-tests-for...

Answer 1

[Алексей]

Корректный тест для данного кода:

<?php

namespace Tests\Feature;

use App\Services\ContentService;
use GuzzleHttp\Client;
use Tests\TestCase;

class ContentServiceTest extends TestCase
{
    public function testSqrt(): void
    {
        $client = new Client();
        $o = new ContentService($client);
        $this->assertEquals(4, $o->sqrt(16));
    }
}

Также желательно указывать аннотации в тестах, для точного сопоставления того что тестируется. Если не указать аннотации, то при построении отчета по code coverage будут учтены не только тестируемые методы, а вообще все вызовы, что использовались в рамках тестирования

<?php

namespace Tests\Feature;

use App\Services\ContentService;
use GuzzleHttp\Client;
use Tests\TestCase;

/**
 * @coversDefaultClass \App\Services\ContentService
 */
class ContentServiceTest extends TestCase
{
    /**
     * @covers ::sqrt
     */
    public function testSqrt(): void
    {
        $client = new Client();
        $o = new ContentService($client);
        $this->assertEquals(4, $o->sqrt(16));
    }
}

Comments

[kaxa3201]

как правильно подключить мой конструктор?

[Алексей]

Конструктор используется при создании класса new ContentService($client), его не нужно подключать

[kaxa3201]

Алексей, мне в дальнейшем пригодиться, чтобы протестировать все методы в этом классе, это не единственный метод у меня

Answer 2

[GTRxShock]

ну как бы

Too few arguments to function Tests\Feature\ContentServiceTest::testsqrt(), 0 passed

и этим все сказано)

кто за вас инжектить будет клиента в тест?
public function testsqrt($client): void

Comments

[kaxa3201]

Простите за глупость, но и так не работает

class ContentServiceTest extends TestCase
{
    public $client;

    public function __construct(Client $client)
    {
        parent:: __construct();
        $this->client = $client;

    }


    public function testsqrt($client): void
    {
        $o = new ContentService($client);
        $this->assertEquals(4, $o->sqrt(16));
    }
}

[nyatmeat15]

Следует использовать функцию setUp для инициализации каких то сервисов и классов, чтобы в каждом сервисе не делать инициализацию

Answer 3

[Павел Терещенко]

<?php

namespace Tests\Feature;

use App\Services\ContentService;
use GuzzleHttp\Client;
use Tests\TestCase;

class ContentServiceTest extends TestCase
{
    public function testsqrt(): void
    {
        $config = [];
        $client = new GuzzleHttp\Client($config);
        $o = new ContentService($client);
        $this->assertEquals(4, $o->sqrt(16));
    }
}

Если $client будет использоваться ещё где то в тестовом классе, вынести его инициализацию.