From 03d425bdd9a639446ae72ce6556d753f2f162f83 Mon Sep 17 00:00:00 2001
From: n-peugnet <n.peugnet@free.fr>
Date: Thu, 23 Apr 2020 15:38:17 +0200
Subject: tests(ci): problemMatchers, 3 php versions, optims

Add problem matchers for phpcs, phpstan and phpunit.
Add composer cache
Test over multiple php version
---
 .github/matchers/phpcs.json   | 19 ++++++++++
 .github/matchers/phpstan.json | 16 ++++++++
 .github/matchers/phpunit.json | 25 ++++++++++++
 .github/workflows/php.yml     | 88 ++++++++++++++++++++++++++++++++-----------
 4 files changed, 127 insertions(+), 21 deletions(-)
 create mode 100644 .github/matchers/phpcs.json
 create mode 100644 .github/matchers/phpstan.json
 create mode 100644 .github/matchers/phpunit.json

(limited to '.github')

diff --git a/.github/matchers/phpcs.json b/.github/matchers/phpcs.json
new file mode 100644
index 0000000..339af9a
--- /dev/null
+++ b/.github/matchers/phpcs.json
@@ -0,0 +1,19 @@
+{
+    "problemMatcher": [
+        {
+            "owner": "phpcs",
+            "fileLocation": "relative",
+            "pattern": [
+                {
+                    "regexp": "^\"(.+)\",(\\d+),(\\d+),(.+),\"(.+)\",(.+),(\\d+),(\\d+)$",
+                    "file": 1,
+                    "line": 2,
+                    "column": 3,
+                    "severity": 4,
+                    "message": 5,
+                    "code": 6
+                }
+            ]
+        }
+    ]
+}
\ No newline at end of file
diff --git a/.github/matchers/phpstan.json b/.github/matchers/phpstan.json
new file mode 100644
index 0000000..3e6bc6c
--- /dev/null
+++ b/.github/matchers/phpstan.json
@@ -0,0 +1,16 @@
+{
+    "problemMatcher": [
+        {
+            "owner": "phpstan",
+            "severity": "error",
+            "pattern": [
+                {
+                    "regexp": "^(.+):(\\d+):(.+)$",
+                    "file": 1,
+                    "line": 2,
+                    "message": 3
+                }
+            ]
+        }
+    ]
+}
\ No newline at end of file
diff --git a/.github/matchers/phpunit.json b/.github/matchers/phpunit.json
new file mode 100644
index 0000000..0869357
--- /dev/null
+++ b/.github/matchers/phpunit.json
@@ -0,0 +1,25 @@
+{
+    "problemMatcher": [
+        {
+            "owner": "phpunit",
+            "fileLocation": "absolute",
+            "pattern": [
+                {
+                    "regexp": "^\\d+\\)\\s.*$"
+                },
+                {
+                    "regexp": "^(.*)$",
+                    "message": 1
+                },
+                {
+                    "regexp": "^\\s*$"
+                },
+                {
+                    "regexp": "^(.*):(\\d+)$",
+                    "file": 1,
+                    "line": 2
+                }
+            ]
+        }
+    ]
+}
\ No newline at end of file
diff --git a/.github/workflows/php.yml b/.github/workflows/php.yml
index 479a9f9..039daca 100644
--- a/.github/workflows/php.yml
+++ b/.github/workflows/php.yml
@@ -1,46 +1,92 @@
-name: build
+name: checks
 
 on:
   - push
   - pull_request
 
 jobs:
-  php-build:
+  test-php:
+    name: test (php ${{ matrix.php-versions }})
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        php-versions: ['7.2', '7.3', '7.4']
     steps:
       - name: Git checkout
         uses: actions/checkout@v2
 
+      - name: Setup PHP, with composer and extensions
+        uses: shivammathur/setup-php@v2 #https://github.com/shivammathur/setup-php
+        with:
+          php-version: ${{ matrix.php-versions }}
+          coverage: xdebug
+
       - name: Validate composer.json and composer.lock
         run: composer validate
 
+      - name: Get composer cache directory
+        id: composer-cache
+        run: echo "::set-output name=dir::$(composer config cache-files-dir)"
+
+      - name: Cache dependencies
+        uses: actions/cache@v1
+        with:
+          path: ${{ steps.composer-cache.outputs.dir }}
+          key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-composer-
+
       - name: Install dependencies
         run: composer install --prefer-dist --no-progress --no-suggest
 
-      - name: Run test suite
-        run: make check -k
-
-      - name: PHPStan xml report
-        if: always()
+      - name: PHPUnit
         run: |
-          mkdir -p build/phpstan
-          vendor/bin/phpstan analyse --no-progress --error-format=checkstyle > build/phpstan/checkstyle.xml
-
-      - name: Analysis Publisher
-        uses: digirati-labs/analysis-publisher@0.0.1
-        if: failure()
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-        with:
-          args: >
-            --report-type=checkstyle --path=**/build/phpcs/checkstyle.xml
-            --report-type=checkstyle --path=**/build/phpstan/checkstyle.xml
-            --publisher=github_check
+          echo "::add-matcher::.github/matchers/phpunit.json"
+          make test
+          echo "::remove-matcher owner=phpunit::"
 
       - name: Coveralls publish
-        if: always()
         env:
           COVERALLS_REPO_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         run: |
           composer require --quiet --no-interaction cedx/coveralls
           vendor/bin/coveralls build/phpunit/cov.xml
+
+  lint:
+    name: lint
+    runs-on: ubuntu-latest
+    steps:
+      - name: Git checkout
+        uses: actions/checkout@v2
+
+      - name: Validate composer.json and composer.lock
+        run: composer validate
+
+      - name: Get composer cache directory
+        id: composer-cache
+        run: echo "::set-output name=dir::$(composer config cache-files-dir)"
+
+      - name: Cache dependencies
+        uses: actions/cache@v1
+        with:
+          path: ${{ steps.composer-cache.outputs.dir }}
+          key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-composer-
+
+      - name: Install dependencies
+        run: composer install --prefer-dist --no-progress --no-suggest
+
+      - name: PHPCS
+        if: always()
+        run: |
+          echo "::add-matcher::.github/matchers/phpcs.json"
+          vendor/bin/phpcs --report=csv
+          echo "::remove-matcher owner=phpcs::"
+
+      - name: PHPStan
+        if: always()
+        run: |
+          echo "::add-matcher::.github/matchers/phpstan.json"
+          vendor/bin/phpstan analyse --error-format=raw --no-progress
+          echo "::remove-matcher owner=phpstan::"
-- 
cgit v1.2.3