Fix Border and Flip test

thephpleague · Jan 25, 2024 · 66ad315 · 66ad315
1 parent 9d10e69
commit 66ad315
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 5 deletions.
diff --git a/tests/Manipulators/BorderTest.php b/tests/Manipulators/BorderTest.php
@@ -94,7 +94,7 @@ public function testRunOverlay()
         $image = \Mockery::mock(ImageInterface::class, function ($mock) {
             $mock->shouldReceive('width')->andReturn(100)->once();
             $mock->shouldReceive('height')->andReturn(100)->once();
-            $mock->shouldReceive('rectangle')->with(5, 5, 95, 95, \Mockery::on(function ($closure) {
+            $mock->shouldReceive('drawRectangle')->with(5, 5, \Mockery::on(function ($closure) {
                 return true;
             }))->andReturn($mock)->once();
         });
@@ -111,7 +111,7 @@ public function testRunShrink()
             $mock->shouldReceive('width')->andReturn(100)->once();
             $mock->shouldReceive('height')->andReturn(100)->once();
             $mock->shouldReceive('resize')->with(80, 80)->andReturn($mock)->once();
-            $mock->shouldReceive('resizeCanvas')->with(20, 20, 'center', true, 'rgba(0, 0, 0, 0.5)')->andReturn($mock)->once();
+            $mock->shouldReceive('resizeCanvasRelative')->with(20, 20, 'rgba(0, 0, 0, 0.5)', 'center')->andReturn($mock)->once();
         });
 
         $border = new Border();
@@ -123,7 +123,7 @@ public function testRunShrink()
     public function testRunExpand()
     {
         $image = \Mockery::mock(ImageInterface::class, function ($mock) {
-            $mock->shouldReceive('resizeCanvas')->with(20, 20, 'center', true, 'rgba(0, 0, 0, 0.5)')->andReturn($mock)->once();
+            $mock->shouldReceive('resizeCanvasRelative')->with(20, 20, 'rgba(0, 0, 0, 0.5)', 'center')->andReturn($mock)->once();
         });
 
         $border = new Border();

diff --git a/tests/Manipulators/FlipTest.php b/tests/Manipulators/FlipTest.php
@@ -27,8 +27,8 @@ public function testCreateInstance()
     public function testRun()
     {
         $image = \Mockery::mock(ImageInterface::class, function ($mock) {
-            $mock->shouldReceive('flip')->andReturn($mock)->with('h')->once();
-            $mock->shouldReceive('flip')->andReturn($mock)->with('v')->once();
+            $mock->shouldReceive('flip')->andReturn($mock)->once();
+            $mock->shouldReceive('flop')->andReturn($mock)->once();
         });
 
         $this->assertInstanceOf(
@@ -42,6 +42,19 @@ public function testRun()
         );
     }
 
+    public function testRunBoth()
+    {
+        $image = \Mockery::mock(ImageInterface::class, function ($mock) {
+            $mock->shouldReceive('flip')->andReturn($mock)->once();
+            $mock->shouldReceive('flop')->andReturn($mock)->once();
+        });
+
+        $this->assertInstanceOf(
+            ImageInterface::class,
+            $this->manipulator->setParams(['flip' => 'both'])->run($image)
+        );
+    }
+
     public function testGetFlip()
     {
         $this->assertSame('h', $this->manipulator->setParams(['flip' => 'h'])->getFlip());