Merge pull request #1123 from flexion/962_update_validation_messages

#962 Update Validation Messages
flexion · Mar 15, 2019 · 5de9fd7 · 5de9fd7
2 parents 40b8d76 + 10b0f69
commit 5de9fd7
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 12 deletions.
diff --git a/shared/src/business/entities/InitialWorkItemMessage.js b/shared/src/business/entities/InitialWorkItemMessage.js
@@ -28,9 +28,9 @@ joiValidationDecorator(
   }),
   undefined,
   {
-    assigneeId: 'Recipient is a required field.',
-    message: 'Message is a required field.',
-    section: 'Section is a required field.',
+    assigneeId: 'Recipient is required.',
+    message: 'Message is required.',
+    section: 'Section is required.',
   },
 );
 

diff --git a/shared/src/business/entities/InitialWorkItemMessage.test.js b/shared/src/business/entities/InitialWorkItemMessage.test.js
@@ -5,13 +5,13 @@ describe('InitialWorkItemMessage', () => {
     it('should have error messages for missing fields', () => {
       const message = new InitialWorkItemMessage({});
       expect(message.getFormattedValidationErrors().message).toEqual(
-        'Message is a required field.',
+        'Message is required.',
       );
       expect(message.getFormattedValidationErrors().assigneeId).toEqual(
-        'Recipient is a required field.',
+        'Recipient is required.',
       );
       expect(message.getFormattedValidationErrors().section).toEqual(
-        'Section is a required field.',
+        'Section is required.',
       );
     });
 

diff --git a/shared/src/business/useCases/workitems/validateInitialWorkItemMessageInteractor.test.js b/shared/src/business/useCases/workitems/validateInitialWorkItemMessageInteractor.test.js
@@ -17,9 +17,9 @@ describe('validateInitialWorkItemMessage', () => {
     });
 
     expect(errors).toEqual({
-      assigneeId: 'Recipient is a required field.',
-      message: 'Message is a required field.',
-      section: 'Section is a required field.',
+      assigneeId: 'Recipient is required.',
+      message: 'Message is required.',
+      section: 'Section is required.',
     });
   });
 

diff --git a/web-client/e2e/journey/docketClerkStartsNewMessageThreadOnAnswer.js b/web-client/e2e/journey/docketClerkStartsNewMessageThreadOnAnswer.js
@@ -10,9 +10,9 @@ export default test => {
     await test.runSequence('createWorkItemSequence');
 
     expect(test.getState('validationErrors')).toEqual({
-      assigneeId: 'Recipient is a required field.',
-      message: 'Message is a required field.',
-      section: 'Section is a required field.',
+      assigneeId: 'Recipient is required.',
+      message: 'Message is required.',
+      section: 'Section is required.',
     });
 
     await test.runSequence('updateFormValueSequence', {