Skip to content

fix(mock): isolate consumer groups by instance - #2880

Open
btlqql wants to merge 1 commit into
apache:rocketmq-studiofrom
btlqql:codex/btl-mock-consumer-instance-scope
Open

fix(mock): isolate consumer groups by instance#2880
btlqql wants to merge 1 commit into
apache:rocketmq-studiofrom
btlqql:codex/btl-mock-consumer-instance-scope

Conversation

@btlqql

@btlqql btlqql commented Sep 1, 2026

Copy link
Copy Markdown

Summary

  • retain instanceId on newly created mock consumer groups
  • scope mock list, detail, refresh and delete operations to the requested instance
  • allow equal group names in different instances while rejecting duplicates within one instance

Tests

  • npm test -- src/services/consumerService.test.ts (16 passed)
  • npm exec eslint -- src/services/consumerService.ts src/services/consumerService.test.ts
  • git diff --check

Closes #2878

@RockteMQ-AI RockteMQ-AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Summary

Fixes mock data to isolate consumer groups by instance name, preventing cross-contamination between test scenarios. Small, focused change with proper test coverage.

LGTM


Automated review by "github-manager-bot"

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