Fix bug in InjectTemplateListenerTest #5669

Closed
wants to merge 1 commit into
from

2 participants

@Xerkus

InjectTemplateListener in tests yields different results with
ModuleRouteListener::MODULE_NAMESPACE router parameter set.

Fix applies ModuleRouteListener to routematch parameters.

@Xerkus Xerkus Fix bug in InjectTemplateListenerTest
InjectTemplateListener in tests yields different results with
ModuleRouteListener::MODULE_NAMESPACE router parameter set.

Fix applies ModuleRouteListener to routematch parameters.
dd17019
@weierophinney
Zend Framework member

@Xerkus does this just add more tests for robustness?

Also, re-triggered the build, as it was made when an update to the CS checker made all the CS checks blow up.

@weierophinney
Zend Framework member

CS checks are still blowing up, but looks good. Scheduling for 2.2.6.

@weierophinney weierophinney added this to the 2.2.6 milestone Mar 3, 2014
@Xerkus

@weierophinney iirc, this change added more coverage for template listener when __NAMESPACE__ parameter is used and also changed existing tests to apply module route listener so they can assert on proper output.

@weierophinney weierophinney self-assigned this Mar 3, 2014
@weierophinney weierophinney added a commit that closed this pull request Mar 3, 2014
@weierophinney weierophinney Merge branch 'hotfix/5669'
Close #5669
20adb79
@weierophinney weierophinney added a commit that referenced this pull request Mar 3, 2014
@weierophinney weierophinney Merge branch 'hotfix/5669' into develop
Forward port #5669
1cab1a6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment