Merge pull request #1133 from guardian/wordcount-fix

Fix bug in word limit
guardian · Aug 8, 2024 · 0901331 · 0901331
2 parents 9bf3b8a + bb93255
commit 0901331
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/public/video-ui/src/components/FormFields/richtext/utils/richTextHelpers.ts b/public/video-ui/src/components/FormFields/richtext/utils/richTextHelpers.ts
@@ -22,10 +22,7 @@ export const getWords = (text: string): string[] => {
 };
 
 export const isTooLong = (value: string, maxWordLength: number): boolean => {
-  const wordLength = getWords(value).reduce((length, word) => {
-    length += word.length;
-    return length;
-  }, 0);
+  const wordLength = getWords(value).length;
   return (
     wordLength > maxWordLength
   );

diff --git a/public/video-ui/src/test/richTextInput.spec.ts b/public/video-ui/src/test/richTextInput.spec.ts
@@ -21,9 +21,9 @@ describe("getWords", () => {
 });
 
 describe("isTooLong", () => {
-  it('Should return false for phrases less than the character and word limit', () => {
+  it('Should return false for phrases less than the word limit', () => {
     const tooLongString = "So it goes";
-    const maxWords = 100;
+    const maxWords = 3;
 
     const isTooLongResult = isTooLong(tooLongString, maxWords);