Problem/Motivation

Symfony PR 12131 got merged into 2.6, ContainerAwareEventDispatcherTest should use AbstractEventDispatcherTest in order to inherit future enhancements to the upstream base class.

Proposed resolution

Copy over CompiledEventDispacherTest from Symfony PR 12521. Remove it once/if CompiledEventDispacher is merged into Symfony 2.7.

Remaining tasks

User interface changes

API changes

Follow-up to #2377281: Upgrade to Symfony 2.6 stable

CommentFileSizeAuthor
#1 2385699-use-AbstractEventDispatcherTest.diff19.45 KBznerol

Comments

znerol’s picture

znerol commented
Status: Fixed » Needs review
StatusFileSize
new 2385699-use-AbstractEventDispatcherTest.diff19.45 KB
dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

+1

Due to the issue summary its perfectly valid to use a different codestyle here.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Testing code is unfrozen during beta. Committed 25db629 and pushed to 8.0.x. Thanks!

  • alexpott committed 25db629 on 8.0.x 
    Issue #2385699 by znerol: Use upstream AbstractEventDispatcherTest as...

Status: Fixed » Closed (fixed)

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