Upgrade Recharts. (PP-1531)

[tdilauro]

Description

• Upgrades Recharts package to a modern version.
• Moves some more tests over to jest.
• Also adds text wrap balancing to the statistics group headings and descriptions.

Note: This work builds on #126 and should be merged after it.

Motivation and Context

• Updates a transitive dependency with a high-impact vulnerability.

[Jira PP-1531]

How Has This Been Tested?

• Manually tested in dev-server environment.
• Added and migrated tests.
• CI tests pass for associated branch.

Checklist:

• N/A - I have updated the documentation accordingly.
• All new and existing tests passed.

[tdilauro]

I added this and the overall testingFlags approach to:

• provide a higher-level way to support some tricky test scenarios without having to drill a prop through a bunch of components; and
• to keep as much of the mocking near the component, so that it is easier to keep them aligned.

Would appreciate feedback on this. Bad idea? Any better ideas?

[ray-lee]

In general I'd rather have control of the tests be completely inside the test files, rather than interleaved into the components that are tested. If I'm reading this right, this test flag is overriding the width and height props of RechartsResponsiveContainer. My preference would be instead of setting a testing flag, allow specifying the width and height overrides directly. So your context might contain something like

config: { // these key names are off the cuff, I don't care much what they are exactly
  StatsCollectionsBarChart: {
    width: 800,
    height: 800
  } 
}

That way it's easy to see in the test what exactly is happening, and different tests can pass different settings if they want to, instead of having them hardcoded into the component. From the component's POV, it doesn't know or care if it's under test, it just has a feature (that can itself be tested) that some settings are configurable from context.

[tdilauro]

Thanks, @ray-lee! That's very helpful. I think it makes a lot of sense and reduces complexity. I've made changes in that direction.

[tdilauro]

@ray-lee This has been rebased onto main now.

[ray-lee]

Looks good!