fix: update code and tests

Amsterdam · Nov 7, 2023 · 83ec9a3 · 83ec9a3
1 parent 161c782
commit 83ec9a3
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 58 deletions.
diff --git a/src/client/pages/VergunningDetail/RvvSloterweg.tsx b/src/client/pages/VergunningDetail/RvvSloterweg.tsx
@@ -24,7 +24,11 @@ export function getRVVSloterwegLineItems(
       ? new Date(vergunning.dateEnd) < new Date()
       : false;
 
-  const hasDecision = !!vergunning.decision;
+  const hasDecision = [
+    RVV_SLOTERWEG_RESULT_NOT_APPLICABLE,
+    RVV_SLOTERWEG_RESULT_UPDATED_WIHT_NEW_KENTEKEN,
+    RVV_SLOTERWEG_RESULT_MATURED,
+  ].includes(vergunning.decision);
 
   let dateInProgress = vergunning.dateWorkflowActive ?? '';
 
@@ -60,10 +64,12 @@ export function getRVVSloterwegLineItems(
         ? vergunning.dateDecision
         : '',
       description:
-        'Wij hebben uw aanvraag voor een RVV ontheffing Sloterweg verleend.',
+        isGranted && !(hasDecision || isExpired)
+          ? 'Wij hebben uw aanvraag voor een RVV ontheffing Sloterweg verleend.'
+          : '',
       documents: [],
       isActive: isGranted && !(hasDecision || isExpired),
-      isChecked: isGranted,
+      isChecked: isGranted || hasDecision || isExpired,
     },
   ];
 

diff --git a/src/client/pages/VergunningDetail/__snapshots__/VergunningDetail.test.tsx.snap b/src/client/pages/VergunningDetail/__snapshots__/VergunningDetail.test.tsx.snap
@@ -4813,9 +4813,7 @@ exports[`<VergunningDetail /> > RVV Sloterweg > RVV ontheffing Sloterweg (Nieuw/
             >
               <div
                 class="_PanelContent_a12ff2"
-              >
-                Wij hebben uw aanvraag voor een RVV ontheffing Sloterweg verleend.
-              </div>
+              />
             </div>
             <div
               class="_Panel_a12ff2 _Panel--documents_a12ff2"
@@ -5462,9 +5460,7 @@ exports[`<VergunningDetail /> > RVV Sloterweg > RVV ontheffing Sloterweg (Nieuw/
             >
               <div
                 class="_PanelContent_a12ff2"
-              >
-                Wij hebben uw aanvraag voor een RVV ontheffing Sloterweg verleend.
-              </div>
+              />
             </div>
             <div
               class="_Panel_a12ff2 _Panel--documents_a12ff2"
@@ -5805,10 +5801,10 @@ exports[`<VergunningDetail /> > RVV Sloterweg > RVV ontheffing Sloterweg (Wijzig
               class="_StatusConnection_a12ff2"
             >
               <span
-                class="_ConnectLine_a12ff2"
+                class="_ConnectLine_a12ff2 _ConnectLine--checked_a12ff2"
               />
               <span
-                class="_Checkmark_a12ff2"
+                class="_Checkmark_a12ff2 _Checkmark--checked_a12ff2"
               />
             </div>
             <div
@@ -5830,9 +5826,7 @@ exports[`<VergunningDetail /> > RVV Sloterweg > RVV ontheffing Sloterweg (Wijzig
             >
               <div
                 class="_PanelContent_a12ff2"
-              >
-                Wij hebben uw aanvraag voor een RVV ontheffing Sloterweg verleend.
-              </div>
+              />
             </div>
             <div
               class="_Panel_a12ff2 _Panel--documents_a12ff2"
@@ -6177,7 +6171,7 @@ exports[`<VergunningDetail /> > RVV Sloterweg > RVV ontheffing Sloterweg (Wijzig
           </div>
         </li>
         <li
-          class="_ListItem_a12ff2"
+          class="_ListItem_a12ff2 _ListItem--highlight_a12ff2"
         >
           <div
             class="_ListItemInner_a12ff2"
@@ -6187,10 +6181,10 @@ exports[`<VergunningDetail /> > RVV Sloterweg > RVV ontheffing Sloterweg (Wijzig
               class="_StatusConnection_a12ff2"
             >
               <span
-                class="_ConnectLine_a12ff2 _ConnectLine--checked_a12ff2"
+                class="_ConnectLine_a12ff2 _ConnectLine--end-active_a12ff2 _ConnectLine--end-checked_a12ff2"
               />
               <span
-                class="_Checkmark_a12ff2 _Checkmark--checked_a12ff2"
+                class="_Checkmark_a12ff2 _Checkmark--active_a12ff2 _Checkmark--checked_a12ff2"
               />
             </div>
             <div
@@ -6221,44 +6215,6 @@ exports[`<VergunningDetail /> > RVV Sloterweg > RVV ontheffing Sloterweg (Wijzig
             />
           </div>
         </li>
-        <li
-          class="_ListItem_a12ff2 _ListItem--highlight_a12ff2"
-        >
-          <div
-            class="_ListItemInner_a12ff2"
-          >
-            <div
-              aria-label="Vinkje, processtap gereed"
-              class="_StatusConnection_a12ff2"
-            >
-              <span
-                class="_ConnectLine_a12ff2 _ConnectLine--end-active_a12ff2 _ConnectLine--end-checked_a12ff2"
-              />
-              <span
-                class="_Checkmark_a12ff2 _Checkmark--active_a12ff2 _Checkmark--checked_a12ff2"
-              />
-            </div>
-            <div
-              class="_Panel_a12ff2 _Panel--status_a12ff2"
-            >
-              <strong
-                class="_StatusTitle_a12ff2"
-              >
-                Gewijzigd
-              </strong>
-            </div>
-            <div
-              class="_Panel_a12ff2 _Panel--description_a12ff2"
-            >
-              <div
-                class="_PanelContent_a12ff2"
-              />
-            </div>
-            <div
-              class="_Panel_a12ff2 _Panel--documents_a12ff2"
-            />
-          </div>
-        </li>
       </ul>
     </div>
   </main>
@@ -6910,9 +6866,7 @@ exports[`<VergunningDetail /> > RVV Sloterweg > RVV ontheffing Sloterweg (Wijzig
             >
               <div
                 class="_PanelContent_a12ff2"
-              >
-                Wij hebben uw aanvraag voor een RVV ontheffing Sloterweg verleend.
-              </div>
+              />
             </div>
             <div
               class="_Panel_a12ff2 _Panel--documents_a12ff2"