Nicolas PHILIPPE

Results 319 comments of Nicolas PHILIPPE

this issue is pretty old, is there a blocker here? or does it needs a huge effort to be fixed?

Hello, I discovered this issue because I've just upgraded to `v2.8.0` of this lib, and had an error `Unrecognized pipeline stage name: '$listSearchIndexes'` when running `UpdateCommand`. I think I have...

Hi @benito103e these properties *are* reset between tests (or at least, they should be :sweat_smile:) This behavior is tested here: https://github.com/zenstruck/messenger-test/blob/1.x/tests/TransportsAreResetCorrectly/UsingTraitInteractsWithMessengerTest.php Do your test suite actually shutdown the kernel between...

ok you're right, that's a bug :sweat_smile: Maybe a `#[BeforeClass]` would do the job?

I know we had complex problems around this in the past. @kbond any clue why we don't call these methods before each test? ```php TestTransport::resetAll(); TestBus::resetAll(); ```

I think the problem was around blocking/intercepting. maybe the easiest thing to do would be to initialize as well `$queue`, `$dispatched`, `$acknowledged` and `$rejected` in `TestTransport::initialize()`?

Hey! I've thought a little bit more to this problem What if we make `TestTransport` implement `ResetInterface`? this is what is done in `InMemoryTransport` At first I was thinking it...

Hello @mbopp I think that if you use `$messenger->reset()` you won't have the problem. nonetheless, there is a problem with `resetAll()` method: we're resetting `$intercept` and `$catchExceptions` to `null`, but...

hi @sidz we should detect if the current test uses the trait `InteractWithMessenger` (maybe by doing something similar than [here](https://github.com/zenstruck/mailer-test/blob/a7bd13d2418ae58b5384ad3c61a98d5a0bbcad33/src/InteractsWithMailer.php#L19)) And if we detect that the trait was used, we...