[PHP]Testy w Laravel / php Opcje

[trifek]

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 (IMG:style_emoticons/default/smile.gif)

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?(IMG:style_emoticons/default/smile.gif)

[trifek]

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 (IMG:style_emoticons/default/smile.gif)

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

A jak taki test zapisać? (IMG:style_emoticons/default/smile.gif)

Takie testy są ok. Nazywają się testami Integracyjnymi lub akceptacyjnymi ( w zależności kto patrzy (IMG:style_emoticons/default/smile.gif) )
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.

W firmach, które testy najczęściej się pisze? Pewnie te integracyjne? Unit testy idą do klas "nie laravelowych?" (IMG:style_emoticons/default/smile.gif)