Revert "Legg til aktivitet-query med historikk i graphql"

This reverts commit 9b76cd2.

src/main/java/no/nav/veilarbaktivitet/aktivitet/dto/AktivitetDTO.java

@@ -5,7 +5,6 @@
 import lombok.Data;
 import lombok.NoArgsConstructor;
 import lombok.experimental.Accessors;
-import no.nav.veilarbaktivitet.aktivitet.Historikk;
 import no.nav.veilarbaktivitet.aktivitet.domain.AktivitetStatus;
 import no.nav.veilarbaktivitet.aktivitet.domain.AktivitetTransaksjonsType;
 import no.nav.veilarbaktivitet.avtalt_med_nav.ForhaandsorienteringDTO;
@@ -81,6 +80,4 @@ public class AktivitetDTO {
     private StillingFraNavData stillingFraNavData;
 
     private EksternAktivitetDTO eksternAktivitet;
-
-    private Historikk historikk;
 }

src/main/kotlin/no/nav/veilarbaktivitet/aktivitetskort/graphql/AktivitetskortController.kt

@@ -4,18 +4,13 @@ import no.nav.common.types.identer.EnhetId
 import no.nav.common.types.identer.Fnr
 import no.nav.poao.dab.spring_auth.IAuthService
 import no.nav.veilarbaktivitet.aktivitet.AktivitetAppService
-import no.nav.veilarbaktivitet.aktivitet.AktivitetService
-import no.nav.veilarbaktivitet.aktivitet.Historikk
-import no.nav.veilarbaktivitet.aktivitet.HistorikkService
 import no.nav.veilarbaktivitet.aktivitet.domain.AktivitetData
 import no.nav.veilarbaktivitet.aktivitet.dto.AktivitetDTO
 import no.nav.veilarbaktivitet.aktivitet.mappers.AktivitetDTOMapper
 import no.nav.veilarbaktivitet.aktivitetskort.MigreringService
-import no.nav.veilarbaktivitet.config.ownerProviders.AktivitetOwnerProvider
 import no.nav.veilarbaktivitet.person.Person
 import org.springframework.graphql.data.method.annotation.Argument
 import org.springframework.graphql.data.method.annotation.QueryMapping
-import org.springframework.graphql.data.method.annotation.SchemaMapping
 import org.springframework.http.HttpStatus
 import org.springframework.stereotype.Controller
 import org.springframework.web.server.ResponseStatusException
@@ -25,10 +20,6 @@ import org.springframework.web.server.ResponseStatusException
 class AktivitetskortController(
     val appService: AktivitetAppService,
     val migreringService: MigreringService,
-    val historikkService: HistorikkService,
-    val ownerProvider: AktivitetOwnerProvider,
-    val aktivitetService: AktivitetService,
-    val aktivitetAppService: AktivitetAppService,
     val authService: IAuthService
 ) {
 
@@ -53,22 +44,6 @@ class AktivitetskortController(
         return migreringService.visMigrerteArenaAktiviteterHvisToggleAktiv(aktiviter)
     }
 
-    @QueryMapping
-    fun aktivitet(@Argument aktivitetId: Long): AktivitetDTO {
-        val erEksternBruker = authService.erEksternBruker()
-        val eksternBrukerId = ownerProvider.getOwner(aktivitetId.toString())
-            ?: throw ResponseStatusException(HttpStatus.BAD_REQUEST, "No owner found for aktivitetId")
-        authService.sjekkTilgangTilPerson(eksternBrukerId)
-        return aktivitetAppService.hentAktivitet(aktivitetId)
-            .let { AktivitetDTOMapper.mapTilAktivitetDTO(it, erEksternBruker) }
-    }
-
-    @SchemaMapping(typeName="AktivitetDTO", field="historikk")
-    fun getHistorikk(aktivitet: AktivitetDTO): Historikk {
-        val aktivitetId = aktivitet.id.toLong()
-        return historikkService.hentHistorikk(listOf(aktivitetId))[aktivitetId]!!
-    }
-
     private fun getContextUserIdent(fnr: String): Person {
         return when {
             authService.erEksternBruker() -> Person.fnr(authService.getLoggedInnUser().get())

src/main/kotlin/no/nav/veilarbaktivitet/aktivitetskort/graphql/DateScalar.kt

@@ -6,7 +6,6 @@ import graphql.schema.CoercingParseValueException
 import graphql.schema.GraphQLScalarType
 import no.nav.veilarbaktivitet.util.DateUtils
 import java.time.ZoneOffset
-import java.time.ZonedDateTime
 import java.util.*
 
 object DateScalar {
@@ -40,7 +39,6 @@ object DateScalar {
     fun serializeDate(dataFetcherResult: Any): String {
         return when (dataFetcherResult) {
             is Date -> DateUtils.iso8601Fromdate(dataFetcherResult, ZoneOffset.systemDefault())
-            is ZonedDateTime -> dataFetcherResult.toString()
             else -> throw CoercingParseValueException("Failed to parse input in serializeDate")
         }
     }

src/main/kotlin/no/nav/veilarbaktivitet/aktivitetskort/graphql/QueryAktivitetsPerioder.kt

@@ -20,8 +20,7 @@ class Location(val line: String, val column: Int)
 class Extension(val classification: String)
 
 class QueryAktivitetsPerioder(
-    val perioder: List<OppfolgingsPeriode>? = emptyList(),
-    val aktivitet: AktivitetDTO? = null
+    val perioder: List<OppfolgingsPeriode>? = emptyList()
 )
 
 class GraphqlResult(

src/main/resources/graphql/schema.graphqls

@@ -1,25 +1,9 @@
 type Query {
     perioder(fnr: String!): [OppfolgingsPeriode]
-    aktivitet(aktivitetId: String): AktivitetDTO
-}
-
-type Historikk {
-    endringer: [Endring]
-}
-
-type Endring {
-    endretAvType: Innsender
-    endretAv: String
-    tidspunkt: Date
-    beskrivelseForVeileder: String!
-    beskrivelseForBruker: String!
-    beskrivelseForArkiv: String!
 }
 
 type OppfolgingsPeriode {
     id: String
-    start: Date
-    slutt: Date
     aktiviteter: [AktivitetDTO]
 }
 
@@ -86,8 +70,6 @@ type AktivitetDTO {
     stillingFraNavData: StillingFraNavData
 
     eksternAktivitet: EksternAktivitetDTO
-
-    historikk: Historikk
 }
 
 type EksternAktivitetDTO {

src/test/java/no/nav/veilarbaktivitet/util/AktivitetTestService.java

@@ -106,20 +106,6 @@ public GraphqlResult queryAktivitetskort(MockBruker mockBruker, RestassuredUser
         return response.as(GraphqlResult.class);
     }
 
-    public GraphqlResult queryHistorikk(MockBruker mockBruker, RestassuredUser user, String query, Long aktivitetId) {
-        var validatableResponse = user
-                .createRequest()
-                .body("{ \"query\": \""+ query  +"\", \"variables\": { \"aktivitetId\": " + aktivitetId + " } }")
-                .post("http://localhost:" + port + "/veilarbaktivitet/graphql")
-                .then();
-        var response = validatableResponse
-                .assertThat()
-                .statusCode(HttpStatus.OK.value())
-                .extract()
-                .response();
-        return response.as(GraphqlResult.class);
-    }
-
     public List<AktivitetDTO> hentVersjoner(String aktivitetId, MockBruker mockBruker, RestassuredUser user) {
         Response response = user
                 .createRequest()

src/test/kotlin/no/nav/veilarbaktivitet/aktivitetskort/graphql/AktivitetskortControllerTest.kt

@@ -1,10 +1,7 @@
 package no.nav.veilarbaktivitet.aktivitetskort.graphql
 
 import no.nav.veilarbaktivitet.SpringBootTestBase
-<<<<<<< Updated upstream
 import no.nav.veilarbaktivitet.aktivitet.AktivitetsplanController
-=======
->>>>>>> Stashed changes
 import no.nav.veilarbaktivitet.aktivitet.domain.AktivitetStatus
 import no.nav.veilarbaktivitet.aktivitet.dto.AktivitetTypeDTO
 import no.nav.veilarbaktivitet.arkivering.ArkiveringsController
@@ -143,41 +140,9 @@ class AktivitetskortControllerTest: SpringBootTestBase() {
             .put("http://localhost:$port/veilarbaktivitet/api/aktivitet/${opprettetAktivitet.id}/status")
 
         assertThat(result.statusCode).isEqualTo(400)
-        assertThat(result.body.asString()).isEqualTo(
-            """
+        assertThat(result.body.asString()).isEqualTo("""
             {"message":"Kan ikke endre aktivitet i en ferdig status","statusCode":400}
-        """.trimIndent()
-        )
-    }
-
-    @Test
-    fun `skal kunne hente historikk`() {
-        val nyPeriode = UUID.randomUUID()
-        val jobbAktivitet = AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.IJOBB)
-            .toBuilder().oppfolgingsperiodeId(nyPeriode).build()
-        val aktivitet = aktivitetTestService.opprettAktivitet(mockBruker, mockBruker, jobbAktivitet)
-        aktivitetTestService.oppdaterAktivitetStatus(mockBruker, mockVeileder, aktivitet, AktivitetStatus.GJENNOMFORES)
-        val aktivitetIdParam = "\$aktivitetId"
-        val query = """
-            query($aktivitetIdParam: String) {
-                aktivitet(aktivitetId: $aktivitetIdParam) {
-                    historikk {
-                        endringer {
-                            endretAvType,
-                            endretAv,
-                            tidspunkt,
-                            beskrivelseForVeileder,
-                            beskrivelseForBruker,
-                            beskrivelseForArkiv,
-                        }
-                    }
-                }
-            }
-        """.trimIndent().replace("\n", "")
-        val result = aktivitetTestService.queryHistorikk(mockBruker, mockBruker, query, aktivitet.id.toLong())
-        assertThat(result.errors).isNull()
-        assertThat(result.data?.aktivitet?.historikk).isNotNull()
-        assertThat(result.data?.aktivitet?.historikk?.endringer).hasSize(2)
+        """.trimIndent())
     }
 
 }