[Kamishimi]

Test sprawdzający usunięcie użytkownika:

   @test
    void whenUserExist_DeleteUser() {
        User user = userBuilder.buildById(1).build();
        userSaver.save(user);

        delete.deleteUserById(1);
        assertThat(userChecker.userExist(1)).isFalse();
    }

Czy jeśli w teście jest widoczny zapis użytkownika userSaver.save(user);, to nie stanowi to problemu? Czy jakoś inaczej to zrobić?

#java

[siedze_w_samych_gaciach]

@Kamishimi: weź bo komuś konto na wykopie skasujesz:)

[Kamishimi]

@siedze_w_samych_gaciach: hehe

[93michu93]

@Kamishimi: to zalezy jak to robisz generalnie powinienes testowac metode z serwisu, ktora uderza do repository.

Moim zdaniem mozesz uzyc uzyc metody save ale ladniej byloby to wrzucic w osobna metode np. createAndSaveUser i ja zawolac w tescie.

A to delete.deleteUserById to co to za gowno? To delete to serwis jest? Jesli tak to czym predzej zmieniaj nazwe na jakies userService

[Kamishimi]

@93michu93: ten UserSaver wygląda tak:

@Component
public class UserSaver {

    @autowired
    private UserDAO userDAO;

    @Transactional
    public void save(User user) {
        int userId = userDAO.save(user);
        user.getRoles().forEach(role -> userDAO.assignRoleToUser(userId, role.getId()));
    }
}