rename classes

spatie · Jul 4, 2017 · 74d2933 · 74d2933
1 parent 73bff3b
commit 74d2933
Show file tree

Hide file tree

Showing 5 changed files with 73 additions and 73 deletions.
diff --git a/src/ImageOptimizer.php → src/OptimizerChain.php b/src/ImageOptimizer.php → src/OptimizerChain.php
@@ -6,7 +6,7 @@
 use Symfony\Component\Process\Process;
 use Spatie\ImageOptimizer\Optimizers\Optimizer;
 
-class ImageOptimizer
+class OptimizerChain
 {
     protected $optimizers = [];
 

diff --git a/src/ImageOptimizerFactory.php → src/OptimizerChainFactory.php b/src/ImageOptimizerFactory.php → src/OptimizerChainFactory.php
@@ -7,11 +7,11 @@
 use Spatie\ImageOptimizer\Optimizers\Pngquant;
 use Spatie\ImageOptimizer\Optimizers\Jpegoptim;
 
-class ImageOptimizerFactory
+class OptimizerChainFactory
 {
-    public static function create(): ImageOptimizer
+    public static function create(): OptimizerChain
     {
-        return (new ImageOptimizer())
+        return (new OptimizerChain())
             ->addOptimizer(new Jpegoptim([
                 '--strip-all',
                 '--all-progressive',

diff --git a/tests/ImageOptimizerTest.php b/tests/ImageOptimizerTest.php
diff --git a/tests/ImageOptimizerFactoryTest.php → tests/OptimizerChainFactoryTest.php b/tests/ImageOptimizerFactoryTest.php → tests/OptimizerChainFactoryTest.php
@@ -6,18 +6,18 @@
 use Spatie\ImageOptimizer\Optimizers\Gifsicle;
 use Spatie\ImageOptimizer\Optimizers\Pngquant;
 use Spatie\ImageOptimizer\Optimizers\Jpegoptim;
-use Spatie\ImageOptimizer\ImageOptimizerFactory;
+use Spatie\ImageOptimizer\OptimizerChainFactory;
 
-class ImageOptimizerFactoryTest extends TestCase
+class OptimizerChainFactoryTest extends TestCase
 {
-    /** @var \Spatie\ImageOptimizer\ImageOptimizer */
-    protected $imageOptimizer;
+    /** @var \Spatie\ImageOptimizer\OptimizerChain */
+    protected $optimizerChain;
 
     public function setUp()
     {
         parent::setUp();
 
-        $this->imageOptimizer = ImageOptimizerFactory::create()
+        $this->optimizerChain = OptimizerChainFactory::create()
             ->useLogger($this->log);
     }
 
@@ -26,7 +26,7 @@ public function it_can_optimize_a_jpg()
     {
         $tempFilePath = $this->getTempFilePath('test.jpg');
 
-        $this->imageOptimizer->optimize($tempFilePath);
+        $this->optimizerChain->optimize($tempFilePath);
 
         $this->assertDecreasedFileSize($tempFilePath, $this->getTestFilePath('test.jpg'));
 
@@ -38,7 +38,7 @@ public function it_can_optimize_a_png()
     {
         $tempFilePath = $this->getTempFilePath('test.png');
 
-        $this->imageOptimizer->optimize($tempFilePath);
+        $this->optimizerChain->optimize($tempFilePath);
 
         $this->assertDecreasedFileSize($tempFilePath, $this->getTestFilePath('test.png'));
 
@@ -53,7 +53,7 @@ public function it_can_optimize_a_gif()
     {
         $tempFilePath = $this->getTempFilePath('test.gif');
 
-        $this->imageOptimizer->optimize($tempFilePath);
+        $this->optimizerChain->optimize($tempFilePath);
 
         $this->assertDecreasedFileSize($tempFilePath, $this->getTestFilePath('test.gif'));
 
@@ -67,7 +67,7 @@ public function it_will_not_not_touch_a_non_image_file()
 
         $originalContent = file_get_contents($tempFilePath);
 
-        $this->imageOptimizer->optimize($tempFilePath);
+        $this->optimizerChain->optimize($tempFilePath);
 
         $optimizedContent = file_get_contents($tempFilePath);
 

diff --git a/tests/OptimizerChainTest.php b/tests/OptimizerChainTest.php
@@ -0,0 +1,60 @@
+<?php
+
+namespace Spatie\ImageOptimizer\Test;
+
+use Spatie\ImageOptimizer\OptimizerChain;
+use Spatie\ImageOptimizer\Optimizers\Optipng;
+use Spatie\ImageOptimizer\Optimizers\Pngquant;
+use Spatie\ImageOptimizer\Optimizers\Jpegoptim;
+
+class OptimizerChainTest extends TestCase
+{
+    /** @var \Spatie\ImageOptimizer\OptimizerChain; */
+    protected $optimizerChain;
+
+    public function setUp()
+    {
+        parent::setUp();
+
+        $this->optimizerChain = new OptimizerChain();
+    }
+
+    /** @test */
+    public function it_will_not_throw_an_exception_when_not_using_a_logger()
+    {
+        $testImage = $this->getTempFilePath('test.jpg');
+
+        $this->optimizerChain
+            ->addOptimizer(new Jpegoptim())
+            ->optimize($testImage);
+
+        $this->assertDecreasedFileSize($testImage, $this->getTestFilePath('test.jpg'));
+    }
+
+    /** @test */
+    public function it_can_get_all_optimizers()
+    {
+        $this->assertEquals([], $this->optimizerChain->getOptimizers());
+
+        $this->optimizerChain->addOptimizer(new Jpegoptim());
+
+        $this->assertInstanceOf(Jpegoptim::class, $this->optimizerChain->getOptimizers()[0]);
+    }
+
+    /** @test */
+    public function it_can_replace_all_optimizers_with_other_ones()
+    {
+        $this->assertEquals([], $this->optimizerChain->getOptimizers());
+
+        $this->optimizerChain->addOptimizer(new Jpegoptim());
+
+        $this->optimizerChain->setOptimizers([
+            new Optipng(),
+            new Pngquant(),
+        ]);
+
+        $this->assertCount(2, $this->optimizerChain->getOptimizers());
+        $this->assertInstanceOf(Optipng::class, $this->optimizerChain->getOptimizers()[0]);
+        $this->assertInstanceOf(Pngquant::class, $this->optimizerChain->getOptimizers()[1]);
+    }
+}