DSD-1746: FeedbackBox interaction tests

[7emansell]

Fixes JIRA ticket DSD-1746

This PR does the following:

• Adds interaction tests in Storybook for the FeedbackBox component

How has this been tested?

Accessibility concerns or updates

Checklist:

• I have updated the Storybook documentation accordingly.
• I have added relevant accessibility documentation for this pull request.
• All new and existing tests passed.

Front End Review:

• Review the Vercel preview deployment once it is ready.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
nypl-design-system	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 27, 2024 1:50pm

[EdwinGuzman]

It'd be nice to get tests for the other fields but I'm not sure if you can rerender the component and set showCategoryField and showEmailField to true. Maybe those should be set to true for storybook, but that can be separate and a later step.

[EdwinGuzman]

Did not know you can get the mounted DOM directly from the package. I've been doing const screen = within(canvasElement);

[7emansell]

Right, something to remember for Modal for sure bc screen here is not the same as within(canvasElement)

[7emansell]

It'd be nice to get tests for the other fields but I'm not sure if you can rerender the component and set showCategoryField and showEmailField to true. Maybe those should be set to true for storybook, but that can be separate and a later step.

@bigfishdesign13 I think we've done this before with the Searchbar interaction tests (added props to the default component just so they can be tested) so I think it would make sense to do here as well, but lmk

[bigfishdesign13]

It'd be nice to get tests for the other fields but I'm not sure if you can rerender the component and set showCategoryField and showEmailField to true. Maybe those should be set to true for storybook, but that can be separate and a later step.

@bigfishdesign13 I think we've done this before with the Searchbar interaction tests (added props to the default component just so they can be tested) so I think it would make sense to do here as well, but lmk

@7emansell Yes, I think that would be fine to set showCategoryField and showEmailField to true for the With Controls story.

@EdwinGuzman The next release is next Thursday, so there would be time to tackle this change as part of this ticket.