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.
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
Testing extensions for TestableMessageSession are missing #535
Conversation
@adamralph @DavidBoike @andreasohlund Pinged you as reviewers since you might have more insights about why they were removed. Happy to close the PR in case there are reasons why they are no longer supported. Otherwise, I think it would be good to bring them back (patch or minor?) |
I can't remember that we removed them on purpose so 👍 to fix this |
@@ -300,9 +300,12 @@ namespace NServiceBus.Testing | |||
} | |||
public static class TestingExtensions | |||
{ | |||
public static TMessage FindPublishedMessage<TMessage>(this NServiceBus.Testing.TestableMessageSession context) { } | |||
public static TMessage FindPublishedMessage<TMessage>(this NServiceBus.Testing.TestablePipelineContext context) { } | |||
public static TMessage FindReplyMessage<TMessage>(this NServiceBus.Testing.TestableMessageProcessingContext context) { } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not FindReplyMessage on TestableMessageSession as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ReplyMessages only exist on TestableMessageProcessingContex. TestableMessageSession doesn't even have the property.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's a good reason!
Fixes #533
Brings back extension methods for
TestableMessageSession
that were accidentally removed as part of #274