wip

Author: mpociot

src/Drivers/DriverInterface.php

@@ -0,0 +1,8 @@
+<?php
+
+namespace BeyondCode\Mailbox\Drivers;
+
+interface DriverInterface
+{
+    public function register();
+}

src/Drivers/Log.php

@@ -6,9 +6,14 @@
 use BeyondCode\Mailbox\Facades\Mailbox;
 use Illuminate\Log\Events\MessageLogged;
 
-class Log
+class Log implements DriverInterface
 {
 
+    public function register()
+    {
+        app('events')->listen(MessageLogged::class, [$this, 'processLog']);
+    }
+
     public function processLog(MessageLogged $log)
     {
         $email = InboundEmail::fromMessage($log->message);

src/Drivers/Mailgun.php

@@ -0,0 +1,17 @@
+<?php
+
+namespace BeyondCode\Mailbox\Drivers;
+
+use Illuminate\Support\Facades\Route;
+use BeyondCode\Mailbox\Http\Controllers\MailgunController;
+
+class Mailgun implements DriverInterface
+{
+
+    public function register()
+    {
+        Route::prefix(config('mailbox.path'))->group(function () {
+            Route::post('/mailgun/mime', MailgunController::class);
+        });
+    }
+}

src/MailboxManager.php

@@ -3,6 +3,8 @@
 namespace BeyondCode\Mailbox;
 
 use Illuminate\Support\Manager;
+use BeyondCode\Mailbox\Drivers\Log;
+use BeyondCode\Mailbox\Drivers\Mailgun;
 
 class MailboxManager extends Manager
 {
@@ -12,8 +14,18 @@ public function mailbox($name = null)
         return $this->driver($name);
     }
 
+    public function createLogDriver()
+    {
+        return new Log;
+    }
+
+    public function createMailgunDriver()
+    {
+        return new Mailgun;
+    }
+
     public function getDefaultDriver()
     {
-        return $this->app['config']['mailbox.default'];
+        return $this->app['config']['mailbox.driver'];
     }
 }

src/MailboxServiceProvider.php

@@ -21,7 +21,7 @@ public function boot()
             ], 'migrations');
         }
 
-        $this->registerRoutes();
+        $this->registerDriver();
     }
 
     /**
@@ -34,21 +34,10 @@ public function register()
         $this->app->singleton('mailbox', function () {
             return new MailboxRouter($this->app);
         });
-
-        if (config('mail.driver') === 'log') {
-            $this->registerLogDriver();
-        }
-    }
-
-    protected function registerLogDriver()
-    {
-        $this->app['events']->listen(MessageLogged::class, [new Log, 'processLog']);
     }
 
-    protected function registerRoutes()
+    protected function registerDriver()
     {
-        Route::prefix(config('mailbox.path'))->group(function () {
-            Route::post('/mailgun/mime', MailgunController::class);
-        });
+        (new MailboxManager($this->app))->mailbox()->register();
     }
 }

tests/Controllers/MailgunTest.php

@@ -6,6 +6,12 @@
 
 class MailgunTest extends TestCase
 {
+    protected function getEnvironmentSetUp($app)
+    {
+        parent::getEnvironmentSetUp($app);
+
+        $app['config']['mailbox.driver'] = 'mailgun';
+    }
 
     /** @test */
     public function it_verifies_mailgun_signatures()

tests/Drivers/LogTest.php

@@ -11,11 +11,12 @@
 class LogTest extends TestCase
 {
 
-    protected function getPackageProviders($app)
+    protected function getEnvironmentSetUp($app)
     {
-        $app['config']['mail.driver'] = 'log';
+        parent::getEnvironmentSetUp($app);
 
-        return parent::getPackageProviders($app);
+        $app['config']['mail.driver'] = 'log';
+        $app['config']['mailbox.driver'] = 'log';
     }
 
     /** @test */