Fix Builder Proxy (#6)

TitasGailius · Mar 30, 2020 · b48238f · b48238f
1 parent ae2ccf8
commit b48238f
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 2 deletions.
diff --git a/src/Builder.php b/src/Builder.php
@@ -394,8 +394,8 @@ public function __call(string $method, array $parameters)
             return static::$extensions[$method]($this);
         }
 
-        if (method_exists($this->process(), $method)) {
-            return $this->process()->{$method}(...$parameters);
+        if (method_exists($process = $this->process(), $method)) {
+            return $process->{$method}(...$parameters);
         }
 
         throw new BadMethodCallException(sprintf(

diff --git a/tests/BuilderTest.php b/tests/BuilderTest.php
@@ -226,6 +226,22 @@ public function testDataWithMissingBinding()
         $this->assertEquals("Hello, \n", $process->getOutput());
     }
 
+    /**
+     * Test that missing methods are passed to the Proccess instance.
+     *
+     * @return void
+     */
+    public function testBuilderProxy()
+    {
+        $builder = $this->builderWithMockedProccess(function ($mock) {
+            $mock->shouldReceive('getPid')
+                ->once()
+                ->andReturn(123);
+        });
+
+        $this->assertEquals(123, $builder->getPid());
+    }
+
     /**
      * Create a new builder instance with a mocked process instance.
      *