remove event listener for testing purposes

Author: marickvantuil

src/Sender.php

@@ -4,7 +4,6 @@
 
 use Illuminate\Mail\Mailer;
 use Illuminate\Mail\Message;
-use Illuminate\Support\Facades\Event;
 
 class Sender
 {
@@ -21,10 +20,6 @@ public function send(Email $email)
 
         $email->markAsSending();
 
-        if (app()->runningUnitTests()) {
-            Event::dispatch('before.send');
-        }
-
         $this->getMailerInstance()->send([], [], function (Message $message) use ($email) {
             $this->buildMessage($message, $email);
         });

tests/RetryFailedEmailsCommandTest.php

@@ -13,15 +13,13 @@ function setUp()
         parent::setUp();
 
         $this->app['config']['laravel-database-emails.attempts'] = 3;
-
-        Event::listen('before.send', function() {
-            throw new \Exception('Simulating some random error');
-        });
     }
 
     /** @test */
     function an_email_cannot_be_reset_if_the_max_attempt_count_has_not_been_reached()
     {
+        $this->app['config']['mail.driver'] = 'does-not-exist';
+
         $this->sendEmail();
 
         $this->artisan('email:send');

tests/SendEmailsCommandTest.php

@@ -46,33 +46,17 @@ function an_email_should_not_be_sent_once_it_is_marked_as_sent()
         $this->assertEquals($firstSend, $email->fresh()->getSendDate());
     }
 
-    /** @test */
-    function an_email_should_be_locked_so_overlapping_cronjobs_cannot_send_an_already_processing_email()
-    {
-        $email = $this->sendEmail();
-
-        Event::listen('before.send', function () {
-            $this->artisan('email:send');
-        });
-
-        $this->artisan('email:send');
-
-        $this->assertEquals(1, $email->fresh()->getAttempts());
-    }
-
     /** @test */
     function if_an_email_fails_to_be_sent_it_should_be_logged_in_the_database()
     {
-        $email = $this->sendEmail();
+        $this->app['config']['mail.driver'] = 'does-not-exist';
 
-        Event::listen('before.send', function () {
-            throw new \Exception('Simulating some random error');
-        });
+        $email = $this->sendEmail();
 
         $this->artisan('email:send');
 
         $this->assertTrue($email->fresh()->hasFailed());
-        $this->assertContains('Simulating some random error', $email->fresh()->getError());
+        $this->assertContains('Driver [does-not-exist] not supported.', $email->fresh()->getError());
     }
 
     /** @test */
@@ -109,9 +93,7 @@ function an_email_should_never_be_sent_before_its_scheduled_date()
     /** @test */
     function emails_will_be_sent_until_max_try_count_has_been_reached()
     {
-        Event::listen('before.send', function () {
-            throw new \Exception('Simulating some random error');
-        });
+        $this->app['config']['mail.driver'] = 'does-not-exist';
 
         $this->sendEmail();
         $this->assertCount(1, (new Store)->getQueue());