-
-
Notifications
You must be signed in to change notification settings - Fork 13
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
Add a way to handle transport specific interfaces #83
Comments
I'm wondering if we should just make |
It would definitely solve this test but would fail if the test does the opposite |
seems like our hands are tied... |
seen directly with @aegypius: a solution would be to decorate with an anonymous class: new class extends TestTransport implements MessageCountAwareInterface {
use MessageCountAwareTrait;
} but this would cause problem with interface composition ( Let's use a pragmatic solution: since our any thoughts @kbond ? |
I promise I will not 🤣 ! |
I agree!
@aegypius, you're fine with not being able to test the opposite (or using another solution for this test)? |
@kbond yes I am fine with this solution |
One of my tests fails when using this package if i use a
test://
configuration.To be more precise, I am writing a prometheus collector that iterates against transports and expose the number of message to be processed. For that purpose, I use a similar code as :
If I don't use
test://
scheme, the native transport (which hold aMessageCountAwareInterface
interface) is properly listed but when I start using this package to tests queues the transport is no longer listed and my integration test fails.How would you solve this issue ? Do we need to add some sorte of option to the test transport to had these kind of interface ?
The text was updated successfully, but these errors were encountered: