Problem z tworzeniem obiektów w testach

Ale jaki konkretnie masz błąd? I musisz sam nadawać te IDki? nie możesz sprawdzać co ponadawał hibernate?

KamilAdam napisał(a):

Ale jaki konkretnie masz błąd? I musisz sam nadawać te IDki? nie możesz sprawdzać co ponadawał hibernate?

findQuizByStatus_test() działa

findQuizByUserId_Test()
org.opentest4j.AssertionFailedError:
Expected :5
Actual :1

findQuizByCategoryId_Test :
org.opentest4j.AssertionFailedError:
Expected :false
Actual :true

LukaszCh233 napisał(a):

KamilAdam napisał(a):

Ale jaki konkretnie masz błąd? I musisz sam nadawać te IDki? nie możesz sprawdzać co ponadawał hibernate?

findQuizByStatus_test() działa

findQuizByUserId_Test()
org.opentest4j.AssertionFailedError:
Expected :5
Actual :1

findQuizByCategoryId_Test :
org.opentest4j.AssertionFailedError:
Expected :false
Actual :true

No to po co sprawdzasz IDki nadawane przez hibernate? co cię one obchodzą? to szczegół implementacyjny. Sprawdzaj nazwy i styknie

Nie tak powinny być te testy napisane.

@SpringBootTest
@ActiveProfiles("test")
public class QuizRepositoryTest {
    @Autowired
    QuizRepository quizRepository;
    @Autowired
    QuizCategoryRepository quizCategoryRepository;

    @BeforeEach
    public  void setUp() {
        quizRepository.deleteAll();
        quizCategoryRepository.deleteAll();
    }

    @Test
    public void findQuizByUserId_Test() {
        long userId = newUser();
        newQuiz("quiz", userId);
        List<Quiz> quizList = quizRepository.findByUserId(userId);
        assertEquals(quizList.get(0).getTitle(), "quiz");
    }

    @Test
    public void findPublicQuiz() {
        newQuizWithStatus("private-quiz", Status.PRIVATE);
        newQuizWithStatus("public-quiz", Status.PUBLIC);
        List<Quiz> quizList = quizRepository.findByStatus(Status.PUBLIC);
        assertEquals(quizList.get(0).getTitle(), "public-quiz");
    }

    @Test
    public void findQuizByCategoryId() {
        long categoryId = newQuizCategory();
        newQuizWithCategory("quiz-name", categoryId);
        List<Quiz> quizList = quizRepository.findByQuizCategoryId(categoryId);
        assertEquals(quizList.get(0).getTitle(), "quiz-name");
    }
}

1. Nie ma sensu robić asercji na .empty() albo .size(), to sprawia że testy są fragile.
2. Nie ma sensu robić tyle asercji ile Ty robisz, testy mają być specyficzne, pod jedno konkretne zachowanie - nie mają "przyklepać" aktualnego stanu.
3. Nie używaj zahardcodzony'ch wartości, chyba że mają znaczenie domenowe. Jak dodasz więcej recordów do bazy, to te 1L staną się nieprawdziwe.
4. To given/when//then to jakaś próba dopasowania się do BDD?

Riddle napisał(a):

Nie tak powinny być te testy napisane.

Kopiuj

@SpringBootTest
@ActiveProfiles("test")
public class QuizRepositoryTest {
    @Autowired
    QuizRepository quizRepository;
    @Autowired
    QuizCategoryRepository quizCategoryRepository;

    @BeforeEach
    public  void setUp() {
        quizRepository.deleteAll();
        quizCategoryRepository.deleteAll();
    }

    @Test
    public void findQuizByUserId_Test() {
        long userId = newUser();
        newQuiz("quiz", userId);
        List<Quiz> quizList = quizRepository.findByUserId(userId);
        assertEquals(quizList.get(0).getTitle(), "quiz");
    }

    @Test
    public void findPublicQuiz() {
        newQuizWithStatus("private-quiz", Status.PRIVATE);
        newQuizWithStatus("public-quiz", Status.PUBLIC);
        List<Quiz> quizList = quizRepository.findByStatus(Status.PUBLIC);
        assertEquals(quizList.get(0).getTitle(), "public-quiz");
    }

    @Test
    public void findQuizByCategoryId() {
        long categoryId = newQuizCategory();
        newQuizWithCategory("quiz-name", categoryId);
        List<Quiz> quizList = quizRepository.findByQuizCategoryId(categoryId);
        assertEquals(quizList.get(0).getTitle(), "quiz-name");
    }
}

1. Nie ma sensu robić asercji na .empty() albo .size(), to sprawia że testy są fragile.
2. Nie ma sensu robić tyle asercji ile Ty robisz, testy mają być specyficzne, pod jedno konkretne zachowanie - nie mają "przyklepać" aktualnego stanu.
3. Nie używaj zahardcodzony'ch wartości, chyba że mają znaczenie domenowe. Jak dodasz więcej recordów do bazy, to te 1L staną się nieprawdziwe.
4. To given/when//then to jakaś próba dopasowania się do BDD?

Co ja poradzę, że wszędzie są pokazane testy żeby robić w sposób jak zrobiłem, nie mogę znaleźć żadnego sensownego poradnika do tego.
To co ty napisałeś wygląda elegancko.
Jeśli chodzi o te given when then to ktoś mi kiedyś powiedział, że tak jest okey to tak robię.
Ciężko z tymi testami jest żeby to dobrze napisać jak Ci ludzie nie powiedzą co to ogarniają.

LukaszCh233 napisał(a):

Co ja poradzę, że wszędzie są pokazane testy żeby robić w sposób jak zrobiłem, nie mogę znaleźć żadnego sensownego poradnika do tego.
To co ty napisałeś wygląda elegancko.
Jeśli chodzi o te given when then to ktoś mi kiedyś powiedział, że tak jest okey to tak robię.
Ciężko z tymi testami jest żeby to dobrze napisać jak Ci ludzie nie powiedzą co to ogarniają.

Fakt, dużo jest słabych źródeł. Ludzie przykładają zbyt dużą uwagę do formy testów (framework, biblioteka, komentarze given/when/then, etc.), a zbyt małą uwagę do tego jak te testy faktycznie wpływają na ich pracę.

Jak chcesz, to następnym razem będziesz pisał jakiś nowy kod i będziesz się zastanawiał jak napisać dobry test pod to, to załóż nowy wątek i pingnij mnie. Wymyślimy coś.

Riddle napisał(a):

LukaszCh233 napisał(a):

Co ja poradzę, że wszędzie są pokazane testy żeby robić w sposób jak zrobiłem, nie mogę znaleźć żadnego sensownego poradnika do tego.
To co ty napisałeś wygląda elegancko.
Jeśli chodzi o te given when then to ktoś mi kiedyś powiedział, że tak jest okey to tak robię.
Ciężko z tymi testami jest żeby to dobrze napisać jak Ci ludzie nie powiedzą co to ogarniają.

Fakt, dużo jest słabych źródeł. Ludzie przykładają zbyt dużą uwagę do formy testów (framework, biblioteka, komentarze given/when/then, etc.), a zbyt małą uwagę do tego jak te testy faktycznie wpływają na ich pracę.

Jak chcesz, to następnym razem będziesz pisał jakiś nowy kod i będziesz się zastanawiał jak napisać dobry test pod to, to załóż nowy wątek i pingnij mnie. Wymyślimy coś.

O super chętnie skorzystam tylko nie wiem czy ogarnę napisanie testów przed projektem. W trakcie pisania kodu jeszcze nie mam planu konkretnego na projekt a co dopiero napisać testy pod to. Będę się odzywać tylko najpierw musze ogarnąć te 2 projekty co mam zaczęte i będę myśleć nad czymś nowym.

Jeszcze jak możesz mi wyjaśnić co tu sie wydarzyło:

long userId = newUser();  
newQuizWithStatus("private-quiz", Status.PRIVATE);
 long categoryId = newQuizCategory();
 newQuizWithCategory("quiz-name", categoryId);

pierwszy raz coś takiego widzę chyba

LukaszCh233 napisał(a):

O super chętnie skorzystam tylko nie wiem czy ogarnę napisanie testów przed projektem. W trakcie pisania kodu jeszcze nie mam planu konkretnego na projekt a co dopiero napisać testy pod to.

No to zdradzę Ci sekret. Ja też nie (i żaden rozsądny programista) nie ma zazwyczaj planu na projekt. Ale dobra wiadomość jest taka, że nie musisz mieć planu żeby zacząć pisać testy

LukaszCh233 napisał(a):

Będę się odzywać tylko najpierw musze ogarnąć te 2 projekty co mam zaczęte i będę myśleć nad czymś nowym.

Jak będziesz dodawał nową funkcję do istniejącego projektu, to pod to też warto napisać testy. Odezwij się wtedy też.

LukaszCh233 napisał(a):

Jeszcze jak możesz mi wyjaśnić co tu sie wydarzyło:

Kopiuj

long userId = newUser();  
newQuizWithStatus("private-quiz", Status.PRIVATE);
 long categoryId = newQuizCategory();
 newQuizWithCategory("quiz-name", categoryId);

pierwszy raz coś takiego widzę chyba

Twoje testy które pokazałeś w pierwszym wątku które napisałeś testowały implementację. Popularny błąd, zwłaszcza jak się czyta słabej jakości źródła o testach.

To co ja zrobiłem, to spróbowałem wydedukować jaki test faktycznie chcesz napisać. Dla przykładu, wziąłem Twój test:

@Test
public void findQuizByStatus_test() {
  //Given
  Status status = Status.PUBLIC;

  //When
  List<Quiz> quizList = quizRepository.findByStatus(status);

  //Then
  assertFalse(quizList.isEmpty());
  assertEquals(2, quizList.size());
  assertEquals(quizList.get(0).getStatus(), Status.PUBLIC);
  assertEquals(quizList.get(1).getStatus(), Status.PUBLIC);
}

Domyślam się, że ten test miał testować filtrowanie quizów po statusie. Stwierdziłem więc że usunę ten test, i napiszę nowy. Zastanowiłem się, co jest konieczne żeby filtrować po statusie? Muszę mieć quiz, który muszę zwrócić. Ale jeden quiz nie wystarczy, bo wtedy filtrowanie nie miałoby sensu - po co filtrować jeden quiz? Uznałem że koniecznie jest stworzenie dwóch quizów na potrzeby testu - jeden publiczny, jeden prywatny. Test powinien sprawdzić, czy kiedy mamy jeden publiczny i jeden prywatny quiz, to czy zostanie zwrócony ten publiczny. To byłby dobry test.

Podzieliłem to na kroki:

• stwórz prywatny quiz
• stwórz publiczny quiz
• wczytaj quizy filtrowane po tym czy jest publiczny
• sprawdź czy zwrócono ten publiczny quiz

To jest opis Twojego zachowania. Jeśli któregokolwiek z tych kroków by zabrakło, ciężko byłoby powiedzieć że ten test testuje cokolwiek. Alternatywą byłoby stworzyć tylko prywatny quiz (bez publicznego), i upewnić się że zwrócona lista jest pusta.

I potem wystarczyło tylko przepisać te kroki na kod:

• stwórz prywatny quiz -> newQuizWithStatus("private-quiz", Status.PRIVATE)
• stwórz publiczny quiz -> newQuizWithStatus("public-quiz", Status.PUBLIC)
• wczytaj quizy filtrowane po tym czy jest publiczny -> quizRepository.findByStatus(Status.PUBLIC);
• sprawdź czy zwrócono drugi quiz -> assertEquals(quizList.get(0).getTitle(), "public-quiz")

Oczywiście metody newQuizWithStatus() to metoda pomocnicza mająca na celu stworzenie quizu z odpowiednim tytułem i statusem. Należy ją napisać dodatkowo, jako uzupełnienie testu, tak żeby test miał tylko niezbędne kroki, a wszystkie szczegóły były w osobnych metodach.

Riddle napisał(a):

LukaszCh233 napisał(a):

O super chętnie skorzystam tylko nie wiem czy ogarnę napisanie testów przed projektem. W trakcie pisania kodu jeszcze nie mam planu konkretnego na projekt a co dopiero napisać testy pod to.

No to zdradzę Ci sekret. Ja też nie (i żaden rozsądny programista) nie ma zazwyczaj planu na projekt. Ale dobra wiadomość jest taka, że nie musisz mieć planu żeby zacząć pisać testy

LukaszCh233 napisał(a):

Będę się odzywać tylko najpierw musze ogarnąć te 2 projekty co mam zaczęte i będę myśleć nad czymś nowym.

Jak będziesz dodawał nową funkcję do istniejącego projektu, to pod to też warto napisać testy. Odezwij się wtedy też.

LukaszCh233 napisał(a):

Jeszcze jak możesz mi wyjaśnić co tu sie wydarzyło:

Kopiuj

long userId = newUser();  
newQuizWithStatus("private-quiz", Status.PRIVATE);
 long categoryId = newQuizCategory();
 newQuizWithCategory("quiz-name", categoryId);

pierwszy raz coś takiego widzę chyba

Twoje testy które pokazałeś w pierwszym wątku które napisałeś testowały implementację. Popularny błąd, zwłaszcza jak się czyta słabej jakości źródła o testach.

To co ja zrobiłem, to spróbowałem wydedukować jaki test faktycznie chcesz napisać. Dla przykładu, wziąłem Twój test:

Kopiuj

@Test
public void findQuizByStatus_test() {
  //Given
  Status status = Status.PUBLIC;

  //When
  List<Quiz> quizList = quizRepository.findByStatus(status);

  //Then
  assertFalse(quizList.isEmpty());
  assertEquals(2, quizList.size());
  assertEquals(quizList.get(0).getStatus(), Status.PUBLIC);
  assertEquals(quizList.get(1).getStatus(), Status.PUBLIC);
}

Domyślam się, że ten test miał testować filtrowanie quizów po statusie. Stwierdziłem więc że usunę ten test, i napiszę nowy. Zastanowiłem się, co jest konieczne żeby filtrować po statusie? Muszę mieć quiz, który muszę zwrócić. Ale jeden quiz nie wystarczy, bo wtedy filtrowanie nie miałoby sensu - po co filtrować jeden quiz? Uznałem że koniecznie jest stworzenie dwóch quizów na potrzeby testu - jeden publiczny, jeden prywatny. Test powinien sprawdzić, czy kiedy mamy jeden publiczny i jeden prywatny quiz, to czy zostanie zwrócony ten publiczny. To byłby dobry test.

Podzieliłem to na kroki:

• stwórz prywatny quiz
• stwórz publiczny quiz
• wczytaj quizy filtrowane po tym czy jest publiczny
• sprawdź czy zwrócono ten publiczny quiz

To jest opis Twojego zachowania. Jeśli któregokolwiek z tych kroków by zabrakło, ciężko byłoby powiedzieć że ten test testuje cokolwiek. Alternatywą byłoby stworzyć tylko prywatny quiz (bez publicznego), i upewnić się że zwrócona lista jest pusta.

I potem wystarczyło tylko przepisać te kroki na kod:

• stwórz prywatny quiz -> newQuizWithStatus("private-quiz", Status.PRIVATE)
• stwórz publiczny quiz -> newQuizWithStatus("public-quiz", Status.PUBLIC)
• wczytaj quizy filtrowane po tym czy jest publiczny -> quizRepository.findByStatus(Status.PUBLIC);
• sprawdź czy zwrócono drugi quiz -> assertEquals(quizList.get(0).getTitle(), "public-quiz")

Oczywiście metody newQuizWithStatus() to metoda pomocnicza mająca na celu stworzenie quizu z odpowiednim tytułem i statusem. Należy ją napisać dodatkowo, jako uzupełnienie testu, tak żeby test miał tylko niezbędne kroki, a wszystkie szczegóły były w osobnych metodach.

Testy wyglądają schludnie ale roboty chyba jest więcej tworzac specjalne metody do testów.

@Test
    public void findQuizByUserId_Test() {
        createQuiz("quiz", 1L);

        List<Quiz> quizList = quizRepository.findByUserId(1L);

        assertEquals(quizList.get(0).getTitle(), "quiz");
    }

  private void createQuiz(String title, Long userId) {
        QuizCategory quizCategory = new QuizCategory(null, "TestCategory");
        quizCategoryRepository.save(quizCategory);

        Quiz quiz = new Quiz();
        quiz.setTitle(title);
        quiz.setUserId(userId);
        quiz.setQuizCategory(quizCategory);
        quiz.setStatus(Status.PUBLIC);
        quizRepository.save(quiz);
    }

Actual value of parameter 'title' is always 'quiz'
Actual value of parameter 'userId' is always '1'

Test działa ale coś chyba nie tak robię?

LukaszCh233 napisał(a):

Testy wyglądają schludnie ale roboty chyba jest więcej tworzac specjalne metody do testów.

Pisania jest trochę więcej, 5-7 linijek więcej, ale dla programisty to jest tyle co nic. Jednak zysk z tego jest nieoceniony.

LukaszCh233 napisał(a):

Test działa ale coś chyba nie tak robię?

Czy ja wiem? Chyba wszystko dobrze. Jedyne do czego bym się przyczepił, to czy na pewno wszystkie te wartości muszą tam być?

Nie wystarczy coś takiego?

private void createQuiz(String title, Long userId) {
    Quiz quiz = new Quiz();
    quiz.setTitle(title);
    quiz.setUserId(userId);
    quizRepository.save(quiz);
}

Do testów warto przekazać absolutne minimum.

Riddle napisał(a):

LukaszCh233 napisał(a):

Testy wyglądają schludnie ale roboty chyba jest więcej tworzac specjalne metody do testów.

Pisania jest trochę więcej, 5-7 linijek więcej, ale dla programisty to jest tyle co nic. Jednak zysk z tego jest nieoceniony.

LukaszCh233 napisał(a):

Test działa ale coś chyba nie tak robię?

Czy ja wiem? Chyba wszystko dobrze. Jedyne do czego bym się przyczepił, to czy na pewno wszystkie te wartości muszą tam być?

Nie wystarczy coś takiego?

Kopiuj

private void createQuiz(String title, Long userId) {
    Quiz quiz = new Quiz();
    quiz.setTitle(title);
    quiz.setUserId(userId);
    quizRepository.save(quiz);
}

Do testów warto przekazać absolutne minimum.

Tak to wygląda:

@Test
    public void findQuizByUserId_Test() {
        createQuiz("quiz", 1L);

        List<Quiz> quizList = quizRepository.findByUserId(1L);

        assertEquals(quizList.get(0).getTitle(), "quiz");
    }

    @Test
    public void findPublicQuiz() {
        newQuizWithStatus("public", Status.PUBLIC);
        newQuizWithStatus("private", Status.PRIVATE);

        List<Quiz> quizList = quizRepository.findByStatus(Status.PUBLIC);

        assertEquals(quizList.get(0).getTitle(), "public");
    }

    @Test
    public void findQuizByCategoryId() {
        long categoryId = newQuizCategory();
        newQuizWithCategory("quiz", categoryId);

        List<Quiz> quizList = quizRepository.findByQuizCategoryId(categoryId);
        assertEquals(quizList.get(0).getTitle(), "quiz");
    }

    private void createQuiz(String title, Long userId) {
        QuizCategory quizCategory = new QuizCategory(null, "TestCategory");
        quizCategoryRepository.save(quizCategory);

        Quiz quiz = new Quiz();
        quiz.setTitle(title);
        quiz.setUserId(userId);
        quiz.setQuizCategory(quizCategory);
        quiz.setStatus(Status.PUBLIC);
        quizRepository.save(quiz);
    }

    private void newQuizWithStatus(String title, Status status) {
        QuizCategory quizCategory = new QuizCategory(null, "TestCategory");
        quizCategoryRepository.save(quizCategory);

        Quiz quiz = new Quiz();
        quiz.setTitle(title);
        quiz.setUserId(1L);
        quiz.setQuizCategory(quizCategory);
        quiz.setStatus(status);
        quizRepository.save(quiz);
    }

    private void newQuizWithCategory(String title, Long categoryId) {
        Quiz quiz = new Quiz();
        quiz.setTitle(title);
        quiz.setUserId(1L);
        quiz.setStatus(Status.PUBLIC);
        quizRepository.save(quiz);
    }

    private long newQuizCategory() {
        QuizCategory quizCategory = new QuizCategory(null, "TestCategory");
        quizCategoryRepository.save(quizCategory);
        return quizCategory.getId();
    }
}

Test findQuizByCategoryId nie działa bo w metodzie nie dałem setQuizCategory.
Właśnie mam taki problem, że przez to, że mam valid w encjach to musze podawać te wartości bo inaczej jest błąd.

ConstraintViolationImpl{interpolatedMessage='Quiz category must be provided', propertyPath=quizCategory, rootBeanClass=class com.example.quiz_world.quiz.entity.Quiz, messageTemplate='Quiz category must be provided'}

LukaszCh233 napisał(a):

Właśnie mam taki problem, że przez to, że mam valid w encjach to musze podawać te wartości bo inaczej jest błąd.

Kopiuj

ConstraintViolationImpl{interpolatedMessage='Quiz category must be provided', propertyPath=quizCategory, rootBeanClass=class com.example.quiz_world.quiz.entity.Quiz, messageTemplate='Quiz category must be provided'}

Okej, no to faktycznie musisz to dodać.

Testy wyglądają schludnie ale roboty chyba jest więcej tworzac specjalne metody do testów.

Słuszna uwaga. Żeby kod wyglądał schludniej, potrzebne jest więcej roboty :) i dotyczy to także testów, kod to kod. Aczkolwiek takie wyciąganie metod pomocniczych do testów zwykle jest bardzo proste, często nawet do ogarnięcia automatycznym refactoringiem (extract method w IntelliJ pokaże ci wszystkie miejsca, gdzie można to zrobić).

Riddle napisał(a):

LukaszCh233 napisał(a):

Właśnie mam taki problem, że przez to, że mam valid w encjach to musze podawać te wartości bo inaczej jest błąd.

Kopiuj

ConstraintViolationImpl{interpolatedMessage='Quiz category must be provided', propertyPath=quizCategory, rootBeanClass=class com.example.quiz_world.quiz.entity.Quiz, messageTemplate='Quiz category must be provided'}

Okej, no to faktycznie musisz to dodać.

w takim razie jak to poprawić bo musze tu mieć quizCategory

    private void newQuizWithCategory(String title, Long categoryId) {
 QuizCategory quizCategory = new QuizCategory(null, "TestCategory");
        quizCategoryRepository.save(quizCategory);

        Quiz quiz = new Quiz();
        quiz.setTitle(title);
        quiz.setUserId(1L);
        quiz.setQuizCategory();
        quiz.setStatus(Status.PUBLIC);
        quizRepository.save(quiz);
    }

LukaszCh233 napisał(a):

Riddle napisał(a):

LukaszCh233 napisał(a):

Właśnie mam taki problem, że przez to, że mam valid w encjach to musze podawać te wartości bo inaczej jest błąd.

Kopiuj

ConstraintViolationImpl{interpolatedMessage='Quiz category must be provided', propertyPath=quizCategory, rootBeanClass=class com.example.quiz_world.quiz.entity.Quiz, messageTemplate='Quiz category must be provided'}

Okej, no to faktycznie musisz to dodać.

w takim razie jak to poprawić bo musze tu mieć quizCategory

Kopiuj

    private void newQuizWithCategory(String title, Long categoryId) {
 QuizCategory quizCategory = new QuizCategory(null, "TestCategory");
        quizCategoryRepository.save(quizCategory);

        Quiz quiz = new Quiz();
        quiz.setTitle(title);
        quiz.setUserId(1L);
        quiz.setQuizCategory();
        quiz.setStatus(Status.PUBLIC);
        quizRepository.save(quiz);
    }

Dobra juz sam na to wpadłem :) teraz chyba jest dobrze wszystko.

  @Test
    public void findQuizByCategoryId() {
        QuizCategory quizCategory = newQuizCategory();
        newQuizWithCategory("quiz",quizCategory);

        List<Quiz> quizList = quizRepository.findByQuizCategoryId(quizCategory.getId());
        assertEquals(quizList.get(0).getTitle(), "quiz");
    }

  private void newQuizWithCategory(String title, QuizCategory quizCategory) {
        Quiz quiz = new Quiz();
        quiz.setTitle(title);
        quiz.setUserId(1L);
        quiz.setQuizCategory(quizCategory);
        quiz.setStatus(Status.PUBLIC);
        quizRepository.save(quiz);
    }

    private QuizCategory newQuizCategory() {
        QuizCategory quizCategory = new QuizCategory(null, "TestCategory");
        return quizCategoryRepository.save(quizCategory);
    }

A jak z testami service? Może być tak?

@SpringBootTest
@ActiveProfiles("test")
public class QuizCategoryServiceTest {
    @Autowired
    QuizCategoryService quizCategoryService;
    @Autowired
    QuizCategoryRepository quizCategoryRepository;

    @BeforeEach
    public void setUp() {
        quizCategoryRepository.deleteAll();
    }

    @Test
    public void createCategoryForQuiz_test() {
        QuizCategory quizCategory = new QuizCategory(null, "testCategory");

        QuizCategory createCategory = quizCategoryService.createQuizCategory(quizCategory);

        Assertions.assertEquals(createCategory.getName(), "testCategory");
    }

    @Test
    public void findAllQuizCategory_test() {
        QuizCategory quizCategory = new QuizCategory(null, "testCategory");
        QuizCategory quizCategory1 = new QuizCategory(null, "testCategory1");
        quizCategoryRepository.save(quizCategory);
        quizCategoryRepository.save(quizCategory1);

        List<QuizCategoryDTO> quizCategoryList = quizCategoryService.findAllQuizCategories();

        Assertions.assertFalse(quizCategoryList.isEmpty());
        Assertions.assertEquals(quizCategoryList.get(0).name(), "testCategory");
        Assertions.assertEquals(quizCategoryList.get(1).name(), "testCategory1");
    }
}

LukaszCh233 napisał(a):

A jak z testami service? Może być tak?

Nie podchodź do tego że chcesz testować jakiś serwis albo klasę. Pomyśl o tym jakie zachowanie chcesz testować - co ma program robić. Nie to gdzie akurat jakiś kod siedzi.

A co do Twojego kodu, to moim zdaniem te testy nic nie testują One prawie zawsze przejdą. Należałoby je napisać inaczej.

Riddle napisał(a):

LukaszCh233 napisał(a):

A jak z testami service? Może być tak?

Nie podchodź do tego że chcesz testować jakiś serwis albo klasę. Pomyśl o tym jakie zachowanie chcesz testować - co ma program robić. Nie to gdzie akurat jakiś kod siedzi.

A co do Twojego kodu, to moim zdaniem te testy nic nie testują One prawie zawsze przejdą. Należałoby je napisać inaczej.

No np. chce testować tworzenie kategorii do quizu a do tego jest metoda createQuizCategory wiec no musze chyba ją przetestować?

LukaszCh233 napisał(a):

Riddle napisał(a):

LukaszCh233 napisał(a):

A jak z testami service? Może być tak?

Nie podchodź do tego że chcesz testować jakiś serwis albo klasę. Pomyśl o tym jakie zachowanie chcesz testować - co ma program robić. Nie to gdzie akurat jakiś kod siedzi.

A co do Twojego kodu, to moim zdaniem te testy nic nie testują One prawie zawsze przejdą. Należałoby je napisać inaczej.

No np. chce testować tworzenie kategorii do quizu a do tego jest metoda createQuizCategory wiec no musze chyba ją przetestować?

To jak ktoś mi wyjaśnij jak pisać te testy service czy już lepiej zostać przy tym co jest?

@Test
public void createCategoryForQuiz_test() {
    QuizCategory quizCategory = new QuizCategory(null, "testCategory");

    QuizCategory createCategory = quizCategoryService.createQuizCategory(quizCategory);

    Assertions.assertEquals(createCategory.getName(), "testCategory");
}

Tutaj testuje zachowanie tworzenia kategorii do quizu za pomocą metody createQuizCategory. Czyli ta część programu ma po prostu stworzyć mi kategorie do quizów.

LukaszCh233 napisał(a):

Tutaj testuje zachowanie tworzenia kategorii do quizu za pomocą metody createQuizCategory. Czyli ta część programu ma po prostu stworzyć mi kategorie do quizów.

"Tworzenie kategorii quizów" to nie jest dobry test. Równie dobrze mógłbyś napisać assertTrue(true).

Dobry testy to byłby np. "Kiedy dodam quiz A, do kategorii B, i wyszukamy quizy w kategorii B, to zobaczę quiz A", czyli faktyczne zachowanie programu. Oczywiście konkretny test zależy od Twojego przypadku użycia, i od tego co Twój program ma robić.

Mam wrażenie że masz takie przekonanie że napisałeś kod, i teraz chcesz napisać "jakiś test", byleby tylko był; a takie coś nie ma sensu. Testy mają być pomocą przy programowaniu, jeśli tego nie robią to są bezwartościowe.

@Riddle: to taki jeden test użyje kilku metod więc nie trzeba pisać dla każdego oddzielnie. Chyba, że to nie mam patrzeć na te metody wgl czy działają poprawnie. Ty jak coś wytłumaczysz to od razu staje się jaśniejsze czemu nie mogą tak w internecie w poradnikach...

Riddle napisał(a):

LukaszCh233 napisał(a):

Tutaj testuje zachowanie tworzenia kategorii do quizu za pomocą metody createQuizCategory. Czyli ta część programu ma po prostu stworzyć mi kategorie do quizów.

"Tworzenie kategorii quizów" to nie jest dobry test. Równie dobrze mógłbyś napisać assertTrue(true).

Dobry testy to byłby np. "Kiedy dodam quiz A, do kategorii B, i wyszukamy quizy w kategorii B, to zobaczę quiz A", czyli faktyczne zachowanie programu. Oczywiście konkretny test zależy od Twojego przypadku użycia, i od tego co Twój program ma robić.

Mam wrażenie że masz takie przekonanie że napisałeś kod, i teraz chcesz napisać "jakiś test", byleby tylko był; a takie coś nie ma sensu. Testy mają być pomocą przy programowaniu, jeśli tego nie robią to są bezwartościowe.

Co myślisz o tym?

@Test
public void whenQuizIsAddedToCreatedCategory_thenQuizCanBeFoundInThatCategory_test() {
    QuizCategory quizCategory = new QuizCategory(null, "testCategory");
    quizCategoryService.createQuizCategory(quizCategory);

    newQuizWithCategory("test",quizCategory);

    List<QuizDTO> quizzesInCategory = quizService.findQuizByCategory(quizCategory.getId());

    Assertions.assertFalse(quizzesInCategory.isEmpty());
    Assertions.assertEquals(quizzesInCategory.get(0).title(),"test");
}

private void newQuizWithCategory(String title, QuizCategory quizCategory) {
    Quiz quiz = new Quiz();
    quiz.setTitle(title);
    quiz.setUserId(1L);
    quiz.setQuizCategory(quizCategory);
    quiz.setStatus(Status.PUBLIC);
    quizRepository.save(quiz);
}

LukaszCh233 napisał(a):

@Riddle: to taki jeden test użyje kilku metod więc nie trzeba pisać dla każdego oddzielnie. Chyba, że to nie mam patrzeć na te metody wgl czy działają poprawnie.

Tak, często test korzysta z kilku metod. Pamiętaj - testy nie mają testować klas lub metod. Mają testować zachowanie.

Jeśli słyszałeś albo czytałeś jak ktoś kiedyś pisał "ta metoda/klasa nie ma testu, trzeba dopisać", to to jest niepomocne. Należałoby powiedzieć: "ta funkcjonalność programu nie ma testu, należałoby pod nią dopisać tesT".

LukaszCh233 napisał(a):

Co myślisz o tym?

Za dużo rzeczy niepotrzebnych. Nie musisz robić assertFalse(.isEmpty()), jeśli zaraz potem robisz assertEquals(.get(0)). Dodatkowo, czy na prawdę potrzebujesz wszystkie te wartości ustawić w newQuizWithCategory()?

Ja bym to napisał jakoś tak:

@Test
public void quizInCategoryCanBeFoundInThatCategory() {
    newQuizInCategory("apple", "fruits");
    var quizes = quizesInCategory("fruits");
    assertEquals(quizes.get(0).title(), "apple");
}

private List<> quizesInCategory(String categoryName) { // ta funkcja ma zwrócić quizy, po nazwie kategorii
  return quizService.findQuizByCategory(new QuizCategory(null, categoryName).getId()); // tutaj zrób poprawną implementację
}

private void newQuizInCategory(String title, String categoryName) {
    Quiz quiz = new Quiz();
    quiz.setTitle(title); 
    quiz.setQuizCategory(quizCategory(categoryName));
    quiz.setUserId(1L);             // usuń jeśli niepotrzebne
    quiz.setStatus(Status.PUBLIC);  // usuń jeśli niepotrzebne
    quizRepository.save(quiz);
}

private quizCategory(String categoryName) { // ta funkcja ma zwrócić zapisaną kategorię o danej nazwie
  QuizCategory category = new QuizCategory(null, categoryName);
  quizCategoryService.createQuizCategory(category);
  return category;
}

Riddle napisał(a):

LukaszCh233 napisał(a):

@Riddle: to taki jeden test użyje kilku metod więc nie trzeba pisać dla każdego oddzielnie. Chyba, że to nie mam patrzeć na te metody wgl czy działają poprawnie.

Tak, często test korzysta z kilku metod. Pamiętaj - testy nie mają testować klas lub metod. Mają testować zachowanie.

Jeśli słyszałeś albo czytałeś jak ktoś kiedyś pisał "ta metoda/klasa nie ma testu, trzeba dopisać", to to jest niepomocne. Należałoby powiedzieć: "ta funkcjonalność programu nie ma testu, należałoby pod nią dopisać tesT".

LukaszCh233 napisał(a):

Co myślisz o tym?

Za dużo rzeczy niepotrzebnych. Nie musisz robić assertFalse(.isEmpty()), jeśli zaraz potem robisz assertEquals(.get(0)). Dodatkowo, czy na prawdę potrzebujesz wszystkie te wartości ustawić w newQuizWithCategory()?

Ja bym to napisał jakoś tak:

Kopiuj

@Test
public void quizInCategoryCanBeFoundInThatCategory() {
    newQuizInCategory("apple", "fruits");
    var quizes = quizesInCategory("fruits");
    assertEquals(quizes.get(0).title(), "apple");
}

private List<> quizesInCategory(String categoryName) { // ta funkcja ma zwrócić quizy, po nazwie kategorii
  return quizService.findQuizByCategory(new QuizCategory(null, categoryName).getId()); // tutaj zrób poprawną implementację
}

private void newQuizInCategory(String title, String categoryName) {
    Quiz quiz = new Quiz();
    quiz.setTitle(title); 
    quiz.setQuizCategory(quizCategory(categoryName));
    quiz.setUserId(1L);             // usuń jeśli niepotrzebne
    quiz.setStatus(Status.PUBLIC);  // usuń jeśli niepotrzebne
    quizRepository.save(quiz);
}

private quizCategory(String categoryName) { // ta funkcja ma zwrócić zapisaną kategorię o danej nazwie
  QuizCategory category = new QuizCategory(null, categoryName);
  quizCategoryService.createQuizCategory(category);
  return category;
}

Tak jak ostatnio Ci pisałem muszę to dawać bo valid jest w encji

LukaszCh233 napisał(a):

Tak jak ostatnio Ci pisałem muszę to dawać bo valid jest w encji

Jeśli testowanie czegoś staje się bardziej uciążliwe niż musi (np. to że trzeba dodawać te wartości w teście tylko po to żeby byly), to często może to być sygnał że nasz design jest zły, i daje to okazję do zastanowienia się czy na pewno ten valid w encji jest tam niezbędny? Może są latwiejsze sposoby na zapenwienie integralności danych, taki który nie przeszkadza w testowaniu?

Riddle napisał(a):

LukaszCh233 napisał(a):

Tak jak ostatnio Ci pisałem muszę to dawać bo valid jest w encji

Jeśli testowanie czegoś staje się bardziej uciążliwe niż musi (np. to że trzeba dodawać te wartości w teście tylko po to żeby byly), to często może to być sygnał że nasz design jest zły, i daje to okazję do zastanowienia się czy na pewno ten valid w encji jest tam niezbędny? Może są latwiejsze sposoby na zapenwienie integralności danych, taki który nie przeszkadza w testowaniu?

Pewnie jest zamiast notnull w encji to stworzyć jakiegoś exceptiona który będzie to obsługiwał chyba ale wtedy dochodzi trochę w kodzie żeby to obsłużyć. Chyba, że jest jeszcze jakiś sposób?

LukaszCh233 napisał(a):

Pewnie jest zamiast notnull w encji to stworzyć jakiegoś exceptiona który będzie to obsługiwał chyba ale wtedy dochodzi trochę w kodzie żeby to obsłużyć. Chyba, że jest jeszcze jakiś sposób?

Miałem na myśli to że można rozważyć czy w ogóle takich checków nie robić w encji. A tylko w miejscu które je tworzy.

Riddle napisał(a):

LukaszCh233 napisał(a):

Pewnie jest zamiast notnull w encji to stworzyć jakiegoś exceptiona który będzie to obsługiwał chyba ale wtedy dochodzi trochę w kodzie żeby to obsłużyć. Chyba, że jest jeszcze jakiś sposób?

Miałem na myśli to że można rozważyć czy w ogóle takich checków nie robić w encji. A tylko w miejscu które je tworzy.

Czyli po prostu wywalić valid z encji i użyć w controllerach gdzie jest potrzebne? Tylko wtedy mam cały obiekt a nie pojedyncze pola klasy, chyba, że coś mieszam

LukaszCh233 napisał(a):

Czyli po prostu wywalić valid z encji i użyć w controllerach gdzie jest potrzebne? Tylko wtedy mam cały obiekt a nie pojedyncze pola klasy, chyba, że coś mieszam

Nie ma jednej dobrej odpowiedzi.

Sam sobie odpowiedz na pytanie, co jest lepsze:

• Czy lepiej jest mieć valid w encji, i tym samym testy są trudniejsze, bo trzeba tworzyć encje podając te wartości
• Czy lepiej mieć walidację w miejscu które tworzy encję (i stoi między encją i kontrolerem)

Ja bym chyba wolał drugą wersję, ale decyzja jest Twoja. Obie mają swoje wady i zalety.

Riddle napisał(a):

LukaszCh233 napisał(a):

Czyli po prostu wywalić valid z encji i użyć w controllerach gdzie jest potrzebne? Tylko wtedy mam cały obiekt a nie pojedyncze pola klasy, chyba, że coś mieszam

Nie ma jednej dobrej odpowiedzi.

Sam sobie odpowiedz na pytanie, co jest lepsze:

• Czy lepiej jest mieć valid w encji, i tym samym testy są trudniejsze, bo trzeba tworzyć encje podając te wartości
• Czy lepiej mieć walidację w miejscu które tworzy encję (i stoi między encją i kontrolerem)

Ja bym chyba wolał drugą wersję, ale decyzja jest Twoja. Obie mają swoje wady i zalety.

Ale mi mieszasz xD ja tu ledwo co ogarniam a ty mi takie dylematy dajesz. Przez te testy juz mi głowa paruje a tu jeszcze problem z valid. Nie potrafie wymyślić innych testów twoim sposobem.
Bez sensu sie to wydaje ale no nie wiem co mam testować a co nie. Usuwanie wyszukiwanie kategorii update.

 @Test
    public void deleteAllQuizCategoriesAfterFindAll() {
        newQuizCategory("test");
        newQuizCategory("test1");

        List<QuizCategoryDTO> quizCategoryList = quizCategoryService.findAllQuizCategories();

        Assertions.assertEquals(quizCategoryList.size(),2);

        quizCategoryService.deleteAllQuizCategories();

        List<QuizCategory> quizCategories = quizCategoryRepository.findAll();

        Assertions.assertTrue(quizCategories.isEmpty());
    }

Może tak np. jeśli category A zawiera quiz B to aktualizacja category a sprawi, że quiz B uzyska zaktualizowaną category A. Dobry test czy nie?