Skip to content

Add integration tests for error preview formatting - #55452

Open
emily8rown wants to merge 1 commit into
react:mainfrom
emily8rown:export-D91780051
Open

emily8rown wants to merge 1 commit into
react:mainfrom
emily8rown:export-D91780051

Conversation

@emily8rown

Copy link
Copy Markdown
Contributor

Summary:
This diff adds React Native jsinspector integration tests for the Hermes CDP error preview changes made in D79451147.

The new tests cover error description formatting for:

  • Simple errors with messages
  • Errors with empty string messages (should omit the colon)
  • Errors with no message (should omit the colon)
  • Errors with custom name property
  • Custom error classes (transpiled for Hermes compatibility)
  • Custom error classes with empty/no messages

These tests ensure error previews in the DevTools console correctly display the error name and message, matching V8's behavior.

Differential Revision: D91780051

Summary:
This diff adds React Native jsinspector integration tests for the Hermes CDP error preview changes made in D79451147.

The new tests cover error description formatting for:
- Simple errors with messages
- Errors with empty string messages (should omit the colon)
- Errors with no message (should omit the colon)
- Errors with custom `name` property
- Custom error classes (transpiled for Hermes compatibility)
- Custom error classes with empty/no messages

These tests ensure error previews in the DevTools console correctly display the error name and message, matching V8's behavior.

Differential Revision: D91780051
@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Feb 6, 2026
@meta-codesync

meta-codesync Bot commented Feb 6, 2026

Copy link
Copy Markdown

@emily8rown has exported this pull request. If you are a Meta employee, you can view the originating Diff in D91780051.

@react-native-bot

Copy link
Copy Markdown
Collaborator

This PR is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@react-native-bot react-native-bot added the Stale There has been a lack of activity on this issue and it may be closed soon. label Oct 13, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported meta-exported p: Facebook Partner: Facebook Partner Stale There has been a lack of activity on this issue and it may be closed soon.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants