Testy w Laravel / php

Testy w Laravel / php
BA
  • Rejestracja:ponad 6 lat
  • Ostatnio:5 miesięcy
  • Postów:172
0

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
Kopiuj

class HomeTest extends TestCase
{
    public function testHomePageWorkCorrectly()
    {
        //$this->withoutExceptionHandling();
        $response = $this->get('/');

        $response->assertStatus(200);
    }
}

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

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

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

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

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

edytowany 1x, ostatnio: baracadus
MJ
  • Rejestracja:około 13 lat
  • Ostatnio:prawie 3 lata
  • Postów:34
1

w test 1 warto by sprawdzić coś jeszcze poza tym czy response jest ze statusem 200 np jeśli zwracasz coś na stronę to czy te dane się zgadzają.
w test 2 jakieś setup danych do bazy z poziomu testu aby później nie było problemów że testy nie przechodzą bo coś w bazie się zmieniło , chyba że robisz to z poziomu fixtur globalnie do wszystkich testów bo tak też można
w test 3 pierwsze co rzuciło mi się w oczy to może nie związane z samym testem ale do edycji encji używasz metody post a powinien to być put albo patch w zależności od tego czy wysyłasz całą encje czy tylko jej część tak samo do usuwania metoda delete a nie post. 2 pierwsze testy z klasy AdTest są identyczne i robią coś dla mnie nie zrozumiałego mianowicie nazwa pierwszego testu wskazuje na to że sprawdzasz czy reklama się utworzy a wysyłasz na route ads.edit z parametrem id jakieś array poczym sprawdzasz czy ten id jest w bazie bardziej wygląda mi to na edycje , przy edycji nie ma sensu sprawdzania w bazie czy dany id istnieje bo jeżeli by to tam nie było response powinno zwrócić status 404. Ogólnie to ciężko jest mi się konstruktywnie wypowiedzieć na temat testów z 3 klasy ponieważ nie rozumiem logiki którą one testują. Jeszcze jedna kwestia w mojej opinii lepiej mieć w teście statyczne dane niż generować je dynamicznie (faker), ułatwia to debugowanie

serek
  • Rejestracja:około 11 lat
  • Ostatnio:33 minuty
  • Postów:1472
0
mjhallll napisał(a):

w test 1 warto by sprawdzić coś jeszcze poza tym czy response jest ze statusem 200 np jeśli zwracasz coś na stronę to czy te dane się zgadzają.

Test 1 wygląda na zwykły smoke test, czyli wygląda ok. Smoke test ma być szybki do wykonania i ma sprawdzać tylko, czy strona działa (czy zwraca odpowiedni kod statusu). Sprawdzanie, czy dane się zgadzają, itp., to już trochę inne testy.

MJ
Masz racje, jednak widziałem wiele "rozwiązań" w których response ze statusem 200 nie znaczył że strona działa.
serek
Jasne. Najszybciej po prostu sprawdzać, czy strona zwraca inny status niż "5xx". Wtedy wiemy, że nic się nie sypie, ale przy smoke testach nigdy nie mamy pewności, że wszystko działa jak trzeba. Ja jak piszę smoke testy, to rozbijam strony na np. te co mają zwrócić 200 od razu, te co mają zwracać redirect a potem 200. Zawsze to coś, ale nie chroni przed błędnym działaniem nigdy. Tylko trochę ułatwia życie^^
BA
@serek możesz pokazać jakiś przykładowy smoke test?:)
serek
@baracadus: przecież masz, Twój test nr 1 to podręcznikowy smoke test xD Czyli testujesz tylko to, czy strona się nie wywala, a samo poprawne działanie Cię nie obchodzi.
Kliknij, aby dodać treść...

Pomoc 1.18.8

Typografia

Edytor obsługuje składnie Markdown, w której pojedynczy akcent *kursywa* oraz _kursywa_ to pochylenie. Z kolei podwójny akcent **pogrubienie** oraz __pogrubienie__ to pogrubienie. Dodanie znaczników ~~strike~~ to przekreślenie.

Możesz dodać formatowanie komendami , , oraz .

Ponieważ dekoracja podkreślenia jest przeznaczona na linki, markdown nie zawiera specjalnej składni dla podkreślenia. Dlatego by dodać podkreślenie, użyj <u>underline</u>.

Komendy formatujące reagują na skróty klawiszowe: Ctrl+B, Ctrl+I, Ctrl+U oraz Ctrl+S.

Linki

By dodać link w edytorze użyj komendy lub użyj składni [title](link). URL umieszczony w linku lub nawet URL umieszczony bezpośrednio w tekście będzie aktywny i klikalny.

Jeżeli chcesz, możesz samodzielnie dodać link: <a href="link">title</a>.

Wewnętrzne odnośniki

Możesz umieścić odnośnik do wewnętrznej podstrony, używając następującej składni: [[Delphi/Kompendium]] lub [[Delphi/Kompendium|kliknij, aby przejść do kompendium]]. Odnośniki mogą prowadzić do Forum 4programmers.net lub np. do Kompendium.

Wspomnienia użytkowników

By wspomnieć użytkownika forum, wpisz w formularzu znak @. Zobaczysz okienko samouzupełniające nazwy użytkowników. Samouzupełnienie dobierze odpowiedni format wspomnienia, zależnie od tego czy w nazwie użytkownika znajduje się spacja.

Znaczniki HTML

Dozwolone jest używanie niektórych znaczników HTML: <a>, <b>, <i>, <kbd>, <del>, <strong>, <dfn>, <pre>, <blockquote>, <hr/>, <sub>, <sup> oraz <img/>.

Skróty klawiszowe

Dodaj kombinację klawiszy komendą notacji klawiszy lub skrótem klawiszowym Alt+K.

Reprezentuj kombinacje klawiszowe używając taga <kbd>. Oddziel od siebie klawisze znakiem plus, np <kbd>Alt+Tab</kbd>.

Indeks górny oraz dolny

Przykład: wpisując H<sub>2</sub>O i m<sup>2</sup> otrzymasz: H2O i m2.

Składnia Tex

By precyzyjnie wyrazić działanie matematyczne, użyj składni Tex.

<tex>arcctg(x) = argtan(\frac{1}{x}) = arcsin(\frac{1}{\sqrt{1+x^2}})</tex>

Kod źródłowy

Krótkie fragmenty kodu

Wszelkie jednolinijkowe instrukcje języka programowania powinny być zawarte pomiędzy obróconymi apostrofami: `kod instrukcji` lub ``console.log(`string`);``.

Kod wielolinijkowy

Dodaj fragment kodu komendą . Fragmenty kodu zajmujące całą lub więcej linijek powinny być umieszczone w wielolinijkowym fragmencie kodu. Znaczniki ``` lub ~~~ umożliwiają kolorowanie różnych języków programowania. Możemy nadać nazwę języka programowania używając auto-uzupełnienia, kod został pokolorowany używając konkretnych ustawień kolorowania składni:

```javascript
document.write('Hello World');
```

Możesz zaznaczyć również już wklejony kod w edytorze, i użyć komendy  by zamienić go w kod. Użyj kombinacji Ctrl+`, by dodać fragment kodu bez oznaczników języka.

Tabelki

Dodaj przykładową tabelkę używając komendy . Przykładowa tabelka składa się z dwóch kolumn, nagłówka i jednego wiersza.

Wygeneruj tabelkę na podstawie szablonu. Oddziel komórki separatorem ; lub |, a następnie zaznacz szablonu.

nazwisko;dziedzina;odkrycie
Pitagoras;mathematics;Pythagorean Theorem
Albert Einstein;physics;General Relativity
Marie Curie, Pierre Curie;chemistry;Radium, Polonium

Użyj komendy by zamienić zaznaczony szablon na tabelkę Markdown.

Lista uporządkowana i nieuporządkowana

Możliwe jest tworzenie listy numerowanych oraz wypunktowanych. Wystarczy, że pierwszym znakiem linii będzie * lub - dla listy nieuporządkowanej oraz 1. dla listy uporządkowanej.

Użyj komendy by dodać listę uporządkowaną.

1. Lista numerowana
2. Lista numerowana

Użyj komendy by dodać listę nieuporządkowaną.

* Lista wypunktowana
* Lista wypunktowana
** Lista wypunktowana (drugi poziom)

Składnia Markdown

Edytor obsługuje składnię Markdown, która składa się ze znaków specjalnych. Dostępne komendy, jak formatowanie , dodanie tabelki lub fragmentu kodu są w pewnym sensie świadome otaczającej jej składni, i postarają się unikać uszkodzenia jej.

Dla przykładu, używając tylko dostępnych komend, nie możemy dodać formatowania pogrubienia do kodu wielolinijkowego, albo dodać listy do tabelki - mogłoby to doprowadzić do uszkodzenia składni.

W pewnych odosobnionych przypadkach brak nowej linii przed elementami markdown również mógłby uszkodzić składnie, dlatego edytor dodaje brakujące nowe linie. Dla przykładu, dodanie formatowania pochylenia zaraz po tabelce, mogłoby zostać błędne zinterpretowane, więc edytor doda oddzielającą nową linię pomiędzy tabelką, a pochyleniem.

Skróty klawiszowe

Skróty formatujące, kiedy w edytorze znajduje się pojedynczy kursor, wstawiają sformatowany tekst przykładowy. Jeśli w edytorze znajduje się zaznaczenie (słowo, linijka, paragraf), wtedy zaznaczenie zostaje sformatowane.

  • Ctrl+B - dodaj pogrubienie lub pogrub zaznaczenie
  • Ctrl+I - dodaj pochylenie lub pochyl zaznaczenie
  • Ctrl+U - dodaj podkreślenie lub podkreśl zaznaczenie
  • Ctrl+S - dodaj przekreślenie lub przekreśl zaznaczenie

Notacja Klawiszy

  • Alt+K - dodaj notację klawiszy

Fragment kodu bez oznacznika

  • Alt+C - dodaj pusty fragment kodu

Skróty operujące na kodzie i linijkach:

  • Alt+L - zaznaczenie całej linii
  • Alt+, Alt+ - przeniesienie linijki w której znajduje się kursor w górę/dół.
  • Tab/⌘+] - dodaj wcięcie (wcięcie w prawo)
  • Shit+Tab/⌘+[ - usunięcie wcięcia (wycięcie w lewo)

Dodawanie postów:

  • Ctrl+Enter - dodaj post
  • ⌘+Enter - dodaj post (MacOS)