Skip to content

[tests] Fix ReactDOMAttribute-test#35654

Merged
rickhanlonii merged 1 commit intofacebook:mainfrom
rickhanlonii:rh/rm-todos
Jan 28, 2026
Merged

[tests] Fix ReactDOMAttribute-test#35654
rickhanlonii merged 1 commit intofacebook:mainfrom
rickhanlonii:rh/rm-todos

Conversation

@rickhanlonii
Copy link
Member

In #35646 I thought there was a bug in trusted types, but the bug is in jsdom.

For trusted types we still want to check the coersion and throw for a good dev warning, but prod will also throw becuase the browser will implicitly coerce to a string. This ensures there's no behavior difference between dev and prod.

So the right fix is to add in the JSDOM hack that's used in ReactDOMSelect-test.js.

@meta-cla meta-cla bot added the CLA Signed label Jan 28, 2026
@github-actions github-actions bot added the React Core Team Opened by a member of the React Core Team label Jan 28, 2026
@react-sizebot
Copy link

Comparing: 90b2dd4...5de7b73

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.js = 6.84 kB 6.84 kB = 1.88 kB 1.88 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 609.58 kB 608.67 kB = 107.80 kB 107.63 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.84 kB 6.84 kB +0.05% 1.88 kB 1.88 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 675.51 kB 674.60 kB = 118.75 kB 118.58 kB
facebook-www/ReactDOM-prod.classic.js = 695.14 kB 693.92 kB = 122.19 kB 121.98 kB
facebook-www/ReactDOM-prod.modern.js = 685.52 kB 684.31 kB = 120.59 kB 120.37 kB

Significant size changes

Includes any change greater than 0.2%:

Expand to show
Name +/- Base Current +/- gzip Base gzip Current gzip
facebook-react-native/react-dom/cjs/ReactDOMProfiling-prod.js = 602.59 kB 601.37 kB = 106.16 kB 105.94 kB
facebook-react-native/react-dom/cjs/ReactDOMClient-prod.js = 596.93 kB 595.71 kB = 105.03 kB 104.80 kB
oss-experimental/react-reconciler/cjs/react-reconciler-reflection.development.js = 12.79 kB 12.75 kB = 2.72 kB 2.71 kB
oss-stable-semver/react-reconciler/cjs/react-reconciler-reflection.development.js = 12.79 kB 12.75 kB = 2.72 kB 2.71 kB
oss-stable/react-reconciler/cjs/react-reconciler-reflection.development.js = 12.79 kB 12.75 kB = 2.72 kB 2.71 kB
oss-experimental/react-reconciler/cjs/react-reconciler-reflection.production.js = 11.32 kB 11.29 kB = 2.64 kB 2.63 kB
oss-stable-semver/react-reconciler/cjs/react-reconciler-reflection.production.js = 11.32 kB 11.29 kB = 2.64 kB 2.63 kB
oss-stable/react-reconciler/cjs/react-reconciler-reflection.production.js = 11.32 kB 11.29 kB = 2.64 kB 2.63 kB

Generated by 🚫 dangerJS against 5de7b73

@rickhanlonii rickhanlonii merged commit 230772f into facebook:main Jan 28, 2026
240 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments