Problem/Motivation

As a pre-requisite of #2876675: Allow symfony/event-dispatcher 4+ to be installed in Drupal 8 any maybe other issues like #3020303: Consider using Symfony flex to allow switching between major Symfony versions, we need to change the EventSubscriber/RedirectResponseSubscriberTest.

Instead of using using the Symfony\Component\EventDispatcher\EventDispatcher which would change in an incompatible way in Symfony >= 4.3 we must use our own Drupal\Component\EventDispatcher\ContainerAwareEventDispatcher as EventDispatcher;

Proposed resolution

Adjust the test.

Remaining tasks

Commit the patch.

User interface changes

none

API changes

none

Data model changes

none

Release notes snippet

none

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mkalkbrenner created an issue. See original summary.

mkalkbrenner’s picture

mkalkbrenner’s picture

Title: EventSubscriber/RedirectResponseSubscriberTest must not depend on Drupal Container » EventSubscriber/RedirectResponseSubscriberTest must not use the Symfony EventDispatcher but Drupal's
mkalkbrenner’s picture

.

hchonov’s picture

Status: Needs review » Reviewed & tested by the community

Thank you.

catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed 5c98790 and pushed to 8.8.x. Thanks!

  • catch committed 5c98790 on 8.8.x
    Issue #3081582 by mkalkbrenner: EventSubscriber/...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.