Fix: Do not declare test classes in global namespace

Author: localheinz

tests/unit/Framework/Constraint/LogicalXorTest.php

@@ -7,12 +7,10 @@
  * For the full copyright and license information, please view the LICENSE
  * file that was distributed with this source code.
  */
-namespace Framework\Constraint;
+namespace PHPUnit\Framework\Constraint;
 
 use function array_fill;
 use function array_map;
-use PHPUnit\Framework\Constraint\Constraint;
-use PHPUnit\Framework\Constraint\LogicalXor;
 use PHPUnit\Framework\TestCase;
 
 /**

tests/unit/Framework/MockObject/Builder/InvocationMockerTest.php

@@ -8,15 +8,17 @@
  * For the full copyright and license information, please view the LICENSE
  * file that was distributed with this source code.
  */
+namespace PHPUnit\Framework\MockObject;
+
+use ClassWithAllPossibleReturnTypes;
+use Foo;
 use PHPUnit\Framework\Constraint\IsEqual;
 use PHPUnit\Framework\MockObject\Builder\InvocationMocker;
-use PHPUnit\Framework\MockObject\IncompatibleReturnValueException;
-use PHPUnit\Framework\MockObject\InvocationHandler;
-use PHPUnit\Framework\MockObject\Matcher;
 use PHPUnit\Framework\MockObject\Stub\ReturnSelf;
 use PHPUnit\Framework\MockObject\Stub\ReturnStub;
 use PHPUnit\Framework\TestCase;
 use PHPUnit\TestFixture\MockObject\ClassWithImplicitProtocol;
+use stdClass;
 
 /**
  * @covers \PHPUnit\Framework\MockObject\Builder\InvocationMocker

tests/unit/Framework/MockObject/GeneratorTest.php

@@ -7,9 +7,26 @@
  * For the full copyright and license information, please view the LICENSE
  * file that was distributed with this source code.
  */
-use PHPUnit\Framework\MockObject\Generator;
-use PHPUnit\Framework\MockObject\MockObject;
+namespace PHPUnit\Framework\MockObject;
+
+use function class_exists;
+use function md5;
+use function method_exists;
+use function microtime;
+use AbstractMockTestClass;
+use AbstractTrait;
+use AnInterface;
+use AnInterfaceWithReturnType;
+use ClassWithVariadicArgumentMethod;
+use Countable;
+use Exception;
+use ExceptionWithThrowable;
+use FinalClass;
+use InterfaceWithSemiReservedMethodName;
 use PHPUnit\Framework\TestCase;
+use SingletonClass;
+use stdClass;
+use Throwable;
 
 /**
  * @covers \PHPUnit\Framework\MockObject\Generator
@@ -62,7 +79,7 @@ public function testGetMockThrowsExceptionWithNonExistingClass(): void
     {
         $this->expectException(\PHPUnit\Framework\MockObject\RuntimeException::class);
 
-        $this->assertFalse(\class_exists('Tux'));
+        $this->assertFalse(class_exists('Tux'));
 
         $this->generator->getMock('Tux', [], [], '', true, true, false, true, false, null, false);
     }
@@ -71,7 +88,7 @@ public function testGetMockThrowsExceptionWithNonExistingClasses(): void
     {
         $this->expectException(\PHPUnit\Framework\MockObject\RuntimeException::class);
 
-        $this->assertFalse(\class_exists('Tux'));
+        $this->assertFalse(class_exists('Tux'));
 
         $this->generator->getMock(['Tux', false], [], [], '', true, true, false, true, false, null, false);
     }
@@ -87,7 +104,7 @@ public function testGetMockCanCreateNonExistingFunctions(): void
     {
         $mock = $this->generator->getMock(stdClass::class, ['testFunction']);
 
-        $this->assertTrue(\method_exists($mock, 'testFunction'));
+        $this->assertTrue(method_exists($mock, 'testFunction'));
     }
 
     public function testGetMockGeneratorThrowsException(): void
@@ -102,22 +119,22 @@ public function testGetMockBlacklistedMethodNamesPhp7(): void
     {
         $mock = $this->generator->getMock(InterfaceWithSemiReservedMethodName::class);
 
-        $this->assertTrue(\method_exists($mock, 'unset'));
+        $this->assertTrue(method_exists($mock, 'unset'));
         $this->assertInstanceOf(InterfaceWithSemiReservedMethodName::class, $mock);
     }
 
     public function testGetMockForAbstractClassDoesNotFailWhenFakingInterfaces(): void
     {
         $mock = $this->generator->getMockForAbstractClass(Countable::class);
 
-        $this->assertTrue(\method_exists($mock, 'count'));
+        $this->assertTrue(method_exists($mock, 'count'));
     }
 
     public function testGetMockForAbstractClassStubbingAbstractClass(): void
     {
         $mock = $this->generator->getMockForAbstractClass(AbstractMockTestClass::class);
 
-        $this->assertTrue(\method_exists($mock, 'doSomething'));
+        $this->assertTrue(method_exists($mock, 'doSomething'));
     }
 
     public function testGetMockForAbstractClassWithNonExistentMethods(): void
@@ -132,8 +149,8 @@ public function testGetMockForAbstractClassWithNonExistentMethods(): void
             ['nonexistentMethod']
         );
 
-        $this->assertTrue(\method_exists($mock, 'nonexistentMethod'));
-        $this->assertTrue(\method_exists($mock, 'doSomething'));
+        $this->assertTrue(method_exists($mock, 'nonexistentMethod'));
+        $this->assertTrue(method_exists($mock, 'doSomething'));
     }
 
     public function testGetMockForAbstractClassShouldCreateStubsOnlyForAbstractMethodWhenNoMethodsWereInformed(): void
@@ -167,8 +184,8 @@ public function testGetMockForTraitWithNonExistentMethodsAndNonAbstractMethods()
             ['nonexistentMethod']
         );
 
-        $this->assertTrue(\method_exists($mock, 'nonexistentMethod'));
-        $this->assertTrue(\method_exists($mock, 'doSomething'));
+        $this->assertTrue(method_exists($mock, 'nonexistentMethod'));
+        $this->assertTrue(method_exists($mock, 'doSomething'));
         $this->assertTrue($mock->mockableMethod());
         $this->assertTrue($mock->anotherMockableMethod());
     }
@@ -177,7 +194,7 @@ public function testGetMockForTraitStubbingAbstractMethod(): void
     {
         $mock = $this->generator->getMockForTrait(AbstractTrait::class);
 
-        $this->assertTrue(\method_exists($mock, 'doSomething'));
+        $this->assertTrue(method_exists($mock, 'doSomething'));
     }
 
     public function testGetMockForTraitWithNonExistantTrait(): void
@@ -226,7 +243,7 @@ public function testCanImplementInterfacesThatHaveMethodsWithReturnTypes(): void
 
     public function testCanConfigureMethodsForDoubleOfNonExistentClass(): void
     {
-        $className = 'X' . \md5(\microtime());
+        $className = 'X' . md5(microtime());
 
         $mock = $this->generator->getMock($className, ['someMethod']);
 
@@ -235,7 +252,7 @@ public function testCanConfigureMethodsForDoubleOfNonExistentClass(): void
 
     public function testCanInvokeMethodsOfNonExistentClass(): void
     {
-        $className = 'X' . \md5(\microtime());
+        $className = 'X' . md5(microtime());
 
         $mock = $this->generator->getMock($className, ['someMethod']);
 

tests/unit/Framework/MockObject/Matcher/ConsecutiveParametersTest.php

@@ -7,9 +7,12 @@
  * For the full copyright and license information, please view the LICENSE
  * file that was distributed with this source code.
  */
+namespace PHPUnit\Framework\MockObject;
+
 use PHPUnit\Framework\ExpectationFailedException;
 use PHPUnit\Framework\InvalidParameterGroupException;
 use PHPUnit\Framework\TestCase;
+use stdClass;
 
 /**
  * @small

tests/unit/Framework/MockObject/MockBuilderTest.php

@@ -7,7 +7,10 @@
  * For the full copyright and license information, please view the LICENSE
  * file that was distributed with this source code.
  */
-use PHPUnit\Framework\MockObject\MockBuilder;
+namespace PHPUnit\Framework\MockObject;
+
+use ACustomClassName;
+use Mockable;
 use PHPUnit\Framework\TestCase;
 
 /**