move WebTestCase to "Functional" namespace instead of "Units"

[FlorianLB]

WebTestCase is a base class for functional tests not units one.

We can keep the old class for BC or just remove it.

[jubianchi]

To avoid BC break, you can move the class and simply create a class_alias

[FlorianLB]

I don't see how to use class_alias in a PSR-0 autoload context. An idea ?

[jubianchi]

Forget about PSR-0

No seriously : I would add the class_aliascall somewhere here https://github.com/atoum/AtoumBundle/blob/master/AtoumAtoumBundle.php#L21

The call would look like:

<?php

if(false === class_exists('atoum\AtoumBundle\Test\Units\WebTestCase')) {
    class_alias('atoum\AtoumBundle\Test\Functional\WebTestCase', 'atoum\AtoumBundle\Test\Units\WebTestCase');
}

So that, the alias will be automatically created at when the bundle boots and everything should work fine with any autoloader working with PSR-0.

The only drawback I can think of right now is that even the user won't use the WebTestCase the class will still be loaded as class_alias calls the autoloader. But I don't think it's really important to worry about that ;)

[FlorianLB]

It can't work in a CLI/atoum context. The kernel (and bundles) is booted during the test execution not before like in a SF2 app.

[FlorianLB]

(Using bin/atoum, not the SF2 command)