- Update validation error messages to include environment context in SkattekortClient.

- Modify test assertions to reflect updated validation messages in SkattekortClientTest.
- Adjust return values in Steg3 to only include 'q2' for skattekort.
- Enhance condition in MiljoeInfo to check for non-empty environments before displaying alert.

Author: stigus

apps/dolly-backend/src/main/java/no/nav/dolly/bestilling/skattekort/SkattekortClient.java

@@ -61,7 +61,13 @@ public Mono<BestillingProgress> gjenopprett(RsDollyUtvidetBestilling bestilling,
         if (isNull(bestilling.getSkattekort()) || bestilling.getSkattekort().getArbeidsgiverSkatt().isEmpty()) {
             return Mono.empty();
         } else if (!isValidateOK(bestilling)) {
-            return oppdaterStatus(progress, "Avvik: Validering feilet: Trekkode er ikke gyldig");
+            var filteredMiljoer = bestilling.getEnvironments().stream()
+                    .filter(MILJOER_SUPPORTED::contains)
+                    .collect(Collectors.toSet());
+            var miljoer = filteredMiljoer.isEmpty() ? DEFAULT_MILJOER : filteredMiljoer;
+            return oppdaterStatus(progress, miljoer.stream()
+                    .map(miljo -> "%s:Avvik: Validering feilet: Trekkode er ikke gyldig".formatted(miljo))
+                    .collect(Collectors.joining(",")));
         }
 
         var filteredMiljoer = bestilling.getEnvironments().stream()

apps/dolly-backend/src/test/java/no/nav/dolly/bestilling/skattekort/SkattekortClientTest.java

@@ -124,7 +124,7 @@ void shouldFailSkattekortMedTilleggsopplysningOK() {
                         skattekortClient.gjenopprett(bestilling, dollyPerson, new BestillingProgress(), true))
                 .assertNext(progress -> {
                     verify(transactionHelperService).persister(any(), any(), any(), anyString());
-                    assertThat(argumentCaptor.getAllValues(), hasItems(is(equalTo("Avvik: Validering feilet: Trekkode er ikke gyldig"))));
+                    assertThat(argumentCaptor.getAllValues(), hasItems(is(equalTo("q1:Avvik: Validering feilet: Trekkode er ikke gyldig"))));
                 })
                 .verifyComplete();
     }
@@ -213,7 +213,7 @@ void shouldFailSkattekortMedUgyldigTrekkodeOK() {
                         skattekortClient.gjenopprett(bestilling, dollyPerson, new BestillingProgress(), true))
                 .assertNext(progress -> {
                     verify(transactionHelperService).persister(any(), any(), any(), anyString());
-                    assertThat(argumentCaptor.getAllValues(), hasItems(is(equalTo("Avvik: Validering feilet: Trekkode er ikke gyldig"))));
+                    assertThat(argumentCaptor.getAllValues(), hasItems(is(equalTo("q2:Avvik: Validering feilet: Trekkode er ikke gyldig"))));
                 })
                 .verifyComplete();
     }

apps/dolly-frontend/src/main/js/src/components/bestillingsveileder/stegVelger/steg/steg3/Steg3.tsx

@@ -51,7 +51,7 @@ const Steg3 = ({ loadingBestilling }: { loadingBestilling: boolean }) => {
 		} else if (values.sykemelding) {
 			return ['q1']
 		} else if (values.skattekort) {
-			return ['q1', 'q2']
+			return ['q2']
 		}
 		return []
 	}

apps/dolly-frontend/src/main/js/src/components/miljoVelger/MiljoeInfo.tsx

@@ -109,7 +109,7 @@ export const MiljoeInfo = ({ bestillingsdata, dollyEnvironments, tilgjengeligeMi
 					bli sendt til Q1 <br />
 				</Alert>
 			)}
-			{skattekort && bestillingsdata?.environments?.every((env) => env === 'q4') && (
+			{skattekort && bestillingsdata?.environments?.length > 0 && bestillingsdata?.environments?.every((env) => env === 'q4') && (
 				<Alert variant={'info'} style={{ marginTop: 20 }}>
 					Innsending av skattekort er for øyeblikket ikke støttet i Q4, bestillingen vil bli sendt til Q2.
 				</Alert>