Merge branch 'main' of github.com:navikt/veilarbregistrering

src/main/kotlin/no/nav/fo/veilarbregistrering/arbeidssoker/Arbeidssoker.kt

@@ -7,9 +7,8 @@ import java.time.LocalDateTime
 /**
  * Root aggregate - all kommunikasjon mot Arbeidssøker og underliggende elementer skal gå via dette objektet.
  */
-class Arbeidssoker : Observable {
+class Arbeidssoker {
 
-    private val observers: MutableList<Observer> = mutableListOf()
     private var tilstand: ArbeidssokerState = IkkeArbeidssokerState
     private var arbeidssokerperioder: MutableList<Arbeidssokerperiode> = mutableListOf()
 
@@ -25,15 +24,11 @@ class Arbeidssoker : Observable {
     internal fun startPeriode(fraDato: LocalDateTime) {
         this.arbeidssokerperioder.add(Arbeidssokerperiode(fraDato, null))
         this.tilstand = AktivArbeidssokerState
-        //publish event
-        observers.forEach { it.update("ArbeidssokerperiodeStartetEvent") }
     }
 
     internal fun avsluttPeriode(tilDato: LocalDateTime) {
         sistePeriode()?.avslutt(tilDato) ?: throw IllegalStateException("Kan ikke avslutte en periode som ikke finnes")
         this.tilstand = TidligereArbeidssokerState
-        //publish event
-        observers.forEach { it.update("ArbeidssokerperiodeAvsluttetEvent") }
     }
 
     internal fun harVærtInaktivMerEnn28Dager() = sistePeriode()!!.tilDato!!.isBefore(LocalDateTime.now().minusDays(28))
@@ -48,14 +43,6 @@ class Arbeidssoker : Observable {
     internal fun allePerioder(): List<Arbeidssokerperiode> {
         return arbeidssokerperioder
     }
-
-    override fun add(observer: Observer) {
-        observers.add(observer)
-    }
-
-    override fun remove(observer: Observer) {
-        observers.remove(observer)
-    }
 }
 
 private interface ArbeidssokerState {

src/main/kotlin/no/nav/fo/veilarbregistrering/arbeidssoker/perioder/ArbeidssokerService.kt

@@ -1,9 +1,9 @@
 package no.nav.fo.veilarbregistrering.arbeidssoker.perioder
 
 import no.nav.common.featuretoggle.UnleashClient
+import no.nav.fo.veilarbregistrering.arbeidssoker.Arbeidssoker
 import no.nav.fo.veilarbregistrering.arbeidssoker.formidlingsgruppe.FormidlingsgruppeGateway
 import no.nav.fo.veilarbregistrering.arbeidssoker.formidlingsgruppe.FormidlingsgruppeRepository
-import no.nav.fo.veilarbregistrering.arbeidssoker.Arbeidssoker
 import no.nav.fo.veilarbregistrering.bruker.Bruker
 import no.nav.fo.veilarbregistrering.bruker.Periode
 import no.nav.fo.veilarbregistrering.log.logger
@@ -36,7 +36,11 @@ class ArbeidssokerService(
             try {
                 val arbeidssoker = populerNyArbeidssøkermodell(bruker)
                 val overlappendeArbeidssokerperioderLokalt = map(arbeidssoker).overlapperMed(forespurtPeriode)
-                sammenlignNyOgGammelModell(overlappendeArbeidssokerperioderLokalt, overlappendeHistoriskePerioderORDS, forespurtPeriode)
+                sammenlignNyOgGammelModell(
+                    overlappendeArbeidssokerperioderLokalt,
+                    overlappendeHistoriskePerioderORDS,
+                    forespurtPeriode
+                )
             } catch (e: RuntimeException) {
                 logger.warn("Sammenligning av perioder feilet", e)
             }
@@ -51,18 +55,12 @@ class ArbeidssokerService(
     }
 
     private fun populerNyArbeidssøkermodell(bruker: Bruker): Arbeidssoker {
-        val brukerRegistrering =
-            brukerRegistreringRepository.hentBrukerregistreringForFoedselsnummer(bruker.alleFoedselsnummer())
-        val brukerReaktivering =
-            brukerReaktiveringRepository.finnReaktiveringerForFoedselsnummer(bruker.alleFoedselsnummer())
         val formidlingsgruppe =
             formidlingsgruppeRepository.hentFormidlingsgrupperOgMapTilFormidlingsgruppeEndretEvent(bruker.alleFoedselsnummer())
-        //TODO: hente meldekort - vi avventer Meldekort til vi har lært litt mer om modellen.
-
-        val listeMedArbeidssøkerEndringer = brukerRegistrering + brukerReaktivering + formidlingsgruppe
 
         val arbeidssoker = Arbeidssoker()
-        listeMedArbeidssøkerEndringer.sortedBy { it.opprettetTidspunkt() }.forEach { arbeidssoker.behandle(it) }
+
+        formidlingsgruppe.sortedBy { it.opprettetTidspunkt() }.forEach { arbeidssoker.behandle(it) }
 
         return arbeidssoker
     }

src/test/kotlin/no/nav/fo/veilarbregistrering/arbeidssoker/ArbeidssokerTest.kt

@@ -1,8 +1,5 @@
 package no.nav.fo.veilarbregistrering.arbeidssoker
 
-import no.nav.fo.veilarbregistrering.arbeidssoker.Arbeidssoker
-import no.nav.fo.veilarbregistrering.arbeidssoker.Arbeidssokerperiode
-import no.nav.fo.veilarbregistrering.arbeidssoker.Observer
 import no.nav.fo.veilarbregistrering.arbeidssoker.formidlingsgruppe.FormidlingsgruppeEndretEventTestdataBuilder.formidlingsgruppeEndret
 import no.nav.fo.veilarbregistrering.bruker.AktorId
 import no.nav.fo.veilarbregistrering.registrering.ordinaer.OrdinaerBrukerRegistreringTestdataBuilder.gyldigBrukerRegistrering
@@ -22,13 +19,9 @@ class ArbeidssokerTest {
         val registreringsdato = LocalDateTime.now().minusMonths(2)
         val nyRegistrering = gyldigBrukerRegistrering(opprettetDato = registreringsdato)
 
-        var hendelse : String? = null
-
-        arbeidssoker.add(object : Observer { override fun update(event: String) { hendelse = event } })
         arbeidssoker.behandle(nyRegistrering)
 
         assertEquals(Arbeidssokerperiode(registreringsdato, null), arbeidssoker.sistePeriode())
-        assertEquals("ArbeidssokerperiodeStartetEvent", hendelse)
     }
 
     @Test