Skip to content

Conversation

AnastasiiaSvietlova
Copy link
Contributor

@AnastasiiaSvietlova AnastasiiaSvietlova commented Oct 13, 2025

Goal

This PR adds e2e tests for ErrorBoundary in React Native.

Changeset

Add ReactNativeErrorBoundaryScenario and react-native-error-boundary.feature.

Copy link

github-actions bot commented Oct 13, 2025

@bugsnag/browser bundle size diff

Minified Minfied + Gzipped
Before 47.75 kB 14.17 kB
After 47.75 kB 14.17 kB
± No change No change

code coverage diff

<temporarily disabled>

Generated by 🚫 dangerJS against bb8e0c6

@gingerbenw gingerbenw force-pushed the PLAT-13248-error-boundary-e2e-tests branch from 7803fc9 to f25be8b Compare October 15, 2025 11:14
@AnastasiiaSvietlova AnastasiiaSvietlova marked this pull request as ready for review October 15, 2025 13:22
@@ -0,0 +1,11 @@
@react_error_boundary
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this used / required? I don't see @react_error_boundary defined anywhere?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you're right. Deleted.

@react_error_boundary
Feature: React Error Boundary support

Scenario: basic error boundary usage
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Scenario: basic error boundary usage
Scenario: ErrorBoundary component reports errors

@@ -0,0 +1,11 @@
@react_error_boundary
Feature: React Error Boundary support
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Feature: React Error Boundary support
Feature: ReactNative ErrorBoundary support

Copy link
Contributor

@lemnik lemnik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants