Ignore “Character count” private method access for tests

Ideally we should update our tests to access only public methods, perhaps making getters/setters for things like `getCountMessage()` instead?
alphagov · Jun 8, 2023 · 69fedde · 69fedde
1 parent a592344
commit 69fedde
Showing 1 changed file with 17 additions and 0 deletions.
diff --git a/packages/govuk-frontend/src/govuk/components/character-count/character-count.unit.test.mjs b/packages/govuk-frontend/src/govuk/components/character-count/character-count.unit.test.mjs
@@ -55,8 +55,11 @@ describe('CharacterCount', () => {
             i18n: { charactersUnderLimit: { one: 'Custom text. Count: %{count}' } }
           })
 
+          // @ts-expect-error Property 'formatCountMessage' is private
           expect(component.formatCountMessage(1, 'characters')).toEqual('Custom text. Count: 1')
+
           // Other keys remain untouched
+          // @ts-expect-error Property 'formatCountMessage' is private
           expect(component.formatCountMessage(10, 'characters')).toEqual('You have 10 characters remaining')
         })
 
@@ -69,7 +72,10 @@ describe('CharacterCount', () => {
             }
           })
 
+          // @ts-expect-error Property 'formatCountMessage' is private
           expect(component.formatCountMessage(0, 'characters')).toEqual('Custom text.')
+
+          // @ts-expect-error Property 'formatCountMessage' is private
           expect(component.formatCountMessage(0, 'words')).toEqual('Different custom text.')
         })
       })
@@ -81,6 +87,7 @@ describe('CharacterCount', () => {
 
           const component = new CharacterCount($div)
 
+          // @ts-expect-error Property 'formatCountMessage' is private
           expect(component.formatCountMessage(10000, 'words')).toEqual('You have 10.000 words remaining')
         })
 
@@ -93,6 +100,7 @@ describe('CharacterCount', () => {
 
           const component = new CharacterCount($div)
 
+          // @ts-expect-error Property 'formatCountMessage' is private
           expect(component.formatCountMessage(10000, 'words')).toEqual('You have 10.000 words remaining')
         })
       })
@@ -104,8 +112,11 @@ describe('CharacterCount', () => {
 
           const component = new CharacterCount($div)
 
+          // @ts-expect-error Property 'formatCountMessage' is private
           expect(component.formatCountMessage(1, 'characters')).toEqual('Custom text. Count: 1')
+
           // Other keys remain untouched
+          // @ts-expect-error Property 'formatCountMessage' is private
           expect(component.formatCountMessage(10, 'characters')).toEqual('You have 10 characters remaining')
         })
 
@@ -121,9 +132,15 @@ describe('CharacterCount', () => {
                 }
               }
             })
+
+            // @ts-expect-error Property 'formatCountMessage' is private
             expect(component.formatCountMessage(1, 'characters')).toEqual('Custom text. Count: 1')
+
             // Other keys remain untouched
+            // @ts-expect-error Property 'formatCountMessage' is private
             expect(component.formatCountMessage(-10, 'characters')).toEqual('You have 10 characters too many')
+
+            // @ts-expect-error Property 'formatCountMessage' is private
             expect(component.formatCountMessage(0, 'characters')).toEqual('You have 0 characters remaining')
           })
         })