changes in default transport factory

- set null transport if empty
- throw exception in neither dsn nor alias are set.

Author: makasim

DependencyInjection/EnqueueExtension.php

@@ -52,13 +52,6 @@ public function addTransportFactory(TransportFactoryInterface $transportFactory)
      */
     public function load(array $configs, ContainerBuilder $container)
     {
-        // enable null transport by default.
-        array_unshift($configs, [
-            'transport' => [
-                'default' => 'null://',
-            ],
-        ]);
-
         $config = $this->processConfiguration(new Configuration($this->factories), $configs);
 
         $loader = new YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));

Tests/Unit/DependencyInjection/EnqueueExtensionTest.php

@@ -9,8 +9,8 @@
 use Enqueue\Client\TraceableProducer;
 use Enqueue\Null\NullContext;
 use Enqueue\Null\Symfony\NullTransportFactory;
-use Enqueue\Symfony\TransportFactoryInterface;
 use Enqueue\Symfony\DefaultTransportFactory;
+use Enqueue\Symfony\TransportFactoryInterface;
 use Enqueue\Test\ClassExtensionTrait;
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
@@ -116,30 +116,6 @@ public function testShouldUseNullTransportAsDefaultWhenExplicitlyConfigured()
         );
     }
 
-    public function testShouldUseNullTransportAsDefaultConfiguredViaDSN()
-    {
-        $container = new ContainerBuilder();
-
-        $extension = new EnqueueExtension();
-        $extension->addTransportFactory(new NullTransportFactory());
-        $extension->addTransportFactory(new DefaultTransportFactory());
-
-        $extension->load([[
-            'transport' => [
-                'default' => 'null://',
-            ],
-        ]], $container);
-
-        self::assertEquals(
-            'enqueue.transport.default.context',
-            (string) $container->getAlias('enqueue.transport.context')
-        );
-        self::assertEquals(
-            'enqueue.transport.default_null.context',
-            (string) $container->getAlias('enqueue.transport.default.context')
-        );
-    }
-
     public function testShouldConfigureFooTransport()
     {
         $container = new ContainerBuilder();