added getMethodReflection and remove duplicating reflection parts

Author: kokspflanze

test/Github/Tests/Api/AbstractApiTest.php

@@ -4,6 +4,7 @@
 
 use Github\Api\AbstractApi;
 use GuzzleHttp\Psr7\Response;
+use ReflectionMethod;
 
 class AbstractApiTest extends \PHPUnit_Framework_TestCase
 {
@@ -29,8 +30,7 @@ public function shouldPassGETRequestToClient()
 
         $api = $this->getAbstractApiObject($client);
 
-        $method = new \ReflectionMethod($api, 'get');
-        $method->setAccessible(true);
+        $method = $this->getMethodReflection($api, 'get');
 
         $this->assertEquals($expectedArray, $method->invokeArgs($api, ['/path', ['param1' => 'param1value'], ['header1' => 'header1value']]));
     }
@@ -57,8 +57,7 @@ public function shouldPassPOSTRequestToClient()
             ->willReturn($httpClient);
 
         $api = $this->getAbstractApiObject($client);
-        $method = new \ReflectionMethod($api, 'post');
-        $method->setAccessible(true);
+        $method = $this->getMethodReflection($api, 'post');
 
         $this->assertEquals($expectedArray, $method->invokeArgs($api, ['/path', array('param1' => 'param1value'), array('option1' => 'option1value')]));
     }
@@ -85,8 +84,7 @@ public function shouldPassPATCHRequestToClient()
             ->willReturn($httpClient);
 
         $api = $this->getAbstractApiObject($client);
-        $method = new \ReflectionMethod($api, 'patch');
-        $method->setAccessible(true);
+        $method = $this->getMethodReflection($api, 'patch');
 
         $this->assertEquals($expectedArray, $method->invokeArgs($api, ['/path', array('param1' => 'param1value'), array('option1' => 'option1value')]));
     }
@@ -113,8 +111,7 @@ public function shouldPassPUTRequestToClient()
             ->willReturn($httpClient);
 
         $api = $this->getAbstractApiObject($client);
-        $method = new \ReflectionMethod($api, 'put');
-        $method->setAccessible(true);
+        $method = $this->getMethodReflection($api, 'put');
 
         $this->assertEquals($expectedArray, $method->invokeArgs($api, ['/path', array('param1' => 'param1value'), array('option1' => 'option1value')]));
     }
@@ -142,8 +139,7 @@ public function shouldPassDELETERequestToClient()
 
 
         $api = $this->getAbstractApiObject($client);
-        $method = new \ReflectionMethod($api, 'delete');
-        $method->setAccessible(true);
+        $method = $this->getMethodReflection($api, 'delete');
 
         $this->assertEquals($expectedArray, $method->invokeArgs($api, ['/path', array('param1' => 'param1value'), array('option1' => 'option1value')]));
     }
@@ -170,8 +166,7 @@ public function shouldNotPassEmptyRefToClient()
             ->willReturn($httpClient);
 
         $api = $this->getAbstractApiObject($client);
-        $method = new \ReflectionMethod($api, 'get');
-        $method->setAccessible(true);
+        $method = $this->getMethodReflection($api, 'get');
 
         $this->assertInternalType('array', $method->invokeArgs($api, ['/path', array('ref' => null)]));
     }
@@ -188,6 +183,19 @@ protected function getAbstractApiObject($client)
             ->getMock();
     }
 
+    /**
+     * @param $api
+     * @param $methodName
+     * @return ReflectionMethod
+     */
+    public function getMethodReflection($api, $methodName)
+    {
+        $method = new ReflectionMethod($api, $methodName);
+        $method->setAccessible(true);
+
+        return $method;
+    }
+
     /**
      * @return \Github\Client
      */