Skip to content

PHPUnit testing and database isolation #9954

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.

Sign up for GitHub

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

Jump to bottom
Closed
dmaicher opened this issue Jun 22, 2018 · 3 comments
Closed

PHPUnit testing and database isolation #9954

dmaicher opened this issue Jun 22, 2018 · 3 comments
Labels
Doctrine hasPR A Pull Request has already been submitted for this issue. Testing
Milestone
2.8

Comments

@dmaicher
Copy link
Contributor

I'm wondering if it would be ok to update the documentation and mention my bundle as a reference to ease having isolated tests when it comes to database contents.

It could be mentioned on this page: https://symfony.com/doc/current/testing/database.html

This is the bundle:
https://github.com/dmaicher/doctrine-test-bundle
https://packagist.org/packages/dama/doctrine-test-bundle

It now has over 200.000 installs and seems to be working quite well 😊

Its also used on the symfony demo project since a while: https://github.com/symfony/demo/blob/master/composer.json#L32

WDYT? I believe it can be quite useful for many people.
@HeahDude HeahDude added this to the 2.8 milestone Jun 23, 2018
@javiereguiluz
Copy link
Member

I'm 👍 for mentioning this bundle. It solves a useful problem, it works great, we've tested it ourselves (in the Demo app), it's not too recent (it's tested enough), it's getting popular (lots of downloads), etc.

@javiereguiluz javiereguiluz mentioned this issue Jul 6, 2018
Merged
@javiereguiluz javiereguiluz added the hasPR A Pull Request has already been submitted for this issue. label Jul 6, 2018
javiereguiluz added a commit that referenced this issue Jul 9, 2018
@javiereguiluz
minor #10026 Mention the DoctrineTestBundle for tests that interact w…
94cef15 
…ith the database (javiereguiluz)

This PR was merged into the 2.8 branch.

Discussion
----------

Mention the DoctrineTestBundle for tests that interact with the database

This fixes #9954.

Commits
-------

219f617 Mention the DoctrineTestBundle for tests that interact with the database
@javiereguiluz
Copy link
Member

Fixed by #10026.

@dmaicher
Copy link
Contributor Author

dmaicher commented Jul 9, 2018

Thanks for fixing this so fast @javiereguiluz 👍 Did not get to it yet myself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Doctrine hasPR A Pull Request has already been submitted for this issue. Testing
Projects
None yet
Milestone
2.8
Development

No branches or pull requests
3 participants
@javiereguiluz @dmaicher @HeahDude