Forum PHP.pl > [PHP]Testy w Laravel / php

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [PHP]Testy w Laravel / php

trifek

5.04.2020, 15:26:33

Witam serdecznie,
Uczę się pisania testów w Laravelu. Jako że nie wiem jak powinny wyglądać "profesjonalnie" - to chciałem dopytać czy to, co robię ma sens

1. Test 1 - test sprawdzający czy strona działa poprawnie

[PHP] pobierz, plaintext 
class HomeTest extends TestCase
{
    public function testHomePageWorkCorrectly()
    {
        //$this->withoutExceptionHandling();
        $response = $this->get('/');
 
        $response->assertStatus(200);
    }
}
[PHP] pobierz, plaintext

2. Test 2 - test logowania z dobrym i błędnym hasłem

[PHP] pobierz, plaintext 
class LoginTest extends TestCase
{
    public function testUserCanLoginWithCorrectPassword()
    {
        //$this->withoutExceptionHandling();
        $response = $this->post(route('cms.check_admin_login'), [
            'email' => 'lukpeta@icloud.com',
            'password' => 'passw'
        ]);
        $response->assertRedirect('/psCMS/');
        $this->assertTrue(Auth::check());
    }
 
    public function testUserCannotLoginWithInCorrectPassword()
    {
        //$this->withoutExceptionHandling();
        $response = $this->post(route('cms.check_admin_login'), [
            'email' => 'lukpeta@icloud.com',
            'password' => 'xxxxx'
        ]);
        $response->assertRedirect('psCMS/login?error=1');
        $this->assertFalse(Auth::check());
    }
}
[PHP] pobierz, plaintext

3. Test 3 - test modułu reklam. Dodawanie nowej reklamy, sprawdzanie czy istnieje, pobieranie listy reklam, sprawdzanie czy walidacja pól działa, kasowanie

[PHP] pobierz, plaintext 
class AdTest extends TestCase
{
    use \Illuminate\Foundation\Testing\WithFaker;
 
    public function testCreateAd()
    {
        $user = User::find(1);
        $this->be($user);
 
        $ad = factory(Ad::class)->create();
        $this->post(route('ads.edit') . '/' . $ad->id, $ad->toArray());
        $this->assertDatabaseHas('ads', ['id' => $ad->id]);
    }
 
    public function testAdExist()
    {
        $user = User::find(1);
        $this->be($user);
 
        $ad = factory(Ad::class)->create();
        $this->post(route('ads.edit') . '/' . $ad->id, $ad->toArray());
        $this->assertDatabaseHas('ads', ['id' => $ad->id]);
    }
 
 
    public function testGetAdsList()
    {
        $user = User::find(1);
        $this->be($user);
 
        $ads = factory(Ad::class)->create();
        $response = $this->get(route('ads.index'));
        $response->assertSee($ads->title);
    }
 
    public function testCreateAdMustBeCompleted()
    {
        $user = User::find(1);
        $this->be($user);
 
        $response = $this->actingAs($user)->post(route('ads.store'), [
            'title' => null,
            'provincial_id' => null,
            'content' => null,
        ]);
 
        $response->assertSessionHasErrors(['title', 'provincial_id', 'content']);
    }
 
    public function testCreateAdFromForm()
    {
        $user = User::find(1);
        $this->be($user);
        $faker = Faker\Factory::create('pl_PL');
        $title = $faker->sentence($nbWords = 6, $variableNbWords = true);
        $response = $this->actingAs($user)->post(route('ads.store'), [
            'title' => $title,
            'content' => $faker->text($maxNbChars = 200),
            'provincial_id' => $faker->numberBetween(1, 8),
        ]);
 
        $this->assertDatabaseHas('ads', ['title' => $title]);
    }
 
 
    public function testEditAdFromForm()
    {
        $this->withExceptionHandling();
        $user = User::find(1);
        $this->be($user);
        $faker = Faker\Factory::create('pl_PL');
 
        $ad = factory(Ad::class)->create();
        $id = $ad->id;
 
        $title = $faker->sentence($nbWords = 6, $variableNbWords = true);
        $response = $this->actingAs($user)->post(route('ads.update') . '/' . $id, [
            'title' => $title,
            'content' => $faker->text($maxNbChars = 200),
            'provincial_id' => $faker->numberBetween(1, 8),
            'id' => $id
        ]);
 
        //dd($response->getContent());
 
        $this->assertDatabaseHas('ads', ['title' => $title, 'id' => $id]);
    }
 
 
    public function testDeleteAd()
    {
        //$this->withExceptionHandling();
        //$this->withoutExceptionHandling();
        $user = User::find(1);
        $this->be($user);
 
        $ad = factory(Ad::class)->create();
        $id = $ad->id;
 
        $faker = Faker\Factory::create('pl_PL');
        $title = $faker->sentence($nbWords = 6, $variableNbWords = true);
        $response = $this->actingAs($user)->post(route('ads.destroy'), [
            'title' => $title,
            'content' => $faker->text($maxNbChars = 200),
            'provincial_id' => $faker->numberBetween(1, 8),
            //'id'=> $id,
            'id' => [$id]
        ]);
 
        $this->assertDatabaseMissing('ads', ['id' => $ad->id]);
    }
 
 
}
[PHP] pobierz, plaintext

Czy takie testy są poprawne? Czy raczej powinny wyglądać inaczej?

Pyton_000

5.04.2020, 18:44:31

Takie testy są ok. Nazywają się testami Integracyjnymi lub akceptacyjnymi ( w zależności kto patrzy

)
Generalnie tutaj testujesz czy funkcjonalność zadana w ogóle działa czyli zapisujesz i sprawdzasz czy się zrobił.

Do tego możesz dołożyć testy Unit które będą testować wybraną klasę z logiką czyli np. masz klasę która generuje Ci coś a twoim zadaniem jest sprawdzić czy klasa będzie się zachowywała w najróżniejszych przypadkach czyli jak dasz dobre wartości to dobry wynik, jak dasz złe wartości to błąd, jak nie dasz wartości to błąd itd.

Rysh

5.04.2020, 20:04:57

Tak jak napisał kolega powyżej, testy są OK - ale można je napisać o wiele lepiej.

[PHP] pobierz, plaintext 
$this->assertAuthenticated()
[PHP] pobierz, plaintext

zamiast:

[PHP] pobierz, plaintext 
$this->assertTrue(Auth::check());
[PHP] pobierz, plaintext

Tutaj widać, że budowanie URL leży:

[PHP] pobierz, plaintext 
$this->post(route('ads.edit') . '/' . $ad->id, $ad->toArray());
[PHP] pobierz, plaintext

Fakera nie musisz tworzyć za każdym razem, masz przecież trait'a którego możeszużyć:

[PHP] pobierz, plaintext 
<?php
 
namespace Tests\Feature;
 
use Illuminate\Foundation\Testing\WithFaker;
use Tests\TestCase;
 
/**
 * Class WebControllerTest
 */
class WebControllerTest extends TestCase
{
    use WithFaker;
 
    public function testFaker(): void
    {
         $this->faker->email;
    }
}
[PHP] pobierz, plaintext

trifek

6.04.2020, 19:31:47

Cytat(Rysh @ 5.04.2020, 21:04:57 )

Tak jak napisał kolega powyżej, testy są OK - ale można je napisać o wiele lepiej.

[PHP] pobierz, plaintext 
$this->assertAuthenticated()
[PHP] pobierz, plaintext

zamiast:

[PHP] pobierz, plaintext 
$this->assertTrue(Auth::check());
[PHP] pobierz, plaintext

Tutaj widać, że budowanie URL leży:

[PHP] pobierz, plaintext 
$this->post(route('ads.edit') . '/' . $ad->id, $ad->toArray());
[PHP] pobierz, plaintext

Fakera nie musisz tworzyć za każdym razem, masz przecież trait'a którego możeszużyć:

[PHP] pobierz, plaintext 
<?php
 
namespace Tests\Feature;
 
use Illuminate\Foundation\Testing\WithFaker;
use Tests\TestCase;
 
/**
 * Class WebControllerTest
 */
class WebControllerTest extends TestCase
{
    use WithFaker;
 
    public function testFaker(): void
    {
         $this->faker->email;
    }
}
[PHP] pobierz, plaintext

Dziękuję bardzo za uwagi / opinie

[PHP] pobierz, plaintext 
$this->assertFalse(Auth::check());
[PHP] pobierz, plaintext

A jak taki test zapisać?

Cytat(Pyton_000 @ 5.04.2020, 19:44:31 )

Takie testy są ok. Nazywają się testami Integracyjnymi lub akceptacyjnymi ( w zależności kto patrzy

W firmach, które testy najczęściej się pisze? Pewnie te integracyjne? Unit testy idą do klas "nie laravelowych?"

Rysh

7.04.2020, 18:41:10

Z tego co zaobserwowałem, to w firmach rzadko kiedy piszę się testy

Unit: testy które testują daną klasę - tylko klasę, dajesz input do metody i sprawdzasz czy wynik jest spełnia Twoje oczekiwania, nie interesuje Cie za bardzo reszta.
Feature: to testy które testują żądania do strony i oczekiwany response, czyli wysyłasz formularz, odbierasz błąd walidacji lub zwrócone dane. Następnie sprawdzasz czy zwrócone dane są zgodne z oczekiwaniami, sprawdzasz również status HTTP (w przypadku REST API - bardzo ważne) i inne rzeczy (z czasem sam ogarniesz, co testować a czego nie).

Co jest bardzo przydatne moim zdaniem, to code coverage - puszczasz testy na całość, i pokazuje Ci miejsca których nie masz przetestowanych. Czyli żaden kod podczas wykonywania testów nie wszedł do danej instrukcji - czyli przewidziałeś pewną sytuację, ale nie przetestowałeś jej. Tak to wygląda w IDE:

Linijka 45 nie została przetestowana, 44 i 48 już tak.

Warto sobie po testować całe klasy (unit) wraz z kontrolerami (feature). Niektórych czasem nie ma sensu, ale to od programisty zależy. I oczywiście nie testujesz obcych paczek, tylko swój kod który napisałeś.

Pisanie testów, owocuje w przyszłości, gdy zmienisz drobną rzecz, może się okazać że resposne się zmienił - testami to wychwycisz. Bez wychwycenia zmiany response, front może się posypać - czego nikt by nie chciał.
No i zmiany commitujesz dopiero jak wszystkie testy są wykonane z sukcesem. Tutaj warto sobie dodać również automatyczne wykonywanie testów po przesłaniu zmian na GIT, więc nawet jak zapomnisz puścić testów lokalnie, to gitlab czy inny, poinformuje Cię o tym.

Cytat

A jak taki test zapisać? smile.gif

Ja bym zapisał to tak, ale co programista to inny sposób - ja staram się używać, tego co mi podpowiada IDE.

[PHP] pobierz, plaintext 
$this->assertFalse($this->isAuthenticated());
[PHP] pobierz, plaintext

markonix

8.04.2020, 17:42:45

Tutaj parę ściąg, aby rozróżnić integracyjne od jednostkowych:
https://www.google.com/search?q=2+unit+test...off&tbm=vid

Osobiście, ze względu na braki kadrowo/czasowe pisze głównie integracyjne, są one bardziej praktyczne, weryfikują działanie podobnie do testera manualnego, więc po prostu robią to co początkujący programista robi sam po jakimś update - przeklikuje system.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.