Merge branch '2.x'

* 2.x:
  Fix bc break in check api class
  Switch CI setup from travisci to github actions
  Adding GitHub authentication to GraphQL documentation
  bug #915 Fix call to test a webhook method (morrislaptop)
  Remove antiope preview header as checks api is now stable

Author: acrobat

.github/workflows/ci.yml

@@ -0,0 +1,53 @@
+name: CI
+
+on:
+    [push, pull_request]
+
+jobs:
+    test:
+        name: Test on ${{ matrix.php-versions }} PHP
+        runs-on: ubuntu-latest
+
+        strategy:
+            matrix:
+                php-versions: ['7.1', '7.2', '7.3', '7.4']
+
+        steps:
+            - uses: actions/checkout@v2
+
+            - name: Setup PHP
+              uses: shivammathur/setup-php@v2
+              with:
+                  php-version: ${{ matrix.php-versions }}
+                  coverage: xdebug
+
+            - name: Install Composer Dependencies
+              uses: ramsey/composer-install@v1
+
+            - name: Run phpunit
+              run: vendor/bin/phpunit --verbose --coverage-text
+
+    roave_bc_check:
+        name: Roave BC Check
+        runs-on: ubuntu-latest
+        steps:
+            - uses: actions/checkout@v2
+
+            - name: Roave BC Check
+              uses: docker://nyholm/roave-bc-check-ga
+
+    phpstan:
+        runs-on: ubuntu-latest
+
+        steps:
+            - uses: actions/checkout@v2
+
+            - uses: shivammathur/setup-php@v2
+              with:
+                  php-version: 7.1
+
+            - name: Install Composer Dependencies
+              uses: ramsey/composer-install@v1
+
+            - name: Run phpstan
+              run: vendor/bin/phpstan analyse --no-progress

.travis.yml

@@ -9,6 +9,16 @@ Wraps [GitHub v4 API (GraphQL API)](http://developer.github.com/v4/).
 $rateLimits = $client->api('graphql')->execute($query);
 ```
 
+#### Authentication
+
+To use [GitHub v4 API (GraphQL API)](http://developer.github.com/v4/) requests must [authenticated]((../security.md)).
+
+```php
+$client->authenticate($token, null, Github\Client::AUTH_ACCESS_TOKEN);
+
+$result = $client->api('graphql')->execute($query);
+```
+
 #### Use variables
 
 [Variables](https://developer.github.com/v4/guides/forming-calls/#working-with-variables) allow specifying of requested data without dynamical change of a query on a client side.
@@ -28,5 +38,7 @@ $variables = [
     'organizationLogin' => 'KnpLabs'
 ];
 
+$client->authenticate('<your-token>', null, Github\Client::AUTH_ACCESS_TOKEN);
+
 $orgInfo = $client->api('graphql')->execute($query, $variables);
-```
+```

doc/graphql.md

@@ -13,6 +13,7 @@
  */
 class Checks extends AbstractApi
 {
+    // NEXT_MAJOR: remove trait use.
     use AcceptHeaderTrait;
 
     /**
@@ -32,9 +33,6 @@ public function create($username, $repository, array $params = [])
             throw new MissingArgumentException(['name', 'head_sha']);
         }
 
-        // This api is in preview mode, so set the correct accept-header.
-        $this->acceptHeaderValue = 'application/vnd.github.antiope-preview+json';
-
         return $this->post('/repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/check-runs', $params);
     }
 
@@ -50,9 +48,6 @@ public function create($username, $repository, array $params = [])
      */
     public function update($username, $repository, $checkRunId, array $params = [])
     {
-        // This api is in preview mode, so set the correct accept-header.
-        $this->acceptHeaderValue = 'application/vnd.github.antiope-preview+json';
-
         return $this->patch('/repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/check-runs/'.rawurlencode($checkRunId), $params);
     }
 
@@ -68,9 +63,6 @@ public function update($username, $repository, $checkRunId, array $params = [])
      */
     public function all($username, $repository, $ref, $params = [])
     {
-        // This api is in preview mode, so set the correct accept-header.
-        $this->acceptHeaderValue = 'application/vnd.github.antiope-preview+json';
-
         return $this->get('/repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/commits/'.rawurlencode($ref).'/check-runs', $params);
     }
 
@@ -85,9 +77,6 @@ public function all($username, $repository, $ref, $params = [])
      */
     public function show($username, $repository, $checkRunId)
     {
-        // This api is in preview mode, so set the correct accept-header.
-        $this->acceptHeaderValue = 'application/vnd.github.antiope-preview+json';
-
         return $this->get('/repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/check-runs/'.rawurlencode($checkRunId));
     }
 
@@ -102,9 +91,6 @@ public function show($username, $repository, $checkRunId)
      */
     public function annotations($username, $repository, $checkRunId)
     {
-        // This api is in preview mode, so set the correct accept-header.
-        $this->acceptHeaderValue = 'application/vnd.github.antiope-preview+json';
-
         return $this->get('/repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/check-runs/'.rawurlencode($checkRunId).'/annotations');
     }
 }

lib/Github/Api/Repository/Checks.php

@@ -47,7 +47,7 @@ public function ping($username, $repository, $id)
 
     public function test($username, $repository, $id)
     {
-        return $this->post('/repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/hooks/'.rawurlencode($id).'/test');
+        return $this->post('/repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/hooks/'.rawurlencode($id).'/tests');
     }
 
     public function remove($username, $repository, $id)

lib/Github/Api/Repository/Hooks.php

@@ -144,7 +144,7 @@ public function shouldTestHook()
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('post')
-            ->with('/repos/KnpLabs/php-github-api/hooks/123/test')
+            ->with('/repos/KnpLabs/php-github-api/hooks/123/tests')
             ->will($this->returnValue($expectedValue));
 
         $this->assertEquals($expectedValue, $api->test('KnpLabs', 'php-github-api', 123));