Skip to content

Add Metro message command unit tests#2804

Merged
ConnorQi01 merged 2 commits into
masterfrom
dev/v-peq/metro_message_command_tests
Jul 23, 2026
Merged

Add Metro message command unit tests#2804
ConnorQi01 merged 2 commits into
masterfrom
dev/v-peq/metro_message_command_tests

Conversation

@ConnorQi01

Copy link
Copy Markdown
Collaborator

Summary

Add unit coverage for the ReloadApp and ShowDevMenu commands.

Proposed Changes

  • Verify ReloadApp sends the reload message to Metro for the current project.
  • Verify ShowDevMenu sends the devMenu message to Metro for the current project.
  • Verify errors from sendMessageToMetro propagate through both commands.

Test Plan

  • npx prettier --check test\extension\commands\reloadApp.test.ts test\extension\commands\showDevMenu.test.ts
  • npm run build
  • npm test - fails in an existing test: rn-extension commandsRegistered command list assertion. reloadAppCommand and showDevMenuCommand each ran 2 tests with 0 failures.

Closes #2793

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@ConnorQi01

Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
1 pipeline(s) were filtered out due to trigger conditions.

@ConnorQi01
ConnorQi01 merged commit 6f3db13 into master Jul 23, 2026
8 checks passed
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.

[Feature] Add unit tests for Metro message commands

2 participants