[Kamishimi]

Czy w testach powinienem tworzyć zmienne?
Mam taki test:

 @test
    void whenUserExist_DeleteUserSessions() {
        delete.deleteUserById(1);
        assertThat(sessionChecker.hasActiveSession(1)).isFalse();
    }

Doszedłem do wniosku, że do końca może nie być wiadome, co oznacza liczba „1” w tej asercji, więc postanowiłem ją zmienić na:

  @test
    void whenUserExist_DeleteUserSessions() {
        int userId = 1;
        delete.deleteUserById(1);
        assertThat(sessionChecker.hasActiveSession(userId)).isFalse();
    }

Co myślicie? Jaka jest Wasza opinia nt. zmiennych w testach?
#naukaprogramowania #java

[paczelok]

@Kamishimi: pomylił ci się copilot z wykopem kolego zamykam temat następnym razem użyj opcji szukaj

[Kamishimi]

@paczelok: Jeden rabin powie tak inny rabin powie nie

[JoeGlodomor]

@Kamishimi: wydaje mi się, że to nie z tą zmienną jest problem

[Kamishimi]

@JoeGlodomor: Co masz na myśli?

[JoeGlodomor]

@Kamishimi: nie no, może się nie znam, może w świecie javy panują inne obyczaje. Ale moim zdaniem nie powinna się w teście pojawiać jakaś jedynka z dupy.

[PolishPsycho]

@Kamishimi: https://www.oracle.com/java/technologies/javase/codeconventions-namingconventions.html

To po pierwsze.

A po drugie nawet na sonarze masz regułę, że metoda musi się zaczynać od "test".
Po 3 jak używasz zmiennej to wrzucasz jej wartość i do argumentów metody usuwania sesji i jej sprawdzania.

[Kamishimi]

@JoeGlodomor: mój błąd nie zmieniłem "1" na zmienną "userId". Miało być tak:

 @test
    void whenUserExist_DeleteUserSessions() {
        int userId = 1;
        delete.deleteUserById(userId);
        assertThat(sessionChecker.hasActiveSession(userId)).isFalse();
    }

[Kamishimi]

@PolishPsycho:

Po 3 jak używasz zmiennej to wrzucasz jej wartość i do argumentów metody usuwania sesji i jej sprawdzania.

Zapomniałem zmienić w metodzie usuwania, mój błąd.

[PolishPsycho]

@Kamishimi: Obczaj sobie i podepnij do projektu SonarLinta. Fajnie pokazuje konwencje nazewnicze w Javie, bo o ile może językowo jest poprawny Twój kod, to jednak oczy bolą xD

To jest zasada o której pisałem: https://rules.sonarsource.com/java/type/Code%20Smell/RSPEC-3578/

(Chociaż według Google Java Spec według nich underscore w nazwach metod testowych jest ok...)

A co do tego czy możesz używać zmiennej, to już naprawdę zależy. Jak metody nie są jasne to lepiej używać (albo jak

[Kamishimi]

@PolishPsycho: ok, dzięki za pomoc

[Kuriozal]

@Kamishimi: w tym wypadku jeden pies, trzeba miec IQ buta zeby nie skleic ze ta 1 jest userId jezeli metoda nazywa sie "deleteUserById" :) <- EDIT: spojrzalem ze uzywasz tej jedynki w dwoch miejscach, mozna wrzucic w zmienna zeby chociazby wyeliminowac prawdopodobienstwo pomylki jakbys cos zmienial w testach (np pewnego dnia postanowisz ze ten test bedzie usuwal jednak usera o id 2 xD)

@PolishPsycho akurat w testach uzywanie underscore np w

[PolishPsycho]

@Kuriozal: oj tam, gadanie :) Robisz "testIfSessionDeletedAfterUser" i jest git + zgodne z konwencją nazewniczą Sonara :)

[Kuriozal]

@PolishPsycho nigdy nie slyszlam zeby ktos sonarem po testach latal ( ͡° ͜ʖ ͡°)

[PolishPsycho]

@Kuriozal: Nikt nie lata, SonarQube skanuje cały projekt chyba, że robisz exclusions na *Test.java, ew na paczki

[Kuriozal]

@PolishPsycho ale pod testy sa calkowicie inne rulesety niz pod kod produkcyjny, i np defaultowo nie przyczepi sie do underscore :)

[PolishPsycho]

@Kuriozal: No nie masz racji, przecież wysłałem dokładnego linka który obowiązuje dla testów

https://rules.sonarsource.com/java/type/Code%20Smell/RSPEC-3578/

[Kuriozal]

@PolishPsycho ok. My bad :)

To oznacza ze w kazdym projekcie w jakim bylem ktos konfigurujacy sonara wylaczal te rulesety ;D

[PolishPsycho]

@Kuriozal: Spoko :) No ja właśnie siedzę sporo w Sonarze dlatego w miarę kojarzę te rulesety xD

[pulaas]

@Kamishimi: Nie tylko zmienne ale często nawet stałe. A jak coś się powtarza w kilku miejscach, to dobrze jest stworzyć interface, w którym będziesz przechowywał stałe dostępne dla wszystkich testów w aplikacji.

[cochese]

Czy w testach powinienem tworzyć zmienne?

@Kamishimi: Nie, jest to absolutnie zabronione, za każdym razem jak używasz zmiennej w teście JVM eksploduje i zabija małego kotka. (⌐ ͡■ ͜ʖ ͡■)

Testy to kod jak każdy inny. Jeżeli użycie zmiennej zwiększy czytelność kodu albo pozwoli ci zachować DRY (Don't Repeat Yourself) to śmiało.