[CCAP-357] Make the confirmation code input readonly (#1225)

codeforamerica · Feb 28, 2025 · 81be535 · 81be535
1 parent 3a73d9e
commit 81be535
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 19 deletions.
diff --git a/src/main/resources/templates/providerresponse/confirmation-code.html b/src/main/resources/templates/providerresponse/confirmation-code.html
@@ -18,12 +18,13 @@
 
 <script th:inline="javascript">
   $(document).ready(function () {
-    const providerFamilyCode = $('#providerResponseFamilyShortCode')
+    const providerFamilyCode = document.getElementById("providerResponseFamilyShortCode");
 
-    if(!providerFamilyCode.text()){
+    if (!providerFamilyCode.textContent) {
       var confirmationCode = [[${session.confirmationCode}]];
-      if(confirmationCode){
-        providerFamilyCode.val(confirmationCode)
+      if (confirmationCode) {
+        providerFamilyCode.value = confirmationCode;
+        providerFamilyCode.readOnly = true;
       }
     }
   });

diff --git a/src/test/java/org/ilgcc/app/journeys/ProviderresponseProviderRegistrationJourneyTest.java b/src/test/java/org/ilgcc/app/journeys/ProviderresponseProviderRegistrationJourneyTest.java
@@ -1,7 +1,6 @@
 package org.ilgcc.app.journeys;
 
 import static org.assertj.core.api.Assertions.assertThat;
-import static org.awaitility.Awaitility.await;
 
 import formflow.library.data.SubmissionRepository;
 import java.time.OffsetDateTime;
@@ -21,7 +20,7 @@ public class ProviderresponseProviderRegistrationJourneyTest extends AbstractBas
     private static final String CONF_CODE = "A2123B";
 
     @Test
-    void BasicInfoFlow() {
+    void basicInfoFlow() {
         createAValidLink();
 
         testPage.navigateToFlowScreen("providerresponse/registration-basic-info-intro");
@@ -127,7 +126,7 @@ void BasicInfoFlow() {
     }
 
     @Test
-    void LicenseExemptInProviderHomeFlow() {
+    void licenseExemptInProviderHomeFlow() {
         createAValidLink();
         submitFamilyConfirmationCode();
 
@@ -319,7 +318,6 @@ void LicenseExemptInProviderHomeFlow() {
 
         // confirmation-code
         assertThat(testPage.getTitle()).isEqualTo(getEnMessage("provider-response-confirmation-code.title"));
-        testPage.enter("providerResponseFamilyShortCode", CONF_CODE);
         testPage.clickContinue();
 
         // response
@@ -400,7 +398,7 @@ void LicenseExemptInProviderHomeFlow() {
     }
 
     @Test
-    void LicenseExemptInChildHomeFlow() {
+    void licenseExemptInChildHomeFlow() {
         createAValidLink();
         submitFamilyConfirmationCode();
 
@@ -522,7 +520,6 @@ void LicenseExemptInChildHomeFlow() {
 
         // confirmation-code
         assertThat(testPage.getTitle()).isEqualTo(getEnMessage("provider-response-confirmation-code.title"));
-        testPage.enter("providerResponseFamilyShortCode", CONF_CODE);
         testPage.clickContinue();
 
         // response
@@ -582,7 +579,7 @@ void LicenseExemptInChildHomeFlow() {
     }
 
     @Test
-    void LicensedChildCareHomeFlow() {
+    void licensedChildCareHomeFlow() {
         createAValidLink();
         submitFamilyConfirmationCode();
 
@@ -695,7 +692,6 @@ void LicensedChildCareHomeFlow() {
 
         // confirmation-code
         assertThat(testPage.getTitle()).isEqualTo(getEnMessage("provider-response-confirmation-code.title"));
-        testPage.enter("providerResponseFamilyShortCode", CONF_CODE);
         testPage.clickContinue();
 
         // response
@@ -754,7 +750,7 @@ void LicensedChildCareHomeFlow() {
     }
 
     @Test
-    void LicensedChildCareCenterFlow() {
+    void licensedChildCareCenterFlow() {
         createAValidLink();
         submitFamilyConfirmationCode();
 
@@ -849,7 +845,6 @@ void LicensedChildCareCenterFlow() {
 
         // confirmation-code
         assertThat(testPage.getTitle()).isEqualTo(getEnMessage("provider-response-confirmation-code.title"));
-        testPage.enter("providerResponseFamilyShortCode", CONF_CODE);
         testPage.clickContinue();
 
         // response
@@ -908,7 +903,7 @@ void LicensedChildCareCenterFlow() {
     }
 
     @Test
-    void LicensedGroupChildCareHomeFlow() {
+    void licensedGroupChildCareHomeFlow() {
         createAValidLink();
         submitFamilyConfirmationCode();
 
@@ -1012,7 +1007,6 @@ void LicensedGroupChildCareHomeFlow() {
 
         // confirmation-code
         assertThat(testPage.getTitle()).isEqualTo(getEnMessage("provider-response-confirmation-code.title"));
-        testPage.enter("providerResponseFamilyShortCode", CONF_CODE);
         testPage.clickContinue();
 
         // response
@@ -1071,7 +1065,7 @@ void LicensedGroupChildCareHomeFlow() {
     }
 
     @Test
-    void LicenseExemptChildCareCenter() {
+    void licenseExemptChildCareCenter() {
         createAValidLink();
         submitFamilyConfirmationCode();
 
@@ -1202,7 +1196,6 @@ void LicenseExemptChildCareCenter() {
 
         // confirmation-code
         assertThat(testPage.getTitle()).isEqualTo(getEnMessage("provider-response-confirmation-code.title"));
-        testPage.enter("providerResponseFamilyShortCode", CONF_CODE);
         testPage.clickContinue();
 
         // response
@@ -1303,7 +1296,6 @@ private void createAValidLink() {
 
     private void submitFamilyConfirmationCode(){
         testPage.navigateToFlowScreen("providerresponse/confirmation-code");
-        testPage.enter("providerResponseFamilyShortCode", CONF_CODE);
         testPage.clickContinue();
     }
 }