Fix file already exist decision for not accessible url using fall back

tommy-muehle · May 24, 2019 · 5c0cbca · 5c0cbca
1 parent 6e258bd
commit 5c0cbca
Show file tree

Hide file tree

Showing 3 changed files with 104 additions and 6 deletions.
diff --git a/src/Script/Decision/FileAlreadyExistDecision.php b/src/Script/Decision/FileAlreadyExistDecision.php
@@ -16,7 +16,13 @@ class FileAlreadyExistDecision extends AbstractDecision
      */
     public function canProceed(Tool $tool)
     {
-        if (false === $this->helper->isFileAlreadyExist($tool->getFilename(), $tool->getUrl())) {
+        $url = $tool->getUrl();
+
+        if (false === $this->helper->getDownloader()->isAccessible($url)) {
+            $url = $tool->getFallbackUrl();
+        }
+
+        if (false === $this->helper->isFileAlreadyExist($tool->getFilename(), $url)) {
             return true;
         }
 

diff --git a/tests/Script/Decision/FileAlreadyExistDecisionTest.php b/tests/Script/Decision/FileAlreadyExistDecisionTest.php
@@ -2,17 +2,63 @@
 
 namespace Tooly\Tests\Script\Decision;
 
-use Tooly\Factory\ToolFactory;
 use Tooly\Model\Tool;
 use Tooly\Script\Decision\FileAlreadyExistDecision;
+use Tooly\Script\Helper\Downloader;
 
 /**
  * @package Tooly\Tests\Script\Decision
  */
 class FileAlreadyExistDecisionTest extends DecisionTestCase
 {
-    public function testIfFileNotAlreadyExistReturnsTrue()
+    public function testIfFileIsAccessibleAndFileNotAlreadyExistReturnsTrue()
     {
+        $downloader = $this
+            ->getMockBuilder(Downloader::class)
+            ->disableOriginalConstructor()
+            ->getMock();
+
+        $downloader
+            ->expects($this->once())
+            ->method('isAccessible')
+            ->willReturn(true);
+
+        $this->helper
+            ->expects($this->once())
+            ->method('getDownloader')
+            ->willReturn($downloader);
+
+        $this->helper
+            ->expects($this->once())
+            ->method('isFileAlreadyExist')
+            ->willReturn(false);
+
+        $tool = $this
+            ->getMockBuilder(Tool::class)
+            ->disableOriginalConstructor()
+            ->getMock();
+
+        $decision = new FileAlreadyExistDecision($this->configuration, $this->helper);
+        $this->assertTrue($decision->canProceed($tool));
+    }
+
+    public function testIfFileNotAccessibleAndFileNotAlreadyExistReturnsTrue()
+    {
+        $downloader = $this
+            ->getMockBuilder(Downloader::class)
+            ->disableOriginalConstructor()
+            ->getMock();
+
+        $downloader
+            ->expects($this->once())
+            ->method('isAccessible')
+            ->willReturn(false);
+
+        $this->helper
+            ->expects($this->once())
+            ->method('getDownloader')
+            ->willReturn($downloader);
+
         $this->helper
             ->expects($this->once())
             ->method('isFileAlreadyExist')
@@ -27,8 +73,54 @@ public function testIfFileNotAlreadyExistReturnsTrue()
         $this->assertTrue($decision->canProceed($tool));
     }
 
-    public function testIfFileAlreadyExistReturnsFalse()
+    public function testIfFileIsAccessibleAndFileAlreadyExistReturnsFalse()
     {
+        $downloader = $this
+            ->getMockBuilder(Downloader::class)
+            ->disableOriginalConstructor()
+            ->getMock();
+
+        $downloader
+            ->expects($this->once())
+            ->method('isAccessible')
+            ->willReturn(true);
+
+        $this->helper
+            ->expects($this->once())
+            ->method('getDownloader')
+            ->willReturn($downloader);
+
+        $this->helper
+            ->expects($this->once())
+            ->method('isFileAlreadyExist')
+            ->willReturn(true);
+
+        $tool = $this
+            ->getMockBuilder(Tool::class)
+            ->disableOriginalConstructor()
+            ->getMock();
+
+        $decision = new FileAlreadyExistDecision($this->configuration, $this->helper);
+        $this->assertFalse($decision->canProceed($tool));
+    }
+
+    public function testIfFileNotAccessibleAndFileAlreadyExistReturnsFalse()
+    {
+        $downloader = $this
+            ->getMockBuilder(Downloader::class)
+            ->disableOriginalConstructor()
+            ->getMock();
+
+        $downloader
+            ->expects($this->once())
+            ->method('isAccessible')
+            ->willReturn(false);
+
+        $this->helper
+            ->expects($this->once())
+            ->method('getDownloader')
+            ->willReturn($downloader);
+
         $this->helper
             ->expects($this->once())
             ->method('isFileAlreadyExist')

diff --git a/tests/Script/Processor/ProcessTest.php b/tests/Script/Processor/ProcessTest.php
@@ -131,7 +131,7 @@ public function testCanSuccessfullyDownloadAToolViaFallbackUrl()
             ->getMock();
 
         $downloader
-            ->expects($this->exactly(3))
+            ->expects($this->exactly(4))
             ->method('isAccessible')
             ->will($this->onConsecutiveCalls(false, true, false));
 
@@ -148,7 +148,7 @@ public function testCanSuccessfullyDownloadAToolViaFallbackUrl()
             ->willReturn($filesystem);
 
         $this->helper
-            ->expects($this->exactly(4))
+            ->expects($this->exactly(5))
             ->method('getDownloader')
             ->willReturn($downloader);