From 23ad0c67d22677ee271705877603d19b2607c0d4 Mon Sep 17 00:00:00 2001 From: Cato Olsen Date: Tue, 17 Oct 2023 13:22:03 +0200 Subject: [PATCH] =?UTF-8?q?Rettet=20tester=20(satte=20eksplisitt=20ID=20f?= =?UTF-8?q?=C3=B8r=20persistering),=20fjernet=20warnings.?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../service/BestillingMalServiceTest.java | 33 +++++-------------- .../OrganisasjonBestillingMalServiceTest.java | 24 ++++---------- 2 files changed, 15 insertions(+), 42 deletions(-) diff --git a/apps/dolly-backend/src/test/java/no/nav/dolly/service/BestillingMalServiceTest.java b/apps/dolly-backend/src/test/java/no/nav/dolly/service/BestillingMalServiceTest.java index a4349b3ee20..941853f4068 100644 --- a/apps/dolly-backend/src/test/java/no/nav/dolly/service/BestillingMalServiceTest.java +++ b/apps/dolly-backend/src/test/java/no/nav/dolly/service/BestillingMalServiceTest.java @@ -1,17 +1,11 @@ package no.nav.dolly.service; -import com.fasterxml.jackson.databind.ObjectMapper; import no.nav.dolly.MockedJwtAuthenticationTokenUtils; import no.nav.dolly.domain.jpa.Bestilling; import no.nav.dolly.domain.jpa.BestillingMal; import no.nav.dolly.domain.jpa.Bruker; import no.nav.dolly.domain.jpa.Testgruppe; -import no.nav.dolly.repository.BestillingMalRepository; -import no.nav.dolly.repository.BestillingRepository; -import no.nav.dolly.repository.BrukerFavoritterRepository; -import no.nav.dolly.repository.BrukerRepository; -import no.nav.dolly.repository.IdentRepository; -import no.nav.dolly.repository.TestgruppeRepository; +import no.nav.dolly.repository.*; import org.flywaydb.core.Flyway; import org.junit.jupiter.api.AfterEach; import org.junit.jupiter.api.BeforeEach; @@ -31,14 +25,9 @@ import java.time.LocalDate; import java.time.LocalDateTime; -import static org.hamcrest.Matchers.empty; -import static org.hamcrest.Matchers.hasSize; -import static org.hamcrest.Matchers.is; +import static org.hamcrest.Matchers.*; import static org.springframework.boot.test.context.SpringBootTest.WebEnvironment.RANDOM_PORT; -import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.delete; -import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get; -import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.post; -import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.put; +import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.*; import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.jsonPath; import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status; @@ -48,20 +37,18 @@ @EnableAutoConfiguration @ComponentScan("no.nav.dolly") @AutoConfigureMockMvc(addFilters = false) -public class BestillingMalServiceTest { +class BestillingMalServiceTest { private final static String MALNAVN = "test"; private final static String NYTT_MALNAVN = "nyttMalnavn"; private final static String BEST_KRITERIER = "Testeteste"; private static final Bruker DUMMY_EN = Bruker.builder() - .id(1L) .brukerId("testbruker_en") .brukernavn("test_en") .brukertype(Bruker.Brukertype.AZURE) .epost("epost@test_en") .build(); private static final Bruker DUMMY_TO = Bruker.builder() - .id(2L) .brukerId("testbruker_to") .brukernavn("test_to") .brukertype(Bruker.Brukertype.AZURE) @@ -88,8 +75,6 @@ public class BestillingMalServiceTest { @Autowired private IdentRepository identRepository; @Autowired - private ObjectMapper objectMapper; - @Autowired private Flyway flyway; @Transactional @@ -114,8 +99,8 @@ public void afterEach() { void shouldCreateAndGetMaler() throws Exception { - var bruker_en = brukerRepository.findBrukerByBrukerId(DUMMY_EN.getBrukerId()).get(); - var bruker_to = brukerRepository.findBrukerByBrukerId(DUMMY_TO.getBrukerId()).get(); + var bruker_en = brukerRepository.findBrukerByBrukerId(DUMMY_EN.getBrukerId()).orElseThrow(); + var bruker_to = brukerRepository.findBrukerByBrukerId(DUMMY_TO.getBrukerId()).orElseThrow(); saveDummyBestillingMal(bruker_en); saveDummyBestillingMal(bruker_to); @@ -136,7 +121,7 @@ void shouldCreateAndGetMaler() void shouldCreateMalerFromExistingOrder() throws Exception { - var bruker_en = brukerRepository.findBrukerByBrukerId(DUMMY_EN.getBrukerId()).get(); + var bruker_en = brukerRepository.findBrukerByBrukerId(DUMMY_EN.getBrukerId()).orElseThrow(); var testgruppe = saveDummyGruppe(); var bestilling = saveDummyBestilling(bruker_en, testgruppe); @@ -157,7 +142,7 @@ void shouldCreateMalerFromExistingOrder() void shouldCreateUpdateAndDeleteMal() throws Exception { - var bruker_en = brukerRepository.findBrukerByBrukerId(DUMMY_EN.getBrukerId()).get(); + var bruker_en = brukerRepository.findBrukerByBrukerId(DUMMY_EN.getBrukerId()).orElseThrow(); var bestillingMal = saveDummyBestillingMal(bruker_en); mockMvc.perform(put("/api/v1/bestilling/malbestilling/{id}", bestillingMal.getId()) @@ -193,7 +178,6 @@ Bestilling saveDummyBestilling(Bruker bruker, Testgruppe testgruppe) { return bestillingRepository.save( Bestilling .builder() - .id(1L) .gruppe(testgruppe) .ferdig(false) .antallIdenter(1) @@ -215,7 +199,6 @@ Testgruppe saveDummyGruppe() { .navn(TESTGRUPPE) .hensikt(TESTGRUPPE) .datoEndret(LocalDate.now()) - .id(1L) .build() ); } diff --git a/apps/dolly-backend/src/test/java/no/nav/dolly/service/OrganisasjonBestillingMalServiceTest.java b/apps/dolly-backend/src/test/java/no/nav/dolly/service/OrganisasjonBestillingMalServiceTest.java index 9926499ec5e..ae9a697efad 100644 --- a/apps/dolly-backend/src/test/java/no/nav/dolly/service/OrganisasjonBestillingMalServiceTest.java +++ b/apps/dolly-backend/src/test/java/no/nav/dolly/service/OrganisasjonBestillingMalServiceTest.java @@ -1,6 +1,5 @@ package no.nav.dolly.service; -import com.fasterxml.jackson.databind.ObjectMapper; import no.nav.dolly.MockedJwtAuthenticationTokenUtils; import no.nav.dolly.domain.jpa.Bruker; import no.nav.dolly.domain.jpa.OrganisasjonBestilling; @@ -25,14 +24,9 @@ import java.time.LocalDateTime; -import static org.hamcrest.Matchers.empty; -import static org.hamcrest.Matchers.hasSize; -import static org.hamcrest.Matchers.is; +import static org.hamcrest.Matchers.*; import static org.springframework.boot.test.context.SpringBootTest.WebEnvironment.RANDOM_PORT; -import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.delete; -import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get; -import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.post; -import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.put; +import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.*; import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.jsonPath; import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status; @@ -42,20 +36,18 @@ @EnableAutoConfiguration @ComponentScan("no.nav.dolly") @AutoConfigureMockMvc(addFilters = false) -public class OrganisasjonBestillingMalServiceTest { +class OrganisasjonBestillingMalServiceTest { private final static String MALNAVN = "test"; private final static String NYTT_MALNAVN = "nyttMalnavn"; private final static String BEST_KRITERIER = "Testeteste"; private static final Bruker DUMMY_EN = Bruker.builder() - .id(1L) .brukernavn("test_en") .brukerId("testbruker_en") .brukertype(Bruker.Brukertype.AZURE) .epost("epost@test_en") .build(); private static final Bruker DUMMY_TO = Bruker.builder() - .id(2L) .brukernavn("test_to") .brukerId("testbruker_to") .brukertype(Bruker.Brukertype.AZURE) @@ -73,8 +65,6 @@ public class OrganisasjonBestillingMalServiceTest { @Autowired private BrukerRepository brukerRepository; @Autowired - private ObjectMapper objectMapper; - @Autowired private Flyway flyway; @BeforeEach @@ -99,8 +89,8 @@ public void afterEach() { void shouldCreateAndGetMaler() throws Exception { - var bruker_en = brukerRepository.findBrukerByBrukerId(DUMMY_EN.getBrukerId()).get(); - var bruker_to = brukerRepository.findBrukerByBrukerId(DUMMY_TO.getBrukerId()).get(); + var bruker_en = brukerRepository.findBrukerByBrukerId(DUMMY_EN.getBrukerId()).orElseThrow(); + var bruker_to = brukerRepository.findBrukerByBrukerId(DUMMY_TO.getBrukerId()).orElseThrow(); saveDummyBestillingMal(bruker_en); saveDummyBestillingMal(bruker_to); @@ -121,7 +111,7 @@ void shouldCreateAndGetMaler() void shouldCreateMalerFromExistingOrder() throws Exception { - var bruker_en = brukerRepository.findBrukerByBrukerId(DUMMY_EN.getBrukerId()).get(); + var bruker_en = brukerRepository.findBrukerByBrukerId(DUMMY_EN.getBrukerId()).orElseThrow(); var bestilling = saveDummyBestilling(bruker_en); mockMvc.perform(post("/api/v1/organisasjon/bestilling/malbestilling") @@ -141,7 +131,7 @@ void shouldCreateMalerFromExistingOrder() void shouldCreateUpdateAndDeleteMal() throws Exception { - var bruker_to = brukerRepository.findBrukerByBrukerId(DUMMY_TO.getBrukerId()).get(); + var bruker_to = brukerRepository.findBrukerByBrukerId(DUMMY_TO.getBrukerId()).orElseThrow(); var bestillingMal = saveDummyBestillingMal(bruker_to); mockMvc.perform(put("/api/v1/organisasjon/bestilling/malbestilling/{id}", bestillingMal.getId())