[Kamishimi]

Lepiej pisać testy tak

  @test
    void whenEmailExist_ThrowException() {
        assertException(() -> registration.register(registrationHelper.existEmail),
                CredentialValidationException.class, "Email is already exist!.");
    }

czy tak

@test
    void whenEmailExist_ThrowException() {
        // given
        RegisterRequest request = registrationHelper.existEmail;

        // when & then
        assertException(() -> registration.register(request),
                CredentialValidationException.class, "Email is already exist!.");
    }

#java #naukaprogramowania

[iquiet]

@Kamishimi tak

[DROP_TABLE_Usernames]

@Kamishimi: Testy to strata czasu, no chyba że nie za bardzo ogarniasz to klikanie w kąkuter

[Myzreal]

None of the above

[Kamishimi]

@Myzreal uzasadnij proszę

[Myzreal]

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

[dthprf]

@Kamishimi: ( ͡° ͜ʖ ͡°)

[dthprf]

@Kamishimi:

[Kamishimi]

@Myzreal: ale o co chodzi ?
o registrationHelper.existEmail ?
to nie jest stała

[PanMlecz]

@Kamishimi
lepiej tak:

@test
    void whenEmailExist_ThrowException() {
        // given
        RegisterRequest request = registrationHelper.existEmail;

        // when
       Throwable result =  catchThrowable((() -> registration.register(request))

//then
asserty z  CredentialValidationException.class, "Email is already exist!.");

    }

[Kamishimi]

@PanMlecz ciekawe rozwiązanie, nie pomyślałabym żeby tak zrobić

[Kresse]

@Kamishimi: Jedyneczka. Przecież z opisową nazwą metody ten test czyta się w sekundę. Te given, when, then są równie przydatne jak komentarze typu "// Name" na zmiennej nazwanej "name".