symfony
diff --git a/‎src/Symfony/Component/Notifier/Tests/Transport/DsnTest.php
Lines changed: 21 additions & 18 deletions b/‎src/Symfony/Component/Notifier/Tests/Transport/DsnTest.php
Lines changed: 21 additions & 18 deletions
@@ -208,28 +208,37 @@ public function getOptionProvider(): iterable
         ];
     }
 
-    public function testGetRequiredOptionGetsOptionIfSet()
+    /**
+     * @dataProvider getRequiredOptionProvider
+     */
+    public function testGetRequiredOption(string $expectedValue, string $options, string $option)
     {
-        $options = ['with_value' => 'some value'];
-        $dsn = new Dsn('scheme', 'localhost', 'u$er', 'pa$s', '8000', $options, '/channel');
+        $dsn = Dsn::fromString(sprintf('scheme://localhost?%s', $options));
 
-        $this->assertSame('some value', $dsn->getRequiredOption('with_value'));
+        $this->assertSame($expectedValue, $dsn->getRequiredOption($option));
     }
 
-    public function testGetRequiredOptionGetsOptionIfValueIsZero()
+    public function getRequiredOptionProvider(): iterable
     {
-        $options = ['timeout' => 0];
-        $dsn = new Dsn('scheme', 'localhost', 'u$er', 'pa$s', '8000', $options, '/channel');
+        yield [
+            'value',
+            'with_value=value',
+            'with_value'
+        ];
 
-        $this->assertSame(0, $dsn->getRequiredOption('timeout'));
+        yield [
+            '0',
+            'timeout=0',
+            'timeout'
+        ];
     }
 
     /**
      * @dataProvider getRequiredOptionThrowsMissingRequiredOptionExceptionProvider
      */
-    public function testGetRequiredOptionThrowsMissingRequiredOptionException(string $expectedExceptionMessage, array $options, string $option)
+    public function testGetRequiredOptionThrowsMissingRequiredOptionException(string $expectedExceptionMessage, string $options, string $option)
     {
-        $dsn = new Dsn('scheme', 'localhost', 'u$er', 'pa$s', '8000', $options, '/channel');
+        $dsn = Dsn::fromString(sprintf('scheme://localhost?%s', $options));
 
         $this->expectException(MissingRequiredOptionException::class);
         $this->expectExceptionMessage($expectedExceptionMessage);
@@ -241,20 +250,14 @@ public function getRequiredOptionThrowsMissingRequiredOptionExceptionProvider():
     {
         yield [
             'The option "foo_bar" is required but missing.',
-            ['with_value' => 'some value'],
+            'with_value=value',
             'foo_bar',
         ];
 
         yield [
             'The option "with_empty_string" is required but missing.',
-            ['with_empty_string' => ''],
+            'with_empty_string=',
             'with_empty_string',
         ];
-
-        yield [
-            'The option "with_null" is required but missing.',
-            ['with_null' => null],
-            'with_null',
-        ];
     }
 }