Skip to content

[FIXED] Correct test name to TestJetStreamInterestStreamConsumerFilterEdit - #5825

Merged
derekcollison merged 1 commit into
mainfrom
correct-test-name
Aug 23, 2024
Merged

[FIXED] Correct test name to TestJetStreamInterestStreamConsumerFilterEdit#5825
derekcollison merged 1 commit into
mainfrom
correct-test-name

Conversation

@MauriceVanVeen

Copy link
Copy Markdown
Member

Test should've been prefixed with TestJetStream. As noted by @wallyqs: #5818 (comment)

Signed-off-by: Maurice van Veen github@mauricevanveen.com

Signed-off-by: Maurice van Veen <github@mauricevanveen.com>
@MauriceVanVeen
MauriceVanVeen requested a review from a team as a code owner August 23, 2024 20:02

@wallyqs wallyqs left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@derekcollison
derekcollison merged commit 9afb8f4 into main Aug 23, 2024
@derekcollison
derekcollison deleted the correct-test-name branch August 23, 2024 22:35
@wallyqs wallyqs changed the title Correct test name [FIXED] Correct test name to TestJetStreamInterestStreamConsumerFilterEdit Aug 26, 2024
wallyqs added a commit that referenced this pull request Aug 27, 2024
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.

3 participants