STCOM-1343 refactor away from findDOMNode

[JohnC-80]

This touches Accordion (including MetaSection), MCL, and AdvancedSearch so far...
This goes in-hand with folio-org/stripes-react-hotkeys#44 (Merged!)

Summary of changes:

• In general, Instances of Hotkeys are provided with a ref to a wrapping element.
• AdvancedSearch Modal: wrapping div added.
• MCL components for dom-measuring refactor to contain their rendered elements/implement as a child function providing the necessary ref as opposed to using findDOMNode
• Accordion headers wrapped in a div with a ref to that div provided to HotKeys.

[github-actions]

Bigtest Unit Test Results

    1 files  ±0      1 suites  ±0   14s ⏱️ ±0s
1 515 tests ±0  1 507 ✅ ±0  8 💤 ±0  0 ❌ ±0 
1 517 runs  ±0  1 509 ✅ ±0  8 💤 ±0  0 ❌ ±0 

Results for commit 3a2aa86. ± Comparison against base commit 23a0c49.

This pull request removes 1515 and adds 1515 tests. Note that renamed tests count towards both.

Chrome_128_0_0_0_(Linux_x86_64).Accordion - as part of an AccordionSet contents ready for interaction following open ‑ Accordion - as part of an AccordionSet contents ready for interaction following open Child element was filled out successfully
Chrome_128_0_0_0_(Linux_x86_64).Accordion - as part of an AccordionSet keyboard navigation: first accordion ‑ Accordion - as part of an AccordionSet keyboard navigation: first accordion First accordion is in focus
Chrome_128_0_0_0_(Linux_x86_64).Accordion - as part of an AccordionSet keyboard navigation: last accordion ‑ Accordion - as part of an AccordionSet keyboard navigation: last accordion Last accordion is in focus
Chrome_128_0_0_0_(Linux_x86_64).Accordion - as part of an AccordionSet keyboard navigation: next accordion ‑ Accordion - as part of an AccordionSet keyboard navigation: next accordion second accordion is in focus
Chrome_128_0_0_0_(Linux_x86_64).Accordion - as part of an AccordionSet keyboard navigation: previous accordion ‑ Accordion - as part of an AccordionSet keyboard navigation: previous accordion first accordion is in focus
Chrome_128_0_0_0_(Linux_x86_64).Accordion - as part of an AccordionSet ‑ Accordion - as part of an AccordionSet has a button
Chrome_128_0_0_0_(Linux_x86_64).Accordion - as part of an AccordionSet ‑ Accordion - as part of an AccordionSet has no axe errors
Chrome_128_0_0_0_(Linux_x86_64).Accordion - as part of an AccordionSet ‑ Accordion - as part of an AccordionSet should call onRegisterAccordion callback
Chrome_128_0_0_0_(Linux_x86_64).Accordion - updating state of parent component opening the closed accordion changing the text value/parent state ‑ Accordion - updating state of parent component opening the closed accordion changing the text value/parent state renders second accordion as open
Chrome_128_0_0_0_(Linux_x86_64).Accordion - updating state of parent component opening the closed accordion changing the text value/parent state ‑ Accordion - updating state of parent component opening the closed accordion changing the text value/parent state renders state value in textField
…

Chrome_129_0_0_0_(Linux_x86_64).Accordion - as part of an AccordionSet contents ready for interaction following open ‑ Accordion - as part of an AccordionSet contents ready for interaction following open Child element was filled out successfully
Chrome_129_0_0_0_(Linux_x86_64).Accordion - as part of an AccordionSet keyboard navigation: first accordion ‑ Accordion - as part of an AccordionSet keyboard navigation: first accordion First accordion is in focus
Chrome_129_0_0_0_(Linux_x86_64).Accordion - as part of an AccordionSet keyboard navigation: last accordion ‑ Accordion - as part of an AccordionSet keyboard navigation: last accordion Last accordion is in focus
Chrome_129_0_0_0_(Linux_x86_64).Accordion - as part of an AccordionSet keyboard navigation: next accordion ‑ Accordion - as part of an AccordionSet keyboard navigation: next accordion second accordion is in focus
Chrome_129_0_0_0_(Linux_x86_64).Accordion - as part of an AccordionSet keyboard navigation: previous accordion ‑ Accordion - as part of an AccordionSet keyboard navigation: previous accordion first accordion is in focus
Chrome_129_0_0_0_(Linux_x86_64).Accordion - as part of an AccordionSet ‑ Accordion - as part of an AccordionSet has a button
Chrome_129_0_0_0_(Linux_x86_64).Accordion - as part of an AccordionSet ‑ Accordion - as part of an AccordionSet has no axe errors
Chrome_129_0_0_0_(Linux_x86_64).Accordion - as part of an AccordionSet ‑ Accordion - as part of an AccordionSet should call onRegisterAccordion callback
Chrome_129_0_0_0_(Linux_x86_64).Accordion - updating state of parent component opening the closed accordion changing the text value/parent state ‑ Accordion - updating state of parent component opening the closed accordion changing the text value/parent state renders second accordion as open
Chrome_129_0_0_0_(Linux_x86_64).Accordion - updating state of parent component opening the closed accordion changing the text value/parent state ‑ Accordion - updating state of parent component opening the closed accordion changing the text value/parent state renders state value in textField
…

This pull request removes 8 skipped tests and adds 8 skipped tests. Note that renamed tests count towards both.

Chrome_128_0_0_0_(Linux_x86_64).Datepicker defaultOutputFormatter converts non-Latn input to Latn-output ‑ Datepicker defaultOutputFormatter converts non-Latn input to Latn-output returns a Latn (Arabic, 0-9) string
Chrome_128_0_0_0_(Linux_x86_64).Datepicker with Redux Form integration exclude prop selecting excluded date navigation on excluded date ‑ Datepicker with Redux Form integration exclude prop selecting excluded date navigation on excluded date changed cursor to April 2, 2018
Chrome_128_0_0_0_(Linux_x86_64).Editor rendering a basic Editor ‑ Editor rendering a basic Editor has no axe errors. - Editor
Chrome_128_0_0_0_(Linux_x86_64).Popover If the legacy component API is used When the toggle is clicked again ‑ Popover If the legacy component API is used When the toggle is clicked again Hides the popover overlay
Chrome_128_0_0_0_(Linux_x86_64).Popover If the legacy component API is used ‑ Popover If the legacy component API is used Renders the legacy popover and opens the overlay when the toggle is clicked
Chrome_128_0_0_0_(Linux_x86_64).childrenOf propType validator Passing a type that isn't allowed ‑ childrenOf propType validator Passing a type that isn't allowed triggers a propType error
Chrome_128_0_0_0_(Linux_x86_64).childrenOf propType validator Passing a type that isn't allowed ‑ childrenOf propType validator Passing a type that isn't allowed triggers no proptype error
Chrome_128_0_0_0_(Linux_x86_64).childrenOf propType validator not providing a required prop ‑ childrenOf propType validator not providing a required prop triggers a required propType error

Chrome_129_0_0_0_(Linux_x86_64).Datepicker defaultOutputFormatter converts non-Latn input to Latn-output ‑ Datepicker defaultOutputFormatter converts non-Latn input to Latn-output returns a Latn (Arabic, 0-9) string
Chrome_129_0_0_0_(Linux_x86_64).Datepicker with Redux Form integration exclude prop selecting excluded date navigation on excluded date ‑ Datepicker with Redux Form integration exclude prop selecting excluded date navigation on excluded date changed cursor to April 2, 2018
Chrome_129_0_0_0_(Linux_x86_64).Editor rendering a basic Editor ‑ Editor rendering a basic Editor has no axe errors. - Editor
Chrome_129_0_0_0_(Linux_x86_64).Popover If the legacy component API is used When the toggle is clicked again ‑ Popover If the legacy component API is used When the toggle is clicked again Hides the popover overlay
Chrome_129_0_0_0_(Linux_x86_64).Popover If the legacy component API is used ‑ Popover If the legacy component API is used Renders the legacy popover and opens the overlay when the toggle is clicked
Chrome_129_0_0_0_(Linux_x86_64).childrenOf propType validator Passing a type that isn't allowed ‑ childrenOf propType validator Passing a type that isn't allowed triggers a propType error
Chrome_129_0_0_0_(Linux_x86_64).childrenOf propType validator Passing a type that isn't allowed ‑ childrenOf propType validator Passing a type that isn't allowed triggers no proptype error
Chrome_129_0_0_0_(Linux_x86_64).childrenOf propType validator not providing a required prop ‑ childrenOf propType validator not providing a required prop triggers a required propType error

♻️ This comment has been updated with latest results.

[BogdanDenis]

Looks good, please also update CHANGELOG

[sonarcloud]

Quality Gate passed

Issues
4 New issues
0 Accepted issues

Measures
0 Security Hotspots
88.9% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud