Merge pull request #53 from LukeeeeBennett/disable-cleanup

Disable cleanup artisan command if store_incoming_emails_for_days is INF

Author: mpociot

config/mailbox.php

@@ -29,6 +29,7 @@
      * The amount of days that incoming emails should be stored in your
      * application. You can use the cleanup artisan command to
      * delete all older inbound emails on a regular basis.
+     * Set to INF to disable the cleanup artisan command.
      */
     'store_incoming_emails_for_days' => 7,
 

src/Console/CleanEmails.php

@@ -18,6 +18,12 @@ public function handle()
 
         $maxAgeInDays = config('mailbox.store_incoming_emails_for_days');
 
+        if ($maxAgeInDays === INF)
+        {
+            $this->error($this->signature . ' is disabled because store_incoming_emails_for_days is set to INF.');
+            return 1;
+        }
+
         $cutOffDate = Carbon::now()->subDay($maxAgeInDays)->format('Y-m-d H:i:s');
 
         /** @var InboundEmail $modelClass */

tests/Console/CleanEmailsTest.php

@@ -40,4 +40,25 @@ public function it_can_clean_the_statistics()
 
         $this->assertCount(0, InboundEmail::where('created_at', '<', $cutOffDate)->get());
     }
+
+    /** @test */
+    public function it_errors_if_max_age_inf()
+    {
+        $this->app['config']->set('mailbox.store_incoming_emails_for_days', INF);
+
+        Collection::times(60)->each(function (int $index) {
+            InboundEmail::forceCreate([
+                'message' => Str::random(),
+                'created_at' => Carbon::now()->subDays($index)->startOfDay(),
+            ]);
+        });
+
+        $this->assertCount(60, InboundEmail::all());
+
+        $this->artisan('mailbox:clean')
+             ->expectsOutput('mailbox:clean is disabled because store_incoming_emails_for_days is set to INF.')
+             ->assertExitCode(1);
+
+        $this->assertCount(60, InboundEmail::all());
+    }
 }