don't fail if a configured directory doesn't exist (spatie#23)

* don't fail if a configured directory doesn't exist

* styleci

Author: mariavilaro

src/DirectoryCleanupCommand.php

@@ -2,6 +2,7 @@
 
 namespace Spatie\DirectoryCleanup;
 
+use File;
 use Illuminate\Console\Command;
 
 class DirectoryCleanupCommand extends Command
@@ -17,7 +18,9 @@ public function handle()
         $directories = collect(config('laravel-directory-cleanup.directories'));
 
         collect($directories)->each(function ($config, $directory) {
-            $this->deleteFilesIfOlderThanMinutes($directory, $config['deleteAllOlderThanMinutes']);
+            if (File::isDirectory($directory)) {
+                $this->deleteFilesIfOlderThanMinutes($directory, $config['deleteAllOlderThanMinutes']);
+            }
         });
 
         $this->comment('All done!');

tests/DirectoryCleanupTest.php

@@ -105,6 +105,27 @@ public function it_can_cleanup_the_directories_specified_in_the_config_file_but_
         }
     }
 
+    /** @test */
+    public function it_doesnt_fail_if_a_configured_dir_doesnt_exist()
+    {
+        $directories[$this->getTempDirectory('nodir', false)] = [
+            'deleteAllOlderThanMinutes' => 3,
+        ];
+
+        $existingDirectory = $this->getTempDirectory(1, true);
+        $directories[$existingDirectory] = [
+            'deleteAllOlderThanMinutes' => 3,
+        ];
+
+        $this->createFile("{$existingDirectory}/5MinutesOld.txt", 5);
+
+        $this->app['config']->set('laravel-directory-cleanup', compact('directories'));
+
+        $this->artisan('clean:directories');
+
+        $this->assertFileNotExists("{$existingDirectory}/5MinutesOld.txt");
+    }
+
     protected function createFile(string $fileName, int $ageInMinutes)
     {
         touch($fileName, Carbon::now()->subMinutes($ageInMinutes)->subSeconds(5)->timestamp);