Skip to content

Commit 538a139

Browse files
committed
Fjernet alt tilhørende featuretoggle ettersom man kan bruke uendret versjon av UnleashService i framtiden og at det ikke er noen toggles i bruk lenger
1 parent 12675b1 commit 538a139

File tree

8 files changed

+0
-66
lines changed

8 files changed

+0
-66
lines changed

pom.xml

-5
Original file line numberDiff line numberDiff line change
@@ -139,11 +139,6 @@
139139
<version>${confluent.version}</version>
140140
</dependency>
141141

142-
<dependency>
143-
<groupId>no.nav.familie.felles</groupId>
144-
<artifactId>unleash</artifactId>
145-
<version>${felles.version}</version>
146-
</dependency>
147142

148143
<!-- Nav-internt -->
149144

src/main/kotlin/no/nav/familie/ef/mottak/featuretoggle/FeatureToggleService.kt

-17
This file was deleted.

src/main/kotlin/no/nav/familie/ef/mottak/featuretoggle/UnleashNextService.kt

-14
This file was deleted.

src/main/kotlin/no/nav/familie/ef/mottak/hendelse/JournalhendelseKafkaListener.kt

-9
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
package no.nav.familie.ef.mottak.hendelse
22

3-
import no.nav.familie.ef.mottak.featuretoggle.FeatureToggleService
4-
import no.nav.familie.ef.mottak.featuretoggle.Toggle
53
import no.nav.joarkjournalfoeringhendelser.JournalfoeringHendelseRecord
64
import org.apache.kafka.clients.consumer.ConsumerRecord
75
import org.slf4j.Logger
@@ -13,10 +11,8 @@ import org.springframework.stereotype.Service
1311
@Service
1412
class JournalhendelseKafkaListener(
1513
val kafkaHåndterer: JournalhendelseKafkaHåndterer,
16-
val featureToggleService: FeatureToggleService,
1714
) {
1815
val secureLogger: Logger = LoggerFactory.getLogger("secureLogger")
19-
val logger: Logger = LoggerFactory.getLogger(JournalhendelseKafkaListener::class.java)
2016

2117
@KafkaListener(
2218
id = "familie-ef-mottak",
@@ -30,10 +26,5 @@ class JournalhendelseKafkaListener(
3026
ack: Acknowledgment,
3127
) {
3228
kafkaHåndterer.håndterHendelse(consumerRecord, ack)
33-
if (featureToggleService.isEnabled(Toggle.ENDRE_TEST)) {
34-
logger.info("Feature toggle test")
35-
} else {
36-
logger.info("Feature toggle disabled")
37-
}
3829
}
3930
}

src/test/kotlin/no/nav/familie/ef/mottak/hendelse/JournalhendelseServiceTest.kt

-6
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ import io.mockk.every
55
import io.mockk.impl.annotations.MockK
66
import io.mockk.slot
77
import io.mockk.verify
8-
import no.nav.familie.ef.mottak.featuretoggle.FeatureToggleService
98
import no.nav.familie.ef.mottak.integration.IntegrasjonerClient
109
import no.nav.familie.ef.mottak.mockapi.clearAllMocks
1110
import no.nav.familie.ef.mottak.no.nav.familie.ef.mottak.util.JournalføringHendelseRecordVars.JOURNALPOST_DIGITALSØKNAD
@@ -44,9 +43,6 @@ class JournalhendelseServiceTest {
4443
@MockK
4544
lateinit var mockTaskService: TaskService
4645

47-
@MockK(relaxed = true)
48-
lateinit var mockFeatureToggleService: FeatureToggleService
49-
5046
@MockK(relaxed = true)
5147
lateinit var mockHendelseloggRepository: HendelsesloggRepository
5248

@@ -142,8 +138,6 @@ class JournalhendelseServiceTest {
142138
kanal = "NAV_NO",
143139
)
144140

145-
every { mockFeatureToggleService.isEnabled(any()) } returns true
146-
147141
mockJournalfoeringHendelseDbUtil =
148142
JournalfoeringHendelseDbUtil(mockHendelseloggRepository, mockTaskService, mockTaskRepositoryUtvidet)
149143

src/test/kotlin/no/nav/familie/ef/mottak/service/AutomatiskJournalføringServiceTest.kt

-8
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ package no.nav.familie.ef.mottak.service
33
import io.mockk.every
44
import io.mockk.mockk
55
import io.mockk.slot
6-
import no.nav.familie.ef.mottak.featuretoggle.FeatureToggleService
76
import no.nav.familie.ef.mottak.integration.SaksbehandlingClient
87
import no.nav.familie.ef.mottak.mapper.SøknadMapper
98
import no.nav.familie.kontrakter.ef.journalføring.AutomatiskJournalføringRequest
@@ -13,7 +12,6 @@ import no.nav.familie.kontrakter.ef.søknad.Søknadsfelt
1312
import no.nav.familie.kontrakter.felles.ef.StønadType
1413
import no.nav.familie.kontrakter.felles.oppgave.OppgavePrioritet
1514
import org.assertj.core.api.Assertions
16-
import org.junit.jupiter.api.BeforeEach
1715
import org.junit.jupiter.api.Nested
1816
import org.junit.jupiter.api.Test
1917
import java.time.LocalDate
@@ -25,7 +23,6 @@ import kotlin.test.assertTrue
2523

2624
internal class AutomatiskJournalføringServiceTest {
2725
private val saksbehandlingClient = mockk<SaksbehandlingClient>()
28-
private val featureToggleService = mockk<FeatureToggleService>()
2926
private val søknad = SøknadMapper.fromDto(Testdata.søknadOvergangsstønad, true)
3027

3128
private val automatiskJournalføringService =
@@ -43,11 +40,6 @@ internal class AutomatiskJournalføringServiceTest {
4340

4441
private val mappeId = 1L
4542

46-
@BeforeEach
47-
internal fun setUp() {
48-
every { featureToggleService.isEnabled(any()) } returns true
49-
}
50-
5143
@Test
5244
internal fun `Skal returnere true når journalføring i ef-sak går bra `() {
5345
every {

src/test/kotlin/no/nav/familie/ef/mottak/service/OppgaveServiceTest.kt

-3
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ import io.mockk.verify
99
import no.nav.familie.ef.mottak.config.DOKUMENTTYPE_OVERGANGSSTØNAD
1010
import no.nav.familie.ef.mottak.config.DOKUMENTTYPE_SKJEMA_ARBEIDSSØKER
1111
import no.nav.familie.ef.mottak.encryption.EncryptedString
12-
import no.nav.familie.ef.mottak.featuretoggle.FeatureToggleService
1312
import no.nav.familie.ef.mottak.integration.IntegrasjonerClient
1413
import no.nav.familie.ef.mottak.mapper.BehandlesAvApplikasjon
1514
import no.nav.familie.ef.mottak.mapper.OpprettOppgaveMapper
@@ -66,7 +65,6 @@ internal class OppgaveServiceTest {
6665
private val søknadskvitteringService: SøknadskvitteringService = mockk()
6766
private val opprettOppgaveMapper = spyk(OpprettOppgaveMapper())
6867
private val ettersendingService = mockk<EttersendingService>()
69-
private val featureToggleService = mockk<FeatureToggleService>()
7068
private val cacheManager = ConcurrentMapCacheManager()
7169

7270
private val oppgaveService: OppgaveService =
@@ -107,7 +105,6 @@ internal class OppgaveServiceTest {
107105
),
108106
)
109107
every { integrasjonerClient.oppdaterOppgave(any(), any()) } returns 123
110-
every { featureToggleService.isEnabled(any()) } returns true
111108
}
112109

113110
@Test

src/test/kotlin/no/nav/familie/ef/mottak/task/SendPåminnelseOmDokumentasjonsbehovTilDittNavTaskTest.kt

-4
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ import io.mockk.mockk
66
import io.mockk.verify
77
import no.nav.familie.ef.mottak.config.EttersendingConfig
88
import no.nav.familie.ef.mottak.encryption.EncryptedString
9-
import no.nav.familie.ef.mottak.featuretoggle.FeatureToggleService
109
import no.nav.familie.ef.mottak.integration.SaksbehandlingClient
1110
import no.nav.familie.ef.mottak.repository.domain.Ettersending
1211
import no.nav.familie.ef.mottak.repository.domain.Søknad
@@ -31,7 +30,6 @@ internal class SendPåminnelseOmDokumentasjonsbehovTilDittNavTaskTest {
3130
private lateinit var dittNavKafkaProducer: DittNavKafkaProducer
3231
private lateinit var søknadskvitteringService: SøknadskvitteringService
3332
private lateinit var ettersendingService: EttersendingService
34-
private lateinit var featureToggleService: FeatureToggleService
3533
private lateinit var ettersendingConfig: EttersendingConfig
3634
private lateinit var saksbehandlingClient: SaksbehandlingClient
3735

@@ -42,7 +40,6 @@ internal class SendPåminnelseOmDokumentasjonsbehovTilDittNavTaskTest {
4240
dittNavKafkaProducer = mockk(relaxed = true)
4341
søknadskvitteringService = mockk()
4442
ettersendingService = mockk()
45-
featureToggleService = mockk()
4643
ettersendingConfig = mockk()
4744
saksbehandlingClient = mockk()
4845
sendPåminnelseOmDokumentasjonsbehovTilDittNavTask =
@@ -54,7 +51,6 @@ internal class SendPåminnelseOmDokumentasjonsbehovTilDittNavTaskTest {
5451
saksbehandlingClient,
5552
)
5653

57-
every { featureToggleService.isEnabled(any()) } returns true
5854
every { ettersendingConfig.ettersendingUrl } returns URL("https://dummy-url.nav.no")
5955
}
6056

0 commit comments

Comments
 (0)