refuse additional chars in names (#17086)

department-of-veterans-affairs · May 5, 2021 · c3ca27e · c3ca27e
1 parent c0f43ee
commit c3ca27e
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 5 deletions.
diff --git a/...ons/coronavirus-vaccination-expansion/config/personal-information/personal-information.js b/...ons/coronavirus-vaccination-expansion/config/personal-information/personal-information.js
@@ -4,7 +4,7 @@ import { personalInformation } from '../schema-imports';
 import currentOrPastDateUI from 'platform/forms-system/src/js/definitions/currentOrPastDate';
 
 function containsDisallowedCharacters(value) {
-  const disAllowedCharacters = ['(', ')'];
+  const disAllowedCharacters = ['(', ')', ',', ':', ';'];
   for (const character of disAllowedCharacters) {
     if (value.indexOf(character) !== -1) {
       return true;
@@ -28,7 +28,9 @@ export const uiSchema = {
         function(errors, fieldData) {
           const setError = containsDisallowedCharacters(fieldData);
           if (setError) {
-            errors.addError('Please only use letters and no parentheses');
+            errors.addError(
+              'Please only use letters and no special punctuation',
+            );
           }
         },
       ],
@@ -46,7 +48,7 @@ export const uiSchema = {
           const setError = containsDisallowedCharacters(fieldData);
           if (setError) {
             errors.addError(
-              'Please only use your current last name and no parentheses',
+              'Please only use your current last name and no special punctuation',
             );
           }
         },

diff --git a/...ination-expansion/tests/coronavirus-vaccination-expansion-form-validation.cypress.spec.js b/...ination-expansion/tests/coronavirus-vaccination-expansion-form-validation.cypress.spec.js
@@ -31,7 +31,17 @@ describe('COVID-19 SAVE LIVES Act sign up', () => {
       cy.findByLabelText(/Middle name/i).focus();
 
       cy.get('#root_firstName-error-message').contains(
-        'Please only use letters and no parentheses',
+        'Error Please only use letters and no special punctuation',
+      );
+
+      cy.findByLabelText(/First name/i)
+        .clear()
+        .type('Stephen:');
+
+      cy.findByLabelText(/Middle name/i).focus();
+
+      cy.get('#root_firstName-error-message').contains(
+        'Error Please only use letters and no special punctuation',
       );
 
       cy.findByLabelText(/Last name/i)
@@ -41,7 +51,17 @@ describe('COVID-19 SAVE LIVES Act sign up', () => {
       cy.findByLabelText(/Middle name/i).focus();
 
       cy.get('#root_lastName-error-message').contains(
-        'Error Please only use your current last name and no parentheses',
+        'Error Please only use your current last name and no special punctuation',
+      );
+
+      cy.findByLabelText(/Last name/i)
+        .clear()
+        .type('Beers,');
+
+      cy.findByLabelText(/Middle name/i).focus();
+
+      cy.get('#root_lastName-error-message').contains(
+        'Error Please only use your current last name and no special punctuation',
       );
     });
   });