Restore reverse lookup determinism

In 96c2ebf, it looks like I wrongly
assumed I wouldn't be able to allow having 2 classes of the same type
in the type registry without also allowing having 2 objects of the same
type. array_search can perfectly tell the difference between both
situations, so let us continue forbidding that last part, it will allow
us to make sure a given type matches exactly one name.

src/Types/Exception/TypeAlreadyRegistered.php

@@ -0,0 +1,25 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Doctrine\DBAL\Types\Exception;
+
+use Doctrine\DBAL\Types\Type;
+use Exception;
+
+use function get_debug_type;
+use function spl_object_hash;
+use function sprintf;
+
+/** @psalm-immutable */
+final class TypeAlreadyRegistered extends Exception implements TypesException
+{
+    public static function new(Type $type): self
+    {
+        return new self(sprintf(
+            'Type of the class %s@%s is already registered.',
+            get_debug_type($type),
+            spl_object_hash($type),
+        ));
+    }
+}

src/Types/TypeRegistry.php

@@ -5,6 +5,7 @@
 namespace Doctrine\DBAL\Types;
 
 use Doctrine\DBAL\Exception;
+use Doctrine\DBAL\Types\Exception\TypeAlreadyRegistered;
 use Doctrine\DBAL\Types\Exception\TypeNotFound;
 use Doctrine\DBAL\Types\Exception\TypeNotRegistered;
 use Doctrine\DBAL\Types\Exception\TypesAlreadyExists;
@@ -71,6 +72,10 @@ public function register(string $name, Type $type): void
             throw TypesAlreadyExists::new($name);
         }
 
+        if (array_search($type, $this->instances, true) !== false) {
+            throw TypeAlreadyRegistered::new($type);
+        }
+
         $this->instances[$name] = $type;
     }
 

tests/Types/Exception/TypeAlreadyRegisteredTest.php

@@ -0,0 +1,22 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Doctrine\DBAL\Tests\Types\Exception;
+
+use Doctrine\DBAL\Types\Exception\TypeAlreadyRegistered;
+use Doctrine\DBAL\Types\Type;
+use PHPUnit\Framework\TestCase;
+
+class TypeAlreadyRegisteredTest extends TestCase
+{
+    public function testNew(): void
+    {
+        $exception = TypeAlreadyRegistered::new(Type::getType('string'));
+
+        self::assertMatchesRegularExpression(
+            '/Type of the class Doctrine\\\DBAL\\\Types\\\StringType@([0-9a-zA-Z]+) is already registered./',
+            $exception->getMessage(),
+        );
+    }
+}

tests/Types/TypeRegistryTest.php

@@ -95,11 +95,8 @@ public function testRegisterWithAlreadyRegisteredInstance(): void
         $newType = new TextType();
 
         $this->registry->register('type1', $newType);
+        $this->expectException(Exception::class);
         $this->registry->register('type2', $newType);
-        self::assertSame(
-            $this->registry->get('type1'),
-            $this->registry->get('type2'),
-        );
     }
 
     public function testOverride(): void