Skip to content

Commit

Permalink
rename fra requestbody til request
Browse files Browse the repository at this point in the history
  • Loading branch information
ramnav990 committed Nov 3, 2023
1 parent e824a7c commit 98e589a
Show file tree
Hide file tree
Showing 15 changed files with 46 additions and 47 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,7 @@ internal class OppdaterOppgaveHttpClient(
// bare beskrivelsen er påkrevd å fylles ut. Dersom oppgavetypen ikke skal oppdateres
// benytter vi oss av hva som allerede er på oppgaven
oppgavetype = data.oppgavetype ?: Oppgavetype.fromString(oppgave.oppgavetype),
requestBody = requestBody,
request = requestBody,
response = it.body(),
beskrivelse = data.beskrivelse,
).right()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -238,7 +238,7 @@ internal class OppgaveHttpClient(
OppgaveHttpKallResponse(
oppgaveId = oppgaveResponse.getOppgaveId(),
oppgavetype = oppgaveResponse.oppgavetype(),
requestBody = requestBody,
request = requestBody,
response = body,
beskrivelse = beskrivelse,
).right()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ data object OppgaveClientStub : OppgaveClient {
override fun opprettOppgave(config: OppgaveConfig): Either<OppgaveFeil.KunneIkkeOppretteOppgave, OppgaveHttpKallResponse> =
OppgaveHttpKallResponse(
oppgaveId = OppgaveId("stubbedOppgaveId"),
requestBody = "stubbedRequestBody",
request = "stubbedRequestBody",
response = "stubbedResponseBody",
beskrivelse = "stubbedBeskrivelse",
oppgavetype = Oppgavetype.BEHANDLE_SAK,
Expand All @@ -31,7 +31,7 @@ data object OppgaveClientStub : OppgaveClient {
override fun opprettOppgaveMedSystembruker(config: OppgaveConfig): Either<OppgaveFeil.KunneIkkeOppretteOppgave, OppgaveHttpKallResponse> =
OppgaveHttpKallResponse(
oppgaveId = OppgaveId("stubbedOppgaveId"),
requestBody = "stubbedRequestBody",
request = "stubbedRequestBody",
response = "stubbedResponseBody",
beskrivelse = "stubbedBeskrivelse",
oppgavetype = Oppgavetype.BEHANDLE_SAK,
Expand All @@ -40,7 +40,7 @@ data object OppgaveClientStub : OppgaveClient {
override fun lukkOppgave(oppgaveId: OppgaveId): Either<KunneIkkeLukkeOppgave, OppgaveHttpKallResponse> =
OppgaveHttpKallResponse(
oppgaveId = OppgaveId("stubbedOppgaveId"),
requestBody = "stubbedRequest",
request = "stubbedRequest",
response = "stubbedResponse",
beskrivelse = "stubbedBeskrivelse",
oppgavetype = Oppgavetype.BEHANDLE_SAK,
Expand All @@ -49,7 +49,7 @@ data object OppgaveClientStub : OppgaveClient {
override fun lukkOppgaveMedSystembruker(oppgaveId: OppgaveId): Either<KunneIkkeLukkeOppgave, OppgaveHttpKallResponse> =
OppgaveHttpKallResponse(
oppgaveId = OppgaveId("stubbedOppgaveId"),
requestBody = "stubbedRequest",
request = "stubbedRequest",
response = "stubbedResponse",
beskrivelse = "stubbedBeskrivelse",
oppgavetype = Oppgavetype.BEHANDLE_SAK,
Expand All @@ -60,7 +60,7 @@ data object OppgaveClientStub : OppgaveClient {
beskrivelse: String,
): Either<KunneIkkeOppdatereOppgave, OppgaveHttpKallResponse> = OppgaveHttpKallResponse(
oppgaveId = OppgaveId("stubbedOppgaveId"),
requestBody = "stubbedRequest",
request = "stubbedRequest",
response = "stubbedResponse",
beskrivelse = "stubbedBeskrivelse",
oppgavetype = Oppgavetype.BEHANDLE_SAK,
Expand All @@ -71,7 +71,7 @@ data object OppgaveClientStub : OppgaveClient {
oppdatertOppgaveInfo: OppdaterOppgaveInfo,
): Either<KunneIkkeOppdatereOppgave, OppgaveHttpKallResponse> = OppgaveHttpKallResponse(
oppgaveId = OppgaveId("stubbedOppgaveId"),
requestBody = "stubbedRequest",
request = "stubbedRequest",
response = "stubbedResponse",
beskrivelse = "stubbedBeskrivelse",
oppgavetype = Oppgavetype.BEHANDLE_SAK,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ internal class OppdaterHttpClientTest : WiremockBase {
val expected = nyOppgaveHttpKallResponse(
oppgaveId = OppgaveId(oppgaveId.toString()),
oppgavetype = Oppgavetype.BEHANDLE_SAK,
requestBody = expectedBody,
request = expectedBody,
response = patchResponse,
beskrivelse = "Lukket av SU-app (Supplerende Stønad)",
)
Expand All @@ -84,7 +84,7 @@ internal class OppdaterHttpClientTest : WiremockBase {
actual.oppgavetype shouldBe expected.oppgavetype
actual.beskrivelse shouldBe expected.beskrivelse
actual.response shouldBe expected.response
JSONAssert.assertEquals(actual.requestBody, expected.requestBody, true)
JSONAssert.assertEquals(actual.request, expected.request, true)

WireMock.configureFor(wireMockServer.port())
WireMock.verify(1, patchRequested.withRequestBody(equalToJson(expectedBody)))
Expand Down Expand Up @@ -114,7 +114,7 @@ internal class OppdaterHttpClientTest : WiremockBase {
val expected = nyOppgaveHttpKallResponse(
oppgaveId = OppgaveId(oppgaveId.toString()),
oppgavetype = Oppgavetype.BEHANDLE_SAK,
requestBody = expectedBody,
request = expectedBody,
response = patchResponse,
beskrivelse = "Lukket av SU-app (Supplerende Stønad)",
)
Expand All @@ -123,7 +123,7 @@ internal class OppdaterHttpClientTest : WiremockBase {
actual.oppgavetype shouldBe expected.oppgavetype
actual.beskrivelse shouldBe expected.beskrivelse
actual.response shouldBe expected.response
JSONAssert.assertEquals(actual.requestBody, expected.requestBody, true)
JSONAssert.assertEquals(actual.request, expected.request, true)

WireMock.configureFor(wireMockServer.port())
WireMock.verify(1, patchRequested.withRequestBody(equalToJson(expectedBody)))
Expand Down Expand Up @@ -160,7 +160,7 @@ internal class OppdaterHttpClientTest : WiremockBase {
val expected = nyOppgaveHttpKallResponse(
oppgaveId = OppgaveId(oppgaveId.toString()),
oppgavetype = Oppgavetype.BEHANDLE_SAK,
requestBody = expectedBody,
request = expectedBody,
response = patchResponse,
beskrivelse = "Lukket av SU-app (Supplerende Stønad)",
)
Expand All @@ -169,7 +169,7 @@ internal class OppdaterHttpClientTest : WiremockBase {
actual.oppgavetype shouldBe expected.oppgavetype
actual.beskrivelse shouldBe expected.beskrivelse
actual.response shouldBe expected.response
JSONAssert.assertEquals(actual.requestBody, expected.requestBody, true)
JSONAssert.assertEquals(actual.request, expected.request, true)

WireMock.configureFor(wireMockServer.port())
WireMock.verify(1, patchRequested.withRequestBody(equalToJson(expectedBody)))
Expand Down Expand Up @@ -213,7 +213,7 @@ internal class OppdaterHttpClientTest : WiremockBase {
val expected = nyOppgaveHttpKallResponse(
oppgaveId = OppgaveId(oppgaveId.toString()),
oppgavetype = Oppgavetype.BEHANDLE_SAK,
requestBody = expectedBody,
request = expectedBody,
response = patchResponse,
beskrivelse = "en beskrivelse",
)
Expand All @@ -222,7 +222,7 @@ internal class OppdaterHttpClientTest : WiremockBase {
actual.oppgavetype shouldBe expected.oppgavetype
actual.beskrivelse shouldBe expected.beskrivelse
actual.response shouldBe expected.response
JSONAssert.assertEquals(actual.requestBody, expected.requestBody, true)
JSONAssert.assertEquals(actual.request, expected.request, true)

WireMock.configureFor(wireMockServer.port())
WireMock.verify(1, patchRequested.withRequestBody(equalToJson(expectedBody)))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ internal class OppgaveHttpClientTest : WiremockBase {
val expected = nyOppgaveHttpKallResponse(
oppgaveId = oppgaveId,
oppgavetype = Oppgavetype.BEHANDLE_SAK,
requestBody = expectedSaksbehandlingRequest,
request = expectedSaksbehandlingRequest,
response = response,
beskrivelse = "--- 01.01.2021 02:02 - Opprettet av Supplerende Stønad ---\nSøknadId : $søknadId",
)
Expand All @@ -94,7 +94,7 @@ internal class OppgaveHttpClientTest : WiremockBase {
actual.oppgavetype shouldBe expected.oppgavetype
actual.beskrivelse shouldBe expected.beskrivelse
actual.response shouldBe expected.response
JSONAssert.assertEquals(actual.requestBody, expected.requestBody, true)
JSONAssert.assertEquals(actual.request, expected.request, true)

verify(oathMock).onBehalfOfToken(
originalToken = argThat { it shouldBe "Bearer token" },
Expand Down Expand Up @@ -128,7 +128,7 @@ internal class OppgaveHttpClientTest : WiremockBase {
val expected = nyOppgaveHttpKallResponse(
oppgaveId = oppgaveId,
oppgavetype = Oppgavetype.BEHANDLE_SAK,
requestBody = expectedSaksbehandlingRequest,
request = expectedSaksbehandlingRequest,
response = response,
beskrivelse = "--- 01.01.2021 02:02 - Opprettet av Supplerende Stønad ---\nSøknadId : $søknadId",
)
Expand All @@ -148,7 +148,7 @@ internal class OppgaveHttpClientTest : WiremockBase {
actual.oppgavetype shouldBe expected.oppgavetype
actual.beskrivelse shouldBe expected.beskrivelse
actual.response shouldBe expected.response
JSONAssert.assertEquals(actual.requestBody, expected.requestBody, true)
JSONAssert.assertEquals(actual.request, expected.request, true)

verify(tokenoppslagMock).token()
verifyNoMoreInteractions(oathMock, tokenoppslagMock)
Expand Down Expand Up @@ -187,7 +187,7 @@ internal class OppgaveHttpClientTest : WiremockBase {
val expected = nyOppgaveHttpKallResponse(
oppgaveId = oppgaveId,
oppgavetype = Oppgavetype.ATTESTERING,
requestBody = expectedAttesteringRequest,
request = expectedAttesteringRequest,
response = response,
beskrivelse = "--- 01.01.2021 02:02 - Opprettet av Supplerende Stønad ---\nSøknadId : $søknadId",
)
Expand All @@ -196,7 +196,7 @@ internal class OppgaveHttpClientTest : WiremockBase {
actual.oppgavetype shouldBe expected.oppgavetype
actual.beskrivelse shouldBe expected.beskrivelse
actual.response shouldBe expected.response
JSONAssert.assertEquals(actual.requestBody, expected.requestBody, true)
JSONAssert.assertEquals(actual.request, expected.request, true)
}

@Test
Expand Down Expand Up @@ -266,7 +266,7 @@ internal class OppgaveHttpClientTest : WiremockBase {
val expected = nyOppgaveHttpKallResponse(
oppgaveId = oppgaveId,
oppgavetype = Oppgavetype.BEHANDLE_SAK,
requestBody = expectedSaksbehandlingRequest,
request = expectedSaksbehandlingRequest,
response = response,
beskrivelse = "--- 01.01.2021 02:02 - Opprettet av Supplerende Stønad ---\nSaksnummer : $saksnummer",
)
Expand All @@ -275,7 +275,7 @@ internal class OppgaveHttpClientTest : WiremockBase {
actual.oppgavetype shouldBe expected.oppgavetype
actual.beskrivelse shouldBe expected.beskrivelse
actual.response shouldBe expected.response
JSONAssert.assertEquals(actual.requestBody, expected.requestBody, true)
JSONAssert.assertEquals(actual.request, expected.request, true)

verify(oathMock).onBehalfOfToken(
originalToken = argThat { it shouldBe "Bearer token" },
Expand Down Expand Up @@ -315,7 +315,7 @@ internal class OppgaveHttpClientTest : WiremockBase {
val expected = nyOppgaveHttpKallResponse(
oppgaveId = oppgaveId,
oppgavetype = Oppgavetype.BEHANDLE_SAK,
requestBody = expectedSaksbehandlingRequest,
request = expectedSaksbehandlingRequest,
response = response,
beskrivelse = "--- 01.01.2021 02:02 - Opprettet av Supplerende Stønad ---\nSaksnummer : $saksnummer",
)
Expand All @@ -333,7 +333,7 @@ internal class OppgaveHttpClientTest : WiremockBase {
actual.oppgavetype shouldBe expected.oppgavetype
actual.beskrivelse shouldBe expected.beskrivelse
actual.response shouldBe expected.response
JSONAssert.assertEquals(actual.requestBody, expected.requestBody, true)
JSONAssert.assertEquals(actual.request, expected.request, true)

verify(tokenoppslagMock).token()
verifyNoMoreInteractions(oathMock, tokenoppslagMock)
Expand Down Expand Up @@ -383,7 +383,7 @@ internal class OppgaveHttpClientTest : WiremockBase {
val expected = nyOppgaveHttpKallResponse(
oppgaveId = oppgaveId,
oppgavetype = Oppgavetype.ATTESTERING,
requestBody = expectedAttesteringRequest,
request = expectedAttesteringRequest,
response = response,
beskrivelse = "--- 01.01.2021 02:02 - Opprettet av Supplerende Stønad ---\nSaksnummer : $saksnummer",
)
Expand All @@ -392,7 +392,7 @@ internal class OppgaveHttpClientTest : WiremockBase {
actual.oppgavetype shouldBe expected.oppgavetype
actual.beskrivelse shouldBe expected.beskrivelse
actual.response shouldBe expected.response
JSONAssert.assertEquals(actual.requestBody, expected.requestBody, true)
JSONAssert.assertEquals(actual.request, expected.request, true)
}

@Test
Expand Down Expand Up @@ -465,7 +465,7 @@ internal class OppgaveHttpClientTest : WiremockBase {
val expected = nyOppgaveHttpKallResponse(
oppgaveId = oppgaveId,
oppgavetype = Oppgavetype.ATTESTERING,
requestBody = expectedAttesteringRequest,
request = expectedAttesteringRequest,
response = response,
beskrivelse = "--- 01.01.2021 02:02 - Opprettet av Supplerende Stønad ---\nSaksnummer : $saksnummer\nUtfall: Stadfestelse\nRelevante JournalpostIDer: 123, 456\nKlageinstans sin behandling ble avsluttet den 01.01.2021 02:02\n\nDenne oppgaven er kun til opplysning og må lukkes manuelt.",
)
Expand All @@ -474,7 +474,7 @@ internal class OppgaveHttpClientTest : WiremockBase {
actual.oppgavetype shouldBe expected.oppgavetype
actual.beskrivelse shouldBe expected.beskrivelse
actual.response shouldBe expected.response
JSONAssert.assertEquals(actual.requestBody, expected.requestBody, true)
JSONAssert.assertEquals(actual.request, expected.request, true)
}

@Test
Expand Down Expand Up @@ -518,7 +518,7 @@ internal class OppgaveHttpClientTest : WiremockBase {
val expected = nyOppgaveHttpKallResponse(
oppgaveId = oppgaveId,
oppgavetype = Oppgavetype.ATTESTERING,
requestBody = expectedAttesteringRequest,
request = expectedAttesteringRequest,
response = response,
beskrivelse = "--- 01.01.2021 02:02 - Opprettet av Supplerende Stønad ---\nSaksnummer : $saksnummer\nUtfall: Retur\nRelevante JournalpostIDer: 123, 456\nKlageinstans sin behandling ble avsluttet den 01.01.2021 02:02\n\nKlagen krever ytterligere saksbehandling. Lukking av oppgaven håndteres automatisk.",
)
Expand All @@ -527,7 +527,7 @@ internal class OppgaveHttpClientTest : WiremockBase {
actual.oppgavetype shouldBe expected.oppgavetype
actual.beskrivelse shouldBe expected.beskrivelse
actual.response shouldBe expected.response
JSONAssert.assertEquals(actual.requestBody, expected.requestBody, true)
JSONAssert.assertEquals(actual.request, expected.request, true)
}

@Test
Expand Down Expand Up @@ -572,7 +572,7 @@ internal class OppgaveHttpClientTest : WiremockBase {
val expected = nyOppgaveHttpKallResponse(
oppgaveId = oppgaveId,
oppgavetype = Oppgavetype.ATTESTERING,
requestBody = expectedAttesteringRequest,
request = expectedAttesteringRequest,
response = response,
beskrivelse = "--- 01.01.2021 02:02 - Opprettet av Supplerende Stønad ---\nSaksnummer : $saksnummer\nUtfall: Medhold\nRelevante JournalpostIDer: 123, 456\nKlageinstans sin behandling ble avsluttet den 01.01.2021 02:02\n\nKlagen krever ytterligere saksbehandling. Denne oppgaven må lukkes manuelt.",
)
Expand All @@ -581,7 +581,7 @@ internal class OppgaveHttpClientTest : WiremockBase {
actual.oppgavetype shouldBe expected.oppgavetype
actual.beskrivelse shouldBe expected.beskrivelse
actual.response shouldBe expected.response
JSONAssert.assertEquals(actual.requestBody, expected.requestBody, true)
JSONAssert.assertEquals(actual.request, expected.request, true)
}

private fun createRequestBody(
Expand All @@ -592,7 +592,6 @@ internal class OppgaveHttpClientTest : WiremockBase {
oppgavetype: String = "BEH_SAK",
behandlingstype: String = "ae0034",
): String {
println("$beskrivelse, $oppgavetype")
//language=json
return """
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,7 @@ class OpprettOppgaverForInstitusjonsoppholdshendelser(
correlationId = correlationId,
ident = null,
brukerroller = listOf(),
request = oppgaveResponse.requestBody,
request = oppgaveResponse.request,
response = oppgaveResponse.response,
),
relaterteHendelser = hendelserSomManglerOppgaver,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -226,7 +226,7 @@ class OpprettOppgaverForInstitusjonsoppholdshendelserTest {
correlationId = correlationId,
ident = null,
brukerroller = listOf(),
request = "requestbody",
request = "request",
response = "response",
),
beskrivelse = "beskrivelse",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ import no.nav.su.se.bakover.common.domain.oppgave.OppgaveId
data class OppgaveHttpKallResponse(
val oppgaveId: OppgaveId,
val oppgavetype: Oppgavetype,
val requestBody: String,
val request: String,
val response: String,
val beskrivelse: String,
)
4 changes: 2 additions & 2 deletions test-common/src/main/kotlin/oppgave/OppgaveTestData.kt
Original file line number Diff line number Diff line change
Expand Up @@ -9,13 +9,13 @@ val oppgaveId = OppgaveId("123")
fun nyOppgaveHttpKallResponse(
oppgaveId: OppgaveId = no.nav.su.se.bakover.test.oppgave.oppgaveId,
oppgavetype: Oppgavetype = Oppgavetype.BEHANDLE_SAK,
requestBody: String = "requestbody",
request: String = "request",
response: String = "response",
beskrivelse: String = "beskrivelse",
): OppgaveHttpKallResponse = OppgaveHttpKallResponse(
oppgaveId = oppgaveId,
oppgavetype = oppgavetype,
requestBody = requestBody,
request = request,
response = response,
beskrivelse = beskrivelse,
)
Original file line number Diff line number Diff line change
Expand Up @@ -164,7 +164,7 @@ class LukkOppgaveForTilbakekrevingshendelserKonsument(
correlationId = correlationId,
ident = null,
brukerroller = listOf(),
request = it.requestBody,
request = it.request,
response = it.response,
),
beskrivelse = it.beskrivelse,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -161,7 +161,7 @@ class OppdaterOppgaveForTilbakekrevingshendelserKonsument(
correlationId = correlationId,
ident = null,
brukerroller = listOf(),
request = it.requestBody,
request = it.request,
response = it.response,
),
tidligereHendelseId = tidligereOppgaveHendelse.hendelseId,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ class OpprettOppgaveForTilbakekrevingshendelserKonsument(
correlationId = correlationId,
ident = null,
brukerroller = listOf(),
request = oppgaveResponse.requestBody,
request = oppgaveResponse.request,
response = oppgaveResponse.response,
),
beskrivelse = oppgaveResponse.beskrivelse,
Expand Down
Loading

0 comments on commit 98e589a

Please sign in to comment.