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

Fix: Entity test #366

Merged
merged 13 commits into from
Feb 6, 2015
Merged

Fix: Entity test #366

merged 13 commits into from
Feb 6, 2015

Conversation

localheinz
Copy link
Member

This PR

  • reduces visibility of a property in a test to the maximum needed
  • adds a doc-block
  • prevents repetition
  • asserts strict

@Ocramius Ocramius self-assigned this Feb 6, 2015
Ocramius added a commit that referenced this pull request Feb 6, 2015
@Ocramius
Merge pull request #366 from localheinz/fix/entity-test
592d6d1 
Fix: Entity test
@Ocramius Ocramius merged commit 592d6d1 into zendframework:master Feb 6, 2015
@Ocramius
Copy link
Member

Ocramius commented Feb 6, 2015

@localheinz merged, thanks!

@localheinz localheinz deleted the fix/entity-test branch February 6, 2015 11:51
@localheinz localheinz mentioned this pull request Feb 6, 2015
Merged
1 task
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@Ocramius Ocramius

Labels
low priority
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@localheinz @Ocramius