Improve compatibility logic for the EventDispatcher

[TavoNiievez]

See failed job in Codeception/module-laravel#12 . Caused by:

phpunit-wrapper/src/DispatcherWrapper.php

Lines 20 to 26 in f5d311f

	if (!interface_exists(TraceableEventDispatcherInterface::class)) {
	//Symfony 5
	$dispatcher->dispatch($eventObject, $eventType);
	} else {
	//Symfony 2,3 or 4
	$dispatcher->dispatch($eventType, $eventObject);
	}

Suggested change (based on https://github.com/lexik/LexikJWTAuthenticationBundle/pull/669/files):

- use Symfony\Component\EventDispatcher\Debug\TraceableEventDispatcherInterface;
+ use Symfony\Contracts\EventDispatcher\EventDispatcherInterface as ContractsEventDispatcherInterface;

//...

-   if (!interface_exists(TraceableEventDispatcherInterface::class)) {
+   if ($dispatcher instanceof ContractsEventDispatcherInterface) {
        $dispatcher->dispatch($eventObject, $eventType);
    } else {
        $dispatcher->dispatch($eventType, $eventObject);
    }

Related to: Codeception/Codeception#6065

[Naktibalda]

It's complicated. I changed this logic at least once and it still has caveats.
Changing it again can break tests for someone else.

It is on the list of things I'm going to remove in Codeception 5.
I think that it would be the best to wait for Codeception 5 now.

[Naktibalda]

Alright, I looked at Symfony code and this proposal looks good, please raise pull requests to 6.0, 6.5, 7.1, 8.0 and 9.0 branches.

[TavoNiievez]

I already sent the PRs for all the branches of this repo. May I send one also for the 4.1 branch of Codeception\Codeception?

[Naktibalda]

Yes, please update 4.1 branch of codeception/codeception too.

[TavoNiievez]

Thanks, see Codeception/Codeception#6066 , this issue can be closed now.

[Naktibalda]

I released new versions of all branches of phpunit-wrapper.

[TavoNiievez]

thanks!

Do I require 4.1.x-dev of Codeception/Codeception in Laravel module CI job or will you create a 4.1.14 tag?

[Naktibalda]

I released 4.1.14