Skip to content
This repository has been archived by the owner on May 1, 2019. It is now read-only.

Fix: Integration test for User controller fails because of lack of mocking #464

Merged
merged 3 commits into from
Mar 5, 2015

Conversation

localheinz
Copy link
Member

This PR

  • adds a failing assertion
  • mocks Mapper\User to fix the failing test
  • introduces a variable to avoid duplication

Follows #444.
Spotted in #462.

@localheinz localheinz self-assigned this Mar 5, 2015
@localheinz localheinz added the Bug label Mar 5, 2015
@localheinz localheinz force-pushed the fix/user-controller branch 2 times, most recently from bf79b2e to f4fde98 Compare March 5, 2015 08:41
@localheinz localheinz force-pushed the fix/user-controller branch from f4fde98 to 5fff953 Compare March 5, 2015 08:42
@localheinz
Copy link
Member Author

@Ocramius

Need to go now, will work on this later!

@localheinz localheinz force-pushed the fix/user-controller branch 3 times, most recently from 34c1016 to 37bcb86 Compare March 5, 2015 10:56
@localheinz localheinz force-pushed the fix/user-controller branch from 37bcb86 to b14ba58 Compare March 5, 2015 10:56
@localheinz localheinz changed the title [WIP] Fix: User controller can actually not be accessed, at least in test Fix: Integration test for User controller fails because of lack of mocking Mar 5, 2015
@localheinz localheinz assigned Ocramius and unassigned localheinz Mar 5, 2015
Ocramius added a commit that referenced this pull request Mar 5, 2015
Fix: Integration test for User controller fails because of lack of mocking
@Ocramius Ocramius merged commit bb778f3 into zendframework:master Mar 5, 2015
@localheinz localheinz deleted the fix/user-controller branch March 5, 2015 12:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants