fix(TreeView): treeview not passing props to treenode children

[Gopi-Agasthia-S-005CIU744]

Closes #19136

Addressed the issue where TreeView doesn't pass props to TreeNode if TreeNode is wrapped by a component

Changelog

Changed

• Refactored the way TreeView and TreeNode passes the prop to its children to recursively check for TreeNode Component in children

Testing / Reviewing

• In the TreeView story wrap the TreeNode with a div to check if props are passed correctly

PR Checklist

As the author of this PR, before marking ready for review, confirm you:

• Reviewed every line of the diff
• Updated documentation and storybook examples
• Wrote passing tests that cover this change
• Addressed any impact on accessibility (a11y)
• Tested for cross-browser consistency
• Validated that this code is ready for review and status checks should pass

More details can be found in the pull request guide

[github-actions]

All contributors have signed the DCO.
_{Posted by the DCO Assistant Lite bot.}

[netlify]

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

Name	Link
🔨 Latest commit	dfea6b7
🔍 Latest deploy log	https://app.netlify.com/projects/v11-carbon-web-components/deploys/682c1fe37d5da70008f3fc33
😎 Deploy Preview	https://deploy-preview-19417--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 project configuration.

[netlify]

✅ Deploy Preview for v11-carbon-react ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	dfea6b7
🔍 Latest deploy log	https://app.netlify.com/projects/v11-carbon-react/deploys/682c1fe319f3ea00075ef7a0
😎 Deploy Preview	https://deploy-preview-19417--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 project configuration.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.88%. Comparing base (b8c7071) to head (dfea6b7).
Report is 9 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #19417      +/-   ##
==========================================
+ Coverage   84.87%   84.88%   +0.01%     
==========================================
  Files         372      372              
  Lines       14431    14444      +13     
  Branches     4748     4751       +3     
==========================================
+ Hits        12248    12261      +13     
  Misses       2037     2037              
  Partials      146      146              

☔ 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.

[Gopi-Agasthia-S-005CIU744]

I have read the DCO document and I hereby sign the DCO.

[heloiselui]

Thank you so much for doing this! The code looks great!

Just one thing, can you add some tests for lines [188–189] and [211–212], as they are not currently covered by our tests.

Also, a small suggestion for clarity (totally optional, both are functionally equivalent):
I would replace in TreeNode.tsx file, lines [182–184]:

tabIndex: (!(node as ReactElement).props.disabled && -1) || null,

you could use:

tabIndex: node.props.disabled ? null : -1,

[tay1orjones]

Please do not remove checklist items from the pr template. If some do not apply, just ~strike out the text with tildes~

[Gopi-Agasthia-S-005CIU744]

Thank you so much for doing this! The code looks great!

Just one thing, can you add some tests for lines [188–189] and [211–212], as they are not currently covered by our tests.

Also, a small suggestion for clarity (totally optional, both are functionally equivalent): I would replace in TreeNode.tsx file, lines [182–184]:

tabIndex: (!(node as ReactElement).props.disabled && -1) || null,

you could use:

tabIndex: node.props.disabled ? null : -1,

• Added test case wherein a Treenode is wrapped in another component

• Refactored the TreeNode.tsx file -> used tabIndex: node.props.disabled ? null : -1 instead of tabIndex: (!(node as ReactElement).props.disabled && -1) || null

[heloiselui]

LGTM!

[carbon-automation]

Hey there! v11.83.0 was just released that references this issue/PR.