Skip to content

test: add regression test for AutoMQ target-node partition change builder - #3458

Open
sivaramanr38 wants to merge 1 commit into
AutoMQ:mainfrom
sivaramanr38:fix/reg-test-partition-change-logic
Open

test: add regression test for AutoMQ target-node partition change builder#3458
sivaramanr38 wants to merge 1 commit into
AutoMQ:mainfrom
sivaramanr38:fix/reg-test-partition-change-logic

Conversation

@sivaramanr38

Copy link
Copy Markdown

This PR adds a regression test for the AutoMQ‑specific target‑node leader election path in the partition change builder.

What Changed
Introduced a new test in ElasticPartitionChangeBuilderTest.java that exercises the setTargetNode behavior through the change‑record build path.

The test verifies that the emitted change record:

  • assigns the leader,
  • sets the ISR to the target node,
  • sets the replica list to the target node,
  • marks leader recovery as recovered.

Why This Is Useful

  • Strengthens coverage for AutoMQ‑specific partition change logic.
  • Provides confidence in a critical leader election path.
  • Helps prevent regressions without altering production code.

Testing Strategy

  • Verified correctness through the new unit test.
  • CI pipeline validation will ensure full test execution and integration.

@CLAassistant

CLAassistant commented Jul 11, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@sivaramanr38 sivaramanr38 changed the title Add regression test for AutoMQ target-node partition change builder test: add regression test for AutoMQ target-node partition change builder Jul 11, 2026
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.

2 participants