adding regression test. creating fake remote repo

src/Repos/FakeLogsRepo.php

@@ -5,8 +5,8 @@
 
 class FakeLogsRepo implements LogsRepoInterface
 {
-    private $config;
-    private $logs;
+    protected $config;
+    protected $logs;
 
     public function __construct(array $config)
     {

src/Repos/FakeRemoteLogsRepo.php

@@ -1,13 +1,9 @@
 <?php namespace MathiasGrimm\LaravelLogKeeper\Repos;
 
-use Carbon\Carbon;
-use MathiasGrimm\LaravelLogKeeper\Support\LogUtil;
+use Exception;
 
 class FakeRemoteLogsRepo extends FakeLogsRepo
 {
-    private $config;
-    private $logs;
-
     public function __construct(array $config)
     {
         parent::__construct($config);

src/Repos/RemoteLogsRepo.php

@@ -1,5 +1,6 @@
 <?php namespace MathiasGrimm\LaravelLogKeeper\Repos;
 
+use Exception;
 use \Storage;
 use MathiasGrimm\LaravelLogKeeper\Support\LogUtil;
 
@@ -15,7 +16,7 @@ public function __construct(array $config)
         $this->config = $config;
 
         if ($this->config['enabled_remote'] && !$this->config['remote_disk']) {
-            throw new \Exception("remote_disk not configured for Laravel Log Keeper");
+            throw new Exception("remote_disk not configured for Laravel Log Keeper");
         }
 
         $this->localLogPath = storage_path('logs');
@@ -57,7 +58,7 @@ public function getCompressed()
 
     public function compress($log, $compressedName)
     {
-        throw new \Exception("Method not implemented yet");
+        throw new Exception("Method not implemented yet");
         // TODO: Implement compress() method.
     }
 

tests/Repos/FakeRemoteLogsRepoTest.php

@@ -1,16 +1,26 @@
 <?php
 
-
-use MathiasGrimm\LaravelLogKeeper\Repos\FakeLogsRepo;
+use MathiasGrimm\LaravelLogKeeper\Repos\FakeRemoteLogsRepo;
 
 class FakeRemoteLogsRepoTest extends TestCase
 {
-    private function getRepo()
+    /**
+     * @test
+     */
+    public function it_throws_exception_when_enable_remote_is_true_and_remote_disk_is_null()
     {
         $config = config('laravel-log-keeper');
-        $repo   = new FakeLogsRepo($config);
 
-        return $repo;
+        $config['enabled_remote'] = true;
+        $config['remote_disk'   ] = null;
+
+        try {
+            $repo = new FakeRemoteLogsRepo($config);
+            $this->fail('It should not get here');
+        } catch (Exception $e) {
+            $this->assertSame('remote_disk not configured for Laravel Log Keeper', $e->getMessage());
+        }
+
     }
 
 

tests/Services/LogKeeperServiceTest.php

@@ -226,5 +226,4 @@ public function it_does_nothing_remotely_if_enabled_remote_is_false()
         $this->assertSame(["laravel-today-{$today->toDateString()}.log"], $logs);
         $this->assertSame([], $remoteRepo->getCompressed());
     }
-
-}
+}