Skip to content

test(e2e): don't snapshot react-native's command descriptions and order - #2894

Merged
thymikee merged 1 commit into
mainfrom
fix/e2e-config-snapshot
Nov 13, 2026
Merged

thymikee merged 1 commit into
mainfrom
fix/e2e-config-snapshot

Conversation

@thymikee

Copy link
Copy Markdown
Member

Summary

__e2e__/config.test.ts fails on main (e.g. https://github.com/react-native-community/cli/actions/runs/43202706151): the config output snapshot includes commands registered by react-native itself (bundle, start, codegen, spm [action]). React Native 0.88.1 changed the spm description (added sync) and the order those commands are registered in, so the snapshot breaks on a React Native patch release, with no CLI change.

The test now replaces command descriptions with <<REPLACED>>, the same as it already does for options and examples, and sorts commands by name. The snapshot still catches a command being added or removed, or gaining or losing options or examples.

Test Plan

CI (E2E Tests). The snapshot was updated by applying the same transform to the existing one; it contains the same commands.

@thymikee
thymikee merged commit 19d5636 into main Nov 13, 2026
8 checks passed
@thymikee
thymikee deleted the fix/e2e-config-snapshot branch November 13, 2026 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant