Skip to content

[noop] Fail tests on unasserted recoverable errors#35948

Open
eps1lon wants to merge 1 commit intofacebook:mainfrom
eps1lon:sebbie/03-02-_noop_fail_tests_on_unasserted_recoverable_errors
Open

[noop] Fail tests on unasserted recoverable errors#35948
eps1lon wants to merge 1 commit intofacebook:mainfrom
eps1lon:sebbie/03-02-_noop_fail_tests_on_unasserted_recoverable_errors

Conversation

@eps1lon
Copy link
Collaborator

@eps1lon eps1lon commented Mar 3, 2026

react-noop-renderer previously swallowed recoverable errors (from onRecoverableError). Now we pass those through to console.error so that we fail the test unless assertConsoleError

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

react-sizebot commented Mar 3, 2026

Comparing: 4cc5b7a...1b61b6a

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 = 611.79 kB 611.79 kB = 108.12 kB 108.12 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.84 kB 6.84 kB = 1.88 kB 1.88 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 677.72 kB 677.72 kB = 119.08 kB 119.08 kB
facebook-www/ReactDOM-prod.classic.js = 697.72 kB 697.72 kB = 122.59 kB 122.59 kB
facebook-www/ReactDOM-prod.modern.js = 688.03 kB 688.03 kB = 120.97 kB 120.97 kB

Significant size changes

Includes any change greater than 0.2%:

Expand to show
Name +/- Base Current +/- gzip Base gzip Current gzip
react-native/implementations/ReactFabric-profiling.fb.js = 466.22 kB 464.90 kB = 78.57 kB 78.23 kB
react-native/implementations/ReactFabric-profiling.js = 465.40 kB 463.94 kB = 78.28 kB 77.96 kB

Generated by 🚫 dangerJS against 1b61b6a

@eps1lon eps1lon force-pushed the sebbie/03-02-_noop_fail_tests_on_unasserted_recoverable_errors branch 3 times, most recently from 4d351a7 to a098999 Compare March 3, 2026 13:39
@eps1lon eps1lon force-pushed the sebbie/03-02-_noop_fail_tests_on_unasserted_recoverable_errors branch from a098999 to 1b61b6a Compare March 3, 2026 13:40
@eps1lon eps1lon marked this pull request as ready for review March 3, 2026 14:08
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