Inaccuracies in cookbook entry "How to test Doctrine Repositories"

[keymaster]

The reader is told to inherit from OrmTestCase at the following path:

use Doctrine\Tests\OrmTestCase;

However, there is no such class in the standard Symfony distribution (2.05).

Is there perhaps some additional doctrine bundle which needs to be installed - if so, it should be mentioned?

Also, the entry should probably also describe how to do this with xml metadata, not just annotations.

[stof]

@fabpot the issue here is that the Doctrine tests are not included in the tarball including vendors

@beberlei should the base class be in the actual Doctrine code to make it easier to use it ? the test namespace of the vendors generally does not have an autoloader registered (and it would avoid such issue when getting rid of vendor tests)

[beberlei]

Doctrine\Tests\OrmTestCase is not part of the code we distribute to our users. Its part of the tests, no BC guarantee or whatsoever. So i wouldnt advise to use this at all.

[stof]

Ok, then the Symfony doc should be changed.

[weaverryan]

So what do you think is better, showing the example with a big warning or removing entirely? Personally, I think that repo testing should be all "functional" and not unit.

[stof]

I would vote for removing it

[weaverryan]

I've removed the section about unit testing the repository as I agree that we shouldn't recommend doing something that's not guaranteed for BC.

Thanks!