Skip to content

Verify clean & restart packager command works correctly#2489

Merged
EmmaYuan1015 merged 2 commits intomasterfrom
verify-clean-restart-packager-command
Jan 9, 2026
Merged

Verify clean & restart packager command works correctly#2489
EmmaYuan1015 merged 2 commits intomasterfrom
verify-clean-restart-packager-command

Conversation

@EmmaYuan1015
Copy link
Contributor

  • Add executeCommand helper method to ComponentHelper for executing VS Code commands via command palette
  • Add waitPackagerStateIncludes and waitPackagerStateIncludesOneOf helpers to reduce repeated assertion code
  • Add comprehensive smoke test 'Verify Clean & Restart Packager command works correctly' that validates:
    • Packager starts properly before the clean restart
    • Clean & Restart Packager command executes successfully
    • Packager transitions through loading and ready states
    • Metro packager cache is cleaned and restarted successfully

- Add executeCommand helper method to ComponentHelper for executing VS Code commands via command palette
- Add waitPackagerStateIncludes and waitPackagerStateIncludesOneOf helpers to reduce repeated assertion code
- Add comprehensive smoke test 'Verify Clean & Restart Packager command works correctly' that validates:
  - Packager starts properly before the clean restart
  - Clean & Restart Packager command executes successfully
  - Packager transitions through loading and ready states
  - Metro packager cache is cleaned and restarted successfully
@EmmaYuan1015
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

- Remove pre-start packager logic, let command handle startup
- Increase timeout to 180s for CI environments with slower I/O
- Simplify test to focus on command execution and final state verification
@EmmaYuan1015
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@EmmaYuan1015 EmmaYuan1015 marked this pull request as ready for review January 7, 2026 06:30
@EmmaYuan1015 EmmaYuan1015 merged commit 499cb0e into master Jan 9, 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.

[Smoke test] Verify clean & restart packager command works correctly

2 participants