Fix failing jest test.

elastic · Jun 28, 2023 · c2c7d1c · c2c7d1c
1 parent f392350
commit c2c7d1c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/x-pack/plugins/cases/common/schema/index.test.ts b/x-pack/plugins/cases/common/schema/index.test.ts
@@ -51,7 +51,7 @@ describe('schema', () => {
       PathReporter.report(limitedArraySchema(NonEmptyString, 1, 1, 'foobar').decode(['a', 'b']))
     ).toMatchInlineSnapshot(`
       Array [
-        "The length of the field foobar is too short. Array must be of length <= 1.",
+        "The length of the field foobar is too long. Array must be of length <= 1.",
       ]
     `);
   });

diff --git a/x-pack/plugins/cases/common/schema/index.ts b/x-pack/plugins/cases/common/schema/index.ts
@@ -33,10 +33,10 @@ export const limitedArraySchema = <T extends rt.Mixed>(
     (input): input is T[] => rt.array(codec).is(input),
     (input, context) =>
       either.chain(rt.array(codec).validate(input, context), (s) => {
-        const fieldNameErrorMessage =
-          fieldName != null ? `The length of the field ${fieldName} is too short. ` : '';
-
         if (s.length < min) {
+          const fieldNameErrorMessage =
+            fieldName != null ? `The length of the field ${fieldName} is too short. ` : '';
+
           return rt.failure(
             input,
             context,
@@ -45,6 +45,8 @@ export const limitedArraySchema = <T extends rt.Mixed>(
         }
 
         if (s.length > max) {
+          const fieldNameErrorMessage =
+            fieldName != null ? `The length of the field ${fieldName} is too long. ` : '';
           return rt.failure(
             input,
             context,