[Kamishimi]

Cześć, mam taki test.

@test
    void whenUserDeleted_SecondDeletionReturnNotFound() throws Exception {
        Integer userId = idFetcher.fetchByUsername("Mark");

        Cookie sessionId = auth.loginAdminUser();

        mockMvc.perform(delete("/api/v1/users/" + userId).cookie(sessionId))
                .andExpect(status().isOk());

        mockMvc.perform(delete("/api/v1/users/" + userId).cookie(sessionId))
                .andExpect(status().isNotFound());
    }

Zastanawiam się, czy nie powinienem umieścić tego fragmentu kodu w jakiejś metodzie pomocniczej, skoro głównym celem test jest sprawdzenie 404. Co myślicie?

   mockMvc.perform(delete("/api/v1/users/" + userId).cookie(sessionId))
                .andExpect(status().isOk());

#naukaprogramowania #java #programowanie

[GeDox]

@Kamishimi: całego fragmentu nie. Osobną funkcją albo stałą powinno byc:

status().isOk()

status().isNotFound()

elete("/api/v1/users/" + userId).cookie(sessionId)

[Kresse]

@Kamishimi: Po co? Przecież ten test jest czytelny i nazwa metody dokładnie opisuje o co chodzi.

[Kamishimi]

@Kresse: Mam osobny test, który sprawdza, czy usunięcie użytkownika zwraca 200. Nie wiem, czy jest sens sprawdzać to, w kolejnym teście.

@test
void whenUserIsAdmin_SuccessDeletionReturnStatusOk() throws Exception {
Integer userId = idFetcher.fetchByUsername("Mark");

Cookie sessionId = auth.loginAdminUser();

mockMvc.perform(delete("/api/v1/users/" + userId).cookie(sessionId))
.andExpect(status().isOk());
}

@test
void whenUserDeleted_SecondDeletionReturnNotFound() throws Exception {
Integer userId = idFetcher.fetchByUsername("Mark");

Cookie sessionId = auth.loginAdminUser();

mockMvc.perform(delete("/api/v1/users/" + userId).cookie(sessionId))
.andExpect(status().isOk());

mockMvc.perform(delete("/api/v1/users/" + userId).cookie(sessionId))
.andExpect(status().isNotFound());
}