symfony#37180 Added test for stripping collection from dsn

Joe Bennett · Joe Bennett · commit e8c54dd53eab · 2020-06-15T16:31:04.000+10:00
diff --git a/src/Symfony/Component/Lock/Tests/Store/MongoDbStoreTest.php b/src/Symfony/Component/Lock/Tests/Store/MongoDbStoreTest.php
@@ -147,10 +147,16 @@ public function testCollectionDsnPrecedence()
         $client = self::getMongoClient();
 
         $store = new MongoDbStore('mongodb://localhost/test?collection=lock_dsn', ['collection' => 'lock_option']);
-        $r = new \ReflectionObject($store);
-        $p = $r->getProperty('options');
-        $p->setAccessible(true);
-        $options = $p->getValue($store);
+        $storeReflection = new \ReflectionObject($store);
+
+        $uriProperty = $storeReflection->getProperty('uri');
+        $uriProperty->setAccessible(true);
+        $uri = $uriProperty->getValue($store);
+        $this->assertSame('mongodb://localhost/test', $uri);
+
+        $optionsProperty = $storeReflection->getProperty('options');
+        $optionsProperty->setAccessible(true);
+        $options = $optionsProperty->getValue($store);
         $this->assertSame('lock_dsn', $options['collection']);
     }
 
@@ -159,10 +165,10 @@ public function testDatabaseDsnPrecedence()
         $client = self::getMongoClient();
 
         $store = new MongoDbStore('mongodb://localhost/test_dsn', ['database' => 'test_option']);
-        $r = new \ReflectionObject($store);
-        $p = $r->getProperty('options');
-        $p->setAccessible(true);
-        $options = $p->getValue($store);
+        $storeReflection = new \ReflectionObject($store);
+        $optionsProperty = $storeReflection->getProperty('options');
+        $optionsProperty->setAccessible(true);
+        $options = $optionsProperty->getValue($store);
         $this->assertSame('test_dsn', $options['database']);
     }
 }
