apache / apache/rocketmq-dashboard

Add regression coverage for reserved message property detection

Open Beginner friendly
#4,044 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
1.4k
Forks
683
Avg merge
2d 14h
Merged PRs (30d)
58

Description

## Problem
The current test suite does not cover reserved message property detection.

## Expected behavior
- A focused Vitest case locks the existing behavior.
- The targeted test file passes and fails if the covered behavior regresses.

## Scope
Only add regression coverage for $(@{Slug=reserved-message-property; Focus=reserved message property detection; PrTitle=test(message-properties): cover reserved property detection; TestFile=web/src/utils/messagePayloadPreview.test.ts; Mode=existing; Append=describe('message reserved property detection', () => {
it('recognizes reserved message properties case-insensitively after trimming', () => {
expect(isReservedMessageProperty('tags')).toBe(true);
expect(isReservedMessageProperty(' UNIQ_KEY ')).toBe(true);
expect(isReservedMessageProperty('tenant')).toBe(false);
});
});}.TestFile). No runtime behavior changes.

## Acceptance criteria
- [ ] The targeted Vitest file passes.
- [ ] git diff --check is clean.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with web/src/utils/messagePayloadPreview.test.ts and the existing isReservedMessageProperty coverage. Run the targeted Vitest file, then add the specified case for trimmed, case-insensitive reserved properties; done means the test passes and git diff --check is clean.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
testing
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Active
Clarity
Clearly specified
Newbie friendliness
90/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.