Bonjour à tous!

J'ai un controller HomeController qui affiche ma page d'accueil et une liste de tricks (figures de snow).

J'aimerais faire des tests unitaires dessus pour vérifier qu'il revoit bien un objet Reponse. Voici donc le code que j'ai fait:

<?php

namespace App\Controller;

use App\Repository\TrickRepository;
use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
use Symfony\Component\Routing\Annotation\Route;

class HomeController extends AbstractController
{
    /**
     * @Route("/", name="home")
     */
    public function index(TrickRepository $repo)
    {
        $tricks = $repo->findAll();

        return $this->render('home.html.twig', [
            'tricks' => $tricks,
        ]);
    }
}

<?php

namespace App\Tests\Controller;

use App\Controller\HomeController;
use App\Repository\TrickRepository;
use Symfony\Bundle\FrameworkBundle\Test\WebTestCase;
use Symfony\Component\HttpFoundation\Response;

class HomeControllerTest extends WebTestCase
{
    public function testIndex()
    {
        $repo = $this->createMock(TrickRepository::class);
        $homepage = new HomeController();

        $returnTest = $homepage->index($repo);
        $this->assertInstanceOf(Response::class, $returnTest);

    }
}

Si je lance PhpUnit j'ai cette erreur :

There was 1 error:

1) App\Tests\Controller\HomeControllerTest::testIndex
Error: Call to a member function has() on null

C:\Users\Shiyo\Desktop\Formation\Projet\Projet6_SnowTricks\Code\SnowTricks\vendor\symfony\framework-bundle\Controller\AbstractController.php:245
C:\Users\Shiyo\Desktop\Formation\Projet\Projet6_SnowTricks\Code\SnowTricks\vendor\symfony\framework-bundle\Controller\AbstractController.php:257
C:\Users\Shiyo\Desktop\Formation\Projet\Projet6_SnowTricks\Code\SnowTricks\src\Controller\HomeController.php:19
C:\Users\Shiyo\Desktop\Formation\Projet\Projet6_SnowTricks\Code\SnowTricks\tests\Controller\HomeControllerTest.php:18

C'est le AbstractController avec sa fonction has() qui merde car il détecte que la liste des tricks est vide (forcément c'est un mock). Le seul moyen que j'ai trouvé pour enlever cette erreur est de supprimer le

extends AbstractController

en modifiant toute la classe en conséquence pour qu'elle fonctionne quand même correctement. Je trouve nul de modifier tous mes controllers en les rendant moins lisibles juste pour des test unitaires. Il doit y avoir un autre moyen mais je n'arrive pas à trouver lequel.

Des idées?

Salut

Pourquoi tu passe un new HomeController() ?

Salut hous,

Il faut que je crée la classe pour lancer sa fonction index() et tester ce qu'elle retourne, non?

$this->container->has

Et comme t'as directement créer un HomeController tu dois lui donner un ContainerInterface:

$homepgae->setContainer($container); // un object Container que tu dois récupérer...

Mais pourquoi ne pas faire comme dans la doc :

// tests/Controller/PostControllerTest.php
namespace App\Tests\Controller;

use Symfony\Bundle\FrameworkBundle\Test\WebTestCase;

class PostControllerTest extends WebTestCase
{
    public function testShowPost()
    {
        $client = static::createClient();

        // Tu passe juste l'url à tester 
        $client->request('GET', '/post/hello-world');

        $this->assertEquals(200, $client->getResponse()->getStatusCode());
    }
}

Merci pour ta réponse rand0mdev (pas mal le pseudo d'ailleurs ^^).

J'avais fait comme dans la doc mais mon mentor me dit que c'est des tests fonctionnels et non unitaires..

Oui c'est des tests fonctionnels et c'est ce que tu veux faire non? Car tu fais deja interagir pas mal de classes.

D'ailleurs pour de simples tests unitaires tu peux faire hériter ta classe de PHPUnit\Framework\TestCase ca suffit.

tu peux comparer les deux en regardant le code généré par le maker bundle:

php bin/console make:unit-test
php bin/console make:functional-test

Merci je comprend un peu mieux la différence entre les deux.

Il faut pour le projet que je face des tests unitaires uniquement. Je dois simplement vérifier que mon controller renvois un objet Response. Or je ne peut pas le faire à cause du Abstract Controller qui bloque avec sa fonction has().

Normalement ça peut marcher en faisant cela:

    public function testIndex()
    {
        static::bootKernel();

        $repo = $this->createMock(TrickRepository::class);
        $homepage = new HomeController();

        $homepage->setContainer(static::$container);
 
        $returnTest = $homepage->index($repo);
        $this->assertInstanceOf(Response::class, $returnTest);
 
    }

Mais est-ce que ça reste des tests unitaires ? je ne pense pas.

Bien sur ta classe doit hériter de WebTestCase 

En effet comme ça ça fonctionne ! Merci beaucoup!

Je vais me pencher sur la différence entre test unitaire et fonctionnel car visiblement j'ai pas encore tout saisit. Merci pour les infos et le code!