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

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

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);
    }

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

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

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

public function getContent(string $query): ?string

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

не херня жеж!

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

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

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

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

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