Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[WIP] Adding a testing/database cookbook #2407

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Apr 26, 2013
Merged

[WIP] Adding a testing/database cookbook #2407

merged 2 commits into from
Apr 26, 2013

Conversation

Sgoettschkes
Copy link
Contributor

This PR adds a new cookbook for testing with a database involved.

Q A
Doc fix? no
New docs? yes
Applies to 2.0+
Fixed tickets #480

This PR is not ready for merge yet

}
}

As the ``ObjectManager`` gets injected into the class through the constructor, it's
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use some api link here

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@wouterj API links don't work for Doctrine interfaces

->method('getBonus')
->will($this->returnValue(1100));

// Now, mock tthe repository so it returns the mock of the employee
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo: the

weaverryan added a commit that referenced this pull request Apr 26, 2013
[WIP] Adding a testing/database cookbook
@weaverryan weaverryan merged commit 53174e0 into symfony:2.0 Apr 26, 2013
@weaverryan
Copy link
Member

Hey Sebastian!

I've merged this in now, since it already solves much of #480. I only had minor tweaks at sha: 0506361. This is a very nice entry.

Thanks!

@Sgoettschkes Sgoettschkes deleted the issue480 branch April 27, 2013 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants