Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Enhancement] add more test coverage for broker.PeekMessageProcessor #8495

Closed
1 task done
TanXiang7o opened this issue Aug 5, 2024 · 1 comment · Fixed by #8498
Closed
1 task done

[Enhancement] add more test coverage for broker.PeekMessageProcessor #8495

TanXiang7o opened this issue Aug 5, 2024 · 1 comment · Fixed by #8498
Labels
type/enhancement

Comments

@TanXiang7o
Copy link
Contributor

Before Creating the Enhancement Request

  • I have confirmed that this should be classified as an enhancement rather than a bug/feature.

Summary

image
add more test coverage

Motivation

add more test coverage

Describe the Solution You'd Like

add more test coverage

Describe Alternatives You've Considered

add more test coverage

Additional Context

No response
TanXiang7o added a commit to TanXiang7o/rocketmq that referenced this issue Aug 5, 2024
@TanXiang7o
[ISSUE apache#8495]add more test coverage for PeekMessageProcessor
9370175
TanXiang7o added a commit to TanXiang7o/rocketmq that referenced this issue Aug 5, 2024
@TanXiang7o
[ISSUE apache#8495]add more test coverage for PeekMessageProcessor
b3c6e79
@TanXiang7o TanXiang7o mentioned this issue Aug 5, 2024
Merged
RongtongJin pushed a commit that referenced this issue Aug 6, 2024
@TanXiang7o
[ISSUE #8495] Add more test coverage for PeekMessageProcessor (#8498)
87fb1f5 
* [ISSUE #8495]add more test coverage for PeekMessageProcessor

* [ISSUE #8495]add more test coverage for PeekMessageProcessor
@ShannonDing
Copy link
Member

link #8262

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type/enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ISSUE #8495]add more test coverage for PeekMessageProcessor TanXiang7o/rocketmq
2 participants
@ShannonDing @TanXiang7o