Skip to content

Add Doctrine\Tests in composer autoload#401

Closed
cedriclombardot wants to merge 1 commit intodoctrine:masterfrom
cedriclombardot:patch-1
Closed

Add Doctrine\Tests in composer autoload#401
cedriclombardot wants to merge 1 commit intodoctrine:masterfrom
cedriclombardot:patch-1

Conversation

@cedriclombardot
Copy link

No description provided.

@travisbot
Copy link

This pull request fails (merged a7a7e4e into 34ac207).

@henrikbjorn
Copy link

Bad idea, use a bootstrap.php file instead that can add it to the autoloader object returned from requireing the autoload.php file.

@Ocramius
Copy link
Member

@cedriclombardot the test suite is not designed to be re-used. I'd not allow this...

@cedriclombardot
Copy link
Author

@Ocramius
Copy link
Member

@cedriclombardot you actually shouldn't reuse tests of the suite, since no stability is guaranteed. The DriverMock is quite minimal and you can simply copy it over. I am not a fan of copy/paste, but do not rely on ORM's test suite.

@cedriclombardot
Copy link
Author

Thanks

@beberlei
Copy link
Member

@cedriclombardot the Tests folder is not public API, please don't rely on it and copy paste the code.

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.

5 participants