Skip to content

[Test] Add packager port setting status coverage#2630

Open
ConnorQi01 wants to merge 1 commit into
dev/v-peq/packager-port-config-statusfrom
dev/v-peq/packager-port-config-status-tests
Open

[Test] Add packager port setting status coverage#2630
ConnorQi01 wants to merge 1 commit into
dev/v-peq/packager-port-config-statusfrom
dev/v-peq/packager-port-config-status-tests

Conversation

@ConnorQi01
Copy link
Copy Markdown
Collaborator

@ConnorQi01 ConnorQi01 commented May 14, 2026

Summary

Add focused tests for the packager port setting change behavior introduced in #2629.

Proposed Changes

  • Cover configuration changes while packagers are stopped so the cached port is refreshed silently.
  • Cover configuration changes while packagers are active so the running port is preserved and the status indicator marks the pending restart state.
  • Cover unrelated configuration changes.
  • Cover the status bar warning tooltip and clearing behavior after the new port is applied.

Test Plan

  • npm run build
  • npm test

Note: This PR is stacked on #2629 and should be reviewed after the feature PR.

Closes #2628

@ConnorQi01 ConnorQi01 changed the title Add tests for packager port setting status handling [Test] Add packager port setting status coverage May 14, 2026
@ConnorQi01 ConnorQi01 added Test javascript Pull requests that update javascript code labels May 14, 2026
@ConnorQi01 ConnorQi01 linked an issue May 14, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

javascript Pull requests that update javascript code Test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Test] Add packager port setting status coverage

1 participant