RechartsWrapper event tests, and role=application

[PavelVanecek]

Description

Adding a couple of tests before I go ahead and mess with the implementation. I will try to add also tests for the payloads once I make sense of what's going on.

I also followed up on @julianna-langston advice from #4545 (comment) and I added a role='application' to all charts.

Related Issue

#4549

Motivation and Context

tests and a11y

How Has This Been Tested?

npm test

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• I have added a storybook story or extended an existing story to show my changes

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.72%. Comparing base (6a9029a) to head (c75de78).

Additional details and impacted files

@@            Coverage Diff             @@
##              3.x    #4555      +/-   ##
==========================================
+ Coverage   95.66%   95.72%   +0.06%     
==========================================
  Files         117      117              
  Lines       22462    22477      +15     
  Branches     3082     3095      +13     
==========================================
+ Hits        21488    21516      +28     
+ Misses        968      955      -13     
  Partials        6        6              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[julianna-langston]

lgtm

[ckifer]

LGTM!

[ckifer]

I'm sure it could, it's the newest chart