Правильно ли я пишу phpunit тесты?

Question

[ganjo888]

мой метод

public function getContent($query): ?string
    {
        if (isset($query) && !filter_var($query, FILTER_VALIDATE_URL)) {
            echo 'Некорректный Url';
        }
        $queryEncode = urlencode($query);
        if ($result = $this->loadFromCache($queryEncode)) {
            return $result['content'];
        }
        return null;
    }

Вот мой тест, буду благодарен за поправки и оценку моего теста, правильный ли он? Прошу поправить , если это не так

public function testGetContent()
    {
        $url = 'https://phpunit.de';

        $client = $this->createMock(Client::class);

        $contentService = new ContentService($client);
        $result = $contentService->getContent($url);

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

Answer 1

[index0h]

Вообще говоря что бы покрыть этот метод нужно как минимум 3 теста: когда filter_var не отработал, result приводится к true и result приводится к false. Еще не хватает мока метода loadFromCache, я не вижу, что бы вы покрыли его лог ку работы.

Проверка на isset не имеет смысла. Я бы еще добавил type hint на query. Что будет, если туда closure загнать?

Так же если чтото идет не так - бросайте исключения, не надо вот этой мерзостью echo обмазываться.

UPD

ganjo888 Вот вам полезного чтива: https://github.com/index0h/php-conventions#7-тести...

Comments

[ganjo888]

Что я делаю не правильно? Хочу теперь по вашему совету делать, хочу проверить на валидность

public function getContent($query): ?string
    {
        try {
            filter_var($query, FILTER_VALIDATE_URL);

        } catch (\Exception $e) {
            return false;
        }
        $queryEncode = urlencode($query);
        if ($result = $this->loadFromCache($queryEncode)) {
            return $result['content'];
        }
        return null;
    }

public function testGetContentValidUrl()
    {
        $url = 'https://phpunijst.de';

        $client = $this->createMock(Client::class);

        $contentService = new ContentService($client);
        $result = $contentService->getContent($url);

        $this->assertTrue($contentService->getContent($url) !== false);
    }

[index0h]

public function getContent($query): ?string
...
            return false;

Это не валидный код. Если filter_var($query, FILTER_VALIDATE_URL) не отработал корректно - бросайте исключение, а не подавляйте его

[index0h]

Что вам мешает задать хинт на $query?

public function getContent(string $query): ?string

[index0h]

$this->assertTrue($contentService->getContent($url) !== false);

не херня жеж!

$actual = $contentService->getContent($url);

$this->assertSame($expected,  $actual);

[ganjo888]

index0h, прошу прощения , но мне не понятно как написать условие в try catch, чтобы ваш код стработал

[index0h]

ganjo888 А в том то и дело, что try-catch тут не нужен))
Если ваш метод вызывается так, что штатное выполнение невозможно - бросайте исключение. Код, который вызывает ваш метод уже должен как-то это обрабатывать.
Вот по вашему же коду: если я задаю НЕ url - вы можете получить данные по url? Нет, ваш метод не может работать со строками, которые не являются url. Именно по этой причине стоит бросить исключение, штатная работа метода с не корректными аргументами невозможно. Код выше, что вызывает этот метод должен либо не допускать таких ситуаций (что лучше), либо обрабатывать подобные ситуации с try-catch.

[ganjo888]

Спасибо я вас понял,вы лучший