fix tests

Author: streamtw

tests/LfmItemTest.php

@@ -96,6 +96,7 @@ public function testType()
         $this->lfm_path->shouldReceive('isDirectory')->andReturn(false);
         $this->lfm_path->shouldReceive('mimeType')->andReturn('application/plain');
         $this->lfm_path->shouldReceive('path')->with('absolute')->andReturn('foo/bar.baz');
+        $this->lfm_path->shouldReceive('extension')->andReturn('baz');
 
         $this->lfm->shouldReceive('getFileType')->with('baz')->andReturn('File');
 
@@ -107,6 +108,7 @@ public function testType()
     public function testExtension()
     {
         $this->lfm_path->shouldReceive('path')->with('absolute')->andReturn('foo/bar.baz');
+        $this->lfm_path->shouldReceive('extension')->andReturn('baz');
 
         $item = new LfmItem($this->lfm_path, $this->lfm);
 
@@ -160,6 +162,7 @@ public function testIcon()
         $this->lfm_path->shouldReceive('isDirectory')->andReturn(false);
         $this->lfm_path->shouldReceive('mimeType')->andReturn('application/plain');
         $this->lfm_path->shouldReceive('path')->with('absolute')->andReturn('foo/bar.baz');
+        $this->lfm_path->shouldReceive('extension')->andReturn('baz');
 
         $this->lfm->shouldReceive('getFileIcon')->with('baz')->andReturn('fa-file');
 

tests/LfmPathTest.php

@@ -158,15 +158,15 @@ public function testFiles()
         $this->assertInstanceOf(LfmItem::class, $path->files()[0]);
     }
 
-    public function testGet()
+    public function testPretty()
     {
         $helper = m::mock(Lfm::class);
         $helper->shouldReceive('getNameFromPath')->andReturn('bar');
         $helper->shouldReceive('isRunningOnWindows')->andReturn(false);
 
         $path = new LfmPath($helper);
 
-        $this->assertInstanceOf(LfmItem::class, $path->get('foo'));
+        $this->assertInstanceOf(LfmItem::class, $path->pretty('foo'));
     }
 
     public function testDelete()

tests/LfmStorageRepositoryTest.php

@@ -34,7 +34,7 @@ public function setUp()
 
         Storage::shouldReceive('disk')->andReturn($disk);
 
-        $this->storage = new LfmStorageRepository('foo/bar');
+        $this->storage = new LfmStorageRepository('foo/bar', 'local');
     }
 
     public function tearDown()
@@ -62,21 +62,11 @@ public function testFiles()
         $this->assertEquals('foo/bar/baz', $this->storage->files()[0]);
     }
 
-    public function testMakeDirectory()
-    {
-        $this->assertTrue($this->storage->makeDirectory());
-    }
-
     public function testExists()
     {
         $this->assertTrue($this->storage->exists());
     }
 
-    public function testGetFile()
-    {
-        $this->assertTrue($this->storage->getFile());
-    }
-
     public function testMimeType()
     {
         $this->assertEquals('text/plain', $this->storage->mimeType());
@@ -94,9 +84,4 @@ public function testMove()
 
         $this->assertTrue($this->storage->move($new_lfm_path));
     }
-
-    public function testDirectoryIsEmpty()
-    {
-        $this->assertTrue($this->storage->directoryIsEmpty());
-    }
 }

tests/LfmTest.php

@@ -24,6 +24,7 @@ public function testGetStorage()
         $config = m::mock(Config::class);
         $config->shouldReceive('get')->with('lfm.driver')->once()->andReturn('file');
         $config->shouldReceive('get')->with('lfm.driver')->once()->andReturn('storage');
+        $config->shouldReceive('get')->with('lfm.disk')->once()->andReturn('local');
 
         $lfm1 = new Lfm($config);
         $lfm2 = new Lfm($config);
@@ -41,18 +42,6 @@ public function testInput()
         $this->assertEquals('bar', $lfm->input('foo'));
     }
 
-    public function testIsProcessingImages()
-    {
-        $request = m::mock(Request::class);
-        $request->shouldReceive('input')->with('type')->once()->andReturn('image');
-        $request->shouldReceive('input')->with('type')->once()->andReturn('file');
-
-        $lfm = new Lfm(m::mock(Config::class), $request);
-
-        $this->assertTrue($lfm->isProcessingImages());
-        $this->assertFalse($lfm->isProcessingImages());
-    }
-
     public function testGetNameFromPath()
     {
         $this->assertEquals('bar', (new Lfm)->getNameFromPath('foo/bar'));
@@ -84,6 +73,9 @@ public function testGetCategoryName()
                ->with('lfm.folder_categories.image.folder_name', m::type('string'))
                ->once()
                ->andReturn('photos');
+        $config->shouldReceive('get')
+            ->with('lfm.folder_categories')
+            ->andReturn(['file' => [], 'image' => []]);
 
         $request = m::mock(Request::class);
         $request->shouldReceive('input')->with('type')->once()->andReturn('file');
@@ -100,11 +92,18 @@ public function testCurrentLfmType()
         $request = m::mock(Request::class);
         $request->shouldReceive('input')->with('type')->once()->andReturn('file');
         $request->shouldReceive('input')->with('type')->once()->andReturn('image');
+        $request->shouldReceive('input')->with('type')->once()->andReturn('foo');
 
-        $lfm = new Lfm(m::mock(Config::class), $request);
+        $config = m::mock(Config::class);
+        $config->shouldReceive('get')
+            ->with('lfm.folder_categories')
+            ->andReturn(['file' => [], 'image' => []]);
+
+        $lfm = new Lfm($config, $request);
 
         $this->assertEquals('file', $lfm->currentLfmType());
         $this->assertEquals('image', $lfm->currentLfmType());
+        $this->assertEquals('file', $lfm->currentLfmType());
     }
 
     public function testGetUserSlug()