[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)

[Rysh]

Z tego co zaobserwowałem, to w firmach rzadko kiedy piszę się testy (IMG:style_emoticons/default/wink.gif)

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:
(IMG:https://snipboard.io/PifhS9.jpg)

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.

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