fix(popover): prevent toggletip ref error

[tay1orjones]

Reported in slack, this fixes Popover so that it won't incorrectly clone a ToggletipContent child.

The condition was a bit strange and took me a bit to wrap my head around, so I also broke the logic out into something more verbose. I think it helps with readability and being able to more easily understand what's going on there, let me know what you think!

Changelog

New

• add test to cover the previous failure

Changed

• update popover child clone logic to not clone ToggletipContent children

Testing / Reviewing

• Pull this down locally so you can validate there are no console errors in dev.
• Look at all the toggletip stories and other component stories that use Toggletip, there should be no errors in the console
• The new test should pass in CI

[netlify]

✅ Deploy Preview for v11-carbon-react ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	5827ce6
🔍 Latest deploy log	https://app.netlify.com/sites/v11-carbon-react/deploys/66da16ea9253c50008a2d2d3
😎 Deploy Preview	https://deploy-preview-17225--v11-carbon-react.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[netlify]

✅ Deploy Preview for carbon-elements ready!

Name	Link
🔨 Latest commit	5827ce6
🔍 Latest deploy log	https://app.netlify.com/sites/carbon-elements/deploys/66da16eacadbfe000804fc21
😎 Deploy Preview	https://deploy-preview-17225--carbon-elements.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[2nikhiltom]

No warning in console anymore, and the logic is more readable now 💯

[annawen1]

LGTM!