Modify schema update to now simply delete the whole db

[Nek-]

This is about #195

Q	A
BC Break?	yes
new feature?	no
bug fix?	yes
test pass?	not testable with phpspec (the old method was not either)
fixed ticket?	#195

Please notice that I could make another tag like @reset-database which will not break the compatibiity. (it breaks on schema generation, after this patch you need to generate the schema before your test suite)

[vincentchalamon]

This purger requires to build schema manually before running tests. To prevent this, you can reset schema once, maybe something like:

/**
 * @var bool
 */
private $databaseReady = false;

// ...

if (!$this->databaseReady) {
    $metadata = $entityManager->getMetadataFactory()->getAllMetadata();
    $tool = new SchemaTool($this->entityManager);
    $tool->dropSchema($metadata);
    $tool->createSchema($metadata);
    $this->databaseReady = true;
}

[Nek-]

I understand it's a bcbreak. But IMO the good thing to do is to build schema in a different task than purging the database.

It's possible the database is already good when you run your tests on your computer. In this case rebuilding the database is not desired.

[Nek-]

Please note also that your change is a bc break too as it suggest the database is build only once, and if the test needs a real schema reset the second time, it will not be executed.

[vincentchalamon]

OK so in that case, please remove useless use Doctrine\ORM\Tools\SchemaTool; 😉

[ioleo]

Is this going to be merged?