Skip to content
This repository was archived by the owner on Jan 31, 2020. It is now read-only.

BC break with aggregate hydrator #55

Closed
svycka opened this issue Feb 19, 2016 · 1 comment
Closed

BC break with aggregate hydrator #55

svycka opened this issue Feb 19, 2016 · 1 comment
Assignees
Labels
Milestone

Comments

@svycka
Copy link

svycka commented Feb 19, 2016

Just moving issue here because this is where BC break happens.
see zendframework/zend-hydrator#19

weierophinney added a commit to weierophinney/zend-stdlib that referenced this issue Feb 19, 2016
Per zendframework#55 (and originally reported as zendframework/zend-hydrator#19),
this patch does the following:

- Adds tests against the hydrate and extract methods to ensure that the
  event triggered is the zend-stdlib variant (as well as kept the tests
  for the zend-hydrator variants).
- Updated test expectations to check for `triggerEvent()` (and not
  `trigger()`), as the 1.1 version of zend-hydrator is now
  forwards-compatible with zend-eventmanager v3. This also required
  raising the minimum version of zend-hydrator to 1.1 so that tests do
  not need to be varied.
- Fixes the test assumptions in `testAdd()`, as attachment of the
  aggregate listener now calls the `attach()` method of the listener
  class, changing how the mock EM instance is called.
@weierophinney weierophinney added this to the 2.7.6 milestone Feb 19, 2016
@weierophinney weierophinney self-assigned this Feb 19, 2016
@weierophinney
Copy link
Member

Fixed with #56

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants