fix(OverflowMenu): forwardsRef

[AlexanderMelox]

Fixes test issue over in ibm-products carbon-design-system/ibm-products#7199

Currently updating to the latest version of carbon v11.79.0 fails our forwardRef tests

Currently our ButtonMenu component utilizes OverflowMenu under the hood

Changelog

Changed

• Added forwardRef to OverflowMenu

[netlify]

✅ Deploy Preview for v11-carbon-web-components ready!

Name	Link
🔨 Latest commit	e0e458c
🔍 Latest deploy log	https://app.netlify.com/sites/v11-carbon-web-components/deploys/67f7e43f5f05100008c950a5
😎 Deploy Preview	https://deploy-preview-19035--v11-carbon-web-components.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 v11-carbon-react ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	6170b03
🔍 Latest deploy log	https://app.netlify.com/sites/v11-carbon-react/deploys/67f4011b7229500008fe391b
😎 Deploy Preview	https://deploy-preview-19035--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 v11-carbon-react ready!

Name	Link
🔨 Latest commit	e0e458c
🔍 Latest deploy log	https://app.netlify.com/sites/v11-carbon-react/deploys/67f7e43fe25ea7000812ceb9
😎 Deploy Preview	https://deploy-preview-19035--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.

[tay1orjones]

Curious, were you relying on innerRef before?

• OverflowMenu innerRef no longer available #19046

---

Would you mind adding a couple tests to ensure each usage works as expected? Something lightweight like this maybe?

carbon/packages/react/src/components/Breadcrumb/__tests__/Breadcrumb-test.js

Lines 77 to 82 in ae084a4

	it('should accept a `ref` for the outermost element', () => {
	const ref = jest.fn();
	const { container } = render(<Breadcrumb ref={ref} />);
	expect(ref).toHaveBeenCalledWith(container.firstChild);
	});

[AlexanderMelox]

@tay1orjones haha yes I was just on that

[AlexanderMelox]

Pushing the test while we talk 👍🏻

[tay1orjones]

@AlexanderMelox Hey I don't think this will fix #19046 since innerRef is still not used and will remain broken. If you want to fix it here that's cool but for now I'll pull the Fixes line out of the PR body

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.96%. Comparing base (26b2544) to head (e0e458c).
Report is 5 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #19035   +/-   ##
=======================================
  Coverage   84.96%   84.96%           
=======================================
  Files         391      391           
  Lines       14490    14491    +1     
  Branches     4773     4721   -52     
=======================================
+ Hits        12312    12313    +1     
- Misses       2029     2030    +1     
+ Partials      149      148    -1     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.