fix: allows to use options configuration

[aegypius]

Fixes #79

[nikophil]

@kbond don't you think we could make methods TestTransport::isIntercepting(), isCatchingExceptions() and so on public? this way we could test this PR

[kbond]

don't you think we could make methods TestTransport::isIntercepting(), isCatchingExceptions() and so on public?

I don't have a problem with this.

[kbond]

This is really nice @aegypius!

[kbond]

@aegypius, could you fix the deprecations in TestTransportFactoryTest?

[aegypius]

@aegypius, could you fix the deprecations in TestTransportFactoryTest?

Working on it

[aegypius]

Should not self::$supportDelayStamp be resetted too ? https://github.com/aegypius/messenger-test/blob/allows-options-based-configuration/src/Transport/TestTransport.php#L337

[kbond]

Should not self::$supportDelayStamp be resetted too

I'm thinking yes but @nikophil, can you confirm?

[nikophil]

not sure this is needed:

TestTransport::resetAll() is called in a #[After] hook in order to reset everything in TestTransport that could have changed in the test execution: including all different states ($queued, $rejected...) but also $intercept and $catchExceptions which can be modified by methods catchExceptions()/throwExceptions() and unblock()/intercept()

Since we're not allowing $supportDelayStamp to be modified (and we should not do this: this behavior will be the default one in a further major version, and could not be disabled), it is not needed to reset it

[kbond]

not sure this is needed:

Ok, let's leave as is for now.

[kbond]

Thank you @aegypius!