Merge pull request #358 from php-enqueue/bundle-fix-default-transport-and-dsn

[bundle] DefaultTransportFactory should accept DSN like foo:

Author: makasim

pkg/enqueue/Symfony/DefaultTransportFactory.php

@@ -58,7 +58,7 @@ public function addConfiguration(ArrayNodeDefinition $builder)
                     }
 
                     if (is_string($v)) {
-                        return false !== strpos($v, '://') || false !== strpos($v, 'env_') ?
+                        return false !== strpos($v, ':') || false !== strpos($v, 'env_') ?
                             ['dsn' => $v] :
                             ['alias' => $v];
                     }

pkg/enqueue/Tests/Symfony/DefaultTransportFactoryTest.php

@@ -72,6 +72,24 @@ public function testShouldAllowAddConfigurationAsDsn()
         $this->assertEquals(['dsn' => 'dsn://'], $config);
     }
 
+    /**
+     * @see https://github.com/php-enqueue/enqueue-dev/issues/356
+     *
+     * @group bug
+     */
+    public function testShouldAllowAddConfigurationAsDsnWithoutSlashes()
+    {
+        $transport = new DefaultTransportFactory();
+        $tb = new TreeBuilder();
+        $rootNode = $tb->root('foo');
+
+        $transport->addConfiguration($rootNode);
+        $processor = new Processor();
+        $config = $processor->process($tb->buildTree(), ['dsn:']);
+
+        $this->assertEquals(['dsn' => 'dsn:'], $config);
+    }
+
     public function testShouldSetNullTransportByDefault()
     {
         $transport = new DefaultTransportFactory();