Merge pull request #12 from MacPaw/fix/quote_schema_special_chars

fix: quote schema special chars

Author: alekseytupichenkov

src/Doctrine/SchemaConnection.php

@@ -50,6 +50,8 @@ private function ensurePostgreSql(): void
     private function applySearchPath(string $schema): void
     {
         if ($this->_conn !== null) {
+            $schema = $this->getDatabasePlatform()->quoteIdentifier($schema);
+
             $this->_conn->exec('SET search_path TO ' . $schema);
         }
     }

tests/Doctrine/SchemaConnectionTest.php

@@ -23,7 +23,7 @@ public function testConnectSetsSearchPath(): void
 
         $driverConnection->expects($this->once())
             ->method('exec')
-            ->with('SET search_path TO test_schema');
+            ->with('SET search_path TO "test_schema"');
 
         $driver = $this->createMock(Driver::class);
 
@@ -49,6 +49,38 @@ public function testConnectSetsSearchPath(): void
         self::assertTrue($result);
     }
 
+    public function testConnectSetsSearchPathWithSpecialChars(): void
+    {
+        $driverConnection = $this->createMock(DriverConnection::class);
+
+        $driverConnection->expects($this->once())
+            ->method('exec')
+            ->with('SET search_path TO "test-schema/foo"');
+
+        $driver = $this->createMock(Driver::class);
+
+        $driver->method('connect')->willReturn($driverConnection);
+
+        $platform = new PostgreSQLPlatform();
+        $connection = $this->getMockBuilder(SchemaConnection::class)
+            ->setConstructorArgs([[], $driver, new Configuration(), new EventManager()])
+            ->onlyMethods(['getDatabasePlatform', 'fetchOne'])
+            ->getMock();
+
+        $connection->method('getDatabasePlatform')->willReturn($platform);
+        $connection->method('fetchOne')->willReturn(true);
+
+        $resolver = new BaggageSchemaResolver();
+
+        $resolver->setSchema('test-schema/foo');
+
+        SchemaConnection::setSchemaResolver($resolver);
+
+        $result = $connection->connect();
+
+        self::assertTrue($result);
+    }
+
     public function testConnectSkipsWhenNoSchema(): void
     {
         $driverConnection = $this->createMock(DriverConnection::class);