Merge pull request #6 from marc-mabe/gh-workflow

marc-mabe · web-flow · commit 55f58145f42d · 2020-10-15T22:17:07.000+02:00
Migrate from travis + scurinizer to Github workflow + codecov
diff --git a/.github/workflows/test.Dockerfile b/.github/workflows/test.Dockerfile
@@ -0,0 +1,18 @@
+ARG PHP_VERSION=latest
+FROM php:${PHP_VERSION}-cli-alpine
+
+WORKDIR /workdir
+
+# install composer
+RUN curl -sS https://getcomposer.org/installer | php -- --install-dir=/usr/local/bin --filename=composer
+ENV COMPOSER_ALLOW_SUPERUSER=1
+ENV COMPOSER_HTACCESS_PROTECT=0
+ENV COMPOSER_CACHE_DIR=/.composer
+
+# install PHP extension pcov
+RUN apk add --no-cache --virtual .build-deps $PHPIZE_DEPS \
+    && mkdir -p /usr/src/php/ext/pcov && curl -fsSL https://pecl.php.net/get/pcov | tar xvz -C /usr/src/php/ext/pcov --strip 1 \
+    && docker-php-ext-install pcov \
+    && docker-php-ext-enable pcov \
+    && rm -Rf /usr/src/php/ext/pcov \
+    && apk del --no-cache .build-deps
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -0,0 +1,74 @@
+name: Test
+
+on:
+  pull_request:
+  push:
+    branches:
+      - master
+
+jobs:
+  php:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        include:
+          - PHP_VERSION: 7.1
+            CODE_COVERAGE: false
+            COMPOSER_EXTRA_ARGS: --prefer-lowest
+          - PHP_VERSION: 7.1
+            CODE_COVERAGE: false
+          - PHP_VERSION: 7.2
+            CODE_COVERAGE: true
+          - PHP_VERSION: 7.3
+            CODE_COVERAGE: true
+          - PHP_VERSION: 7.4
+            CODE_COVERAGE: true
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: Cache Docker Image
+        id: cache-docker-image
+        uses: actions/cache@v2
+        with:
+          path: /tmp/docker-image.tar
+          key: cache-docker-image-test:${{ matrix.PHP_VERSION }}
+
+      - name: Load Docker Image
+        if: steps.cache-docker-image.outputs.cache-hit == 'true'
+        run: docker load --input /tmp/docker-image.tar
+
+      - name: Build Docker Image
+        if: steps.cache-docker-image.outputs.cache-hit != 'true'
+        run: docker build -f .github/workflows/test.Dockerfile -t 'test:${{ matrix.PHP_VERSION }}' --build-arg 'PHP_VERSION=${{ matrix.PHP_VERSION }}' .
+
+      - name: Cache Composer Cache Files
+        uses: actions/cache@v2
+        with:
+          path: /tmp/composer-cache-files
+          key: cache-composer-cache-files-${{ matrix.PHP_VERSION }}
+          restore-keys: |
+            cache-composer-cache-files-
+
+      - name: Install Composer Dependencies
+        run: |
+          docker run --rm -u "$(id -u):$(id -g)" -v "$(pwd):/workdir" -v '/tmp/composer-cache-files:/.composer' 'test:${{ matrix.PHP_VERSION }}' composer update --no-interaction --no-progress --prefer-dist ${{ matrix.COMPOSER_EXTRA_ARGS }}
+
+      - name: Run Unit Test
+        run: |
+          if [ "${{ matrix.CODE_COVERAGE }}" == "true" ]; then
+            docker run --rm -u "$(id -u):$(id -g)" -v "$(pwd):/workdir" 'test:${{ matrix.PHP_VERSION }}' php -d 'zend.assertions=1' -d 'pcov.enabled=1' ./vendor/bin/phpunit --coverage-clover=.clover.xml
+          else
+            docker run --rm -u "$(id -u):$(id -g)" -v "$(pwd):/workdir" 'test:${{ matrix.PHP_VERSION }}' php -d 'zend.assertions=1' ./vendor/bin/phpunit
+          fi
+
+      - name: Upload Codecov Report
+        uses: codecov/codecov-action@v1
+        if: ${{ matrix.CODE_COVERAGE }}
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}
+          file: .clover.xml
+
+      - name: Export Docker Image
+        if: steps.cache-docker-image.outputs.cache-hit != 'true'
+        run: docker save --output /tmp/docker-image.tar 'test:${{ matrix.PHP_VERSION }}'
diff --git a/.gitignore b/.gitignore
@@ -1,9 +1,8 @@
 # hidden files
 .*
 !.gitignore
-!.travis.yml
+!.github
 
 #composer
 composer.lock
-composer.phar
 vendor
diff --git a/.travis.yml b/.travis.yml
diff --git a/composer.json b/composer.json
@@ -5,12 +5,12 @@
     "keywords": ["enum", "phpstan"],
     "license": "BSD-3-Clause",
     "require": {
-        "php": "~7.1",
+        "php": "^7.1",
         "marc-mabe/php-enum": "^1.1 || ^2.0 || ^3.0 || ^4.0",
         "phpstan/phpstan": "^0.12"
     },
     "require-dev": {
-        "phpunit/phpunit": "^7.5.20"
+        "phpunit/phpunit": "^7.5 | ^8.5 | 9.4"
     },
     "autoload": {
         "psr-4": {
diff --git a/tests/unit/EnumMethodReflectionTest.php b/tests/unit/EnumMethodReflectionTest.php
@@ -25,45 +25,45 @@ class EnumMethodReflectionTest extends TestCase
      */
     protected $reflectionExtension;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->broker = $this->createBroker();
         $this->reflectionExtension = new EnumMethodsClassReflectionExtension();
     }
 
-    public function getDeclaringClass()
+    public function testGetDeclaringClass(): void
     {
         $classReflection  = $this->broker->getClass(VisibilityEnum::class);
         $methodReflection = $this->reflectionExtension->getMethod($classReflection, 'STR');
 
         $this->assertSame($classReflection, $methodReflection->getDeclaringClass());
     }
 
-    public function testShouldBeStatic()
+    public function testShouldBeStatic(): void
     {
         $classReflection  = $this->broker->getClass(VisibilityEnum::class);
         $methodReflection = $this->reflectionExtension->getMethod($classReflection, 'STR');
 
         $this->assertTrue($methodReflection->isStatic());
     }
 
-    public function testShouldNotBePrivate()
+    public function testShouldNotBePrivate(): void
     {
         $classReflection  = $this->broker->getClass(VisibilityEnum::class);
         $methodReflection = $this->reflectionExtension->getMethod($classReflection, 'STR');
 
         $this->assertFalse($methodReflection->isPrivate());
     }
 
-    public function testShouldBePublic()
+    public function testShouldBePublic(): void
     {
         $classReflection  = $this->broker->getClass(VisibilityEnum::class);
         $methodReflection = $this->reflectionExtension->getMethod($classReflection, 'STR');
 
         $this->assertTrue($methodReflection->isPublic());
     }
 
-    public function testGetVariants()
+    public function testGetVariants(): void
     {
         $classReflection  = $this->broker->getClass(VisibilityEnum::class);
         $methodReflection = $this->reflectionExtension->getMethod($classReflection, 'STR');
@@ -72,7 +72,7 @@ public function testGetVariants()
         $this->assertSame(VisibilityEnum::class, $parametersAcceptor->getReturnType()->describe(VerbosityLevel::value()));
     }
 
-    public function testGetDocComment()
+    public function testGetDocComment(): void
     {
         $classReflection = $this->broker->getClass(DocCommentEnum::class);
         $docMethodRefl = $this->reflectionExtension->getMethod($classReflection, 'WITH_DOC_BLOCK');
@@ -88,7 +88,7 @@ public function testGetDocComment()
         $this->assertNotRegExp('/@var/', $docMethodRefl->getDocComment());
     }
 
-    public function testIsDeprecated()
+    public function testIsDeprecated(): void
     {
         $classReflection = $this->broker->getClass(DeprecatedEnum::class);
         $deprecatedRefl = $this->reflectionExtension->getMethod($classReflection, 'DEPRECATED');
@@ -98,7 +98,7 @@ public function testIsDeprecated()
         $this->assertTrue($notDeprecatedRefl->isDeprecated()->no());
     }
 
-    public function testGetDeprecatedDescription()
+    public function testGetDeprecatedDescription(): void
     {
         $classReflection = $this->broker->getClass(DeprecatedEnum::class);
         $deprecatedRefl = $this->reflectionExtension->getMethod($classReflection, 'DEPRECATED');
diff --git a/tests/unit/EnumMethodsClassReflectionExtensionTest.php b/tests/unit/EnumMethodsClassReflectionExtensionTest.php
@@ -22,13 +22,13 @@ class EnumMethodsClassReflectionExtensionTest extends TestCase
      */
     protected $reflectionExtension;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->broker = $this->createBroker();
         $this->reflectionExtension = new EnumMethodsClassReflectionExtension();
     }
 
-    public function testHasMethodSuccess()
+    public function testHasMethodSuccess(): void
     {
         $classReflection = $this->broker->getClass(VisibilityEnum::class);
 
@@ -37,19 +37,19 @@ public function testHasMethodSuccess()
         }
     }
 
-    public function testHasMethodUnknownNotFound()
+    public function testHasMethodUnknownNotFound(): void
     {
         $classReflection = $this->broker->getClass(VisibilityEnum::class);
         $this->assertFalse($this->reflectionExtension->hasMethod($classReflection, 'UNKNOWN'));
     }
 
-    public function testHasMethodNotSubclassOfEnumNotFound()
+    public function testHasMethodNotSubclassOfEnumNotFound(): void
     {
         $classReflection = $this->broker->getClass(NotAnEnum::class);
         $this->assertFalse($this->reflectionExtension->hasMethod($classReflection, 'STR'));
     }
 
-    public function testGetMethodSuccess()
+    public function testGetMethodSuccess(): void
     {
         $classReflection = $this->broker->getClass(VisibilityEnum::class);
 

Original file line number	Diff line number	Diff line change
`@@ -25,45 +25,45 @@ class EnumMethodReflectionTest extends TestCase`
`25`	`25`	`*/`
`26`	`26`	`protected $reflectionExtension;`
`27`	`27`
`28`		`- public function setUp()`
	`28`	`+ public function setUp(): void`
`29`	`29`	`{`
`30`	`30`	`$this->broker = $this->createBroker();`
`31`	`31`	`$this->reflectionExtension = new EnumMethodsClassReflectionExtension();`
`32`	`32`	`}`
`33`	`33`
`34`		`- public function getDeclaringClass()`
	`34`	`+ public function testGetDeclaringClass(): void`
`35`	`35`	`{`
`36`	`36`	`$classReflection = $this->broker->getClass(VisibilityEnum::class);`
`37`	`37`	`$methodReflection = $this->reflectionExtension->getMethod($classReflection, 'STR');`
`38`	`38`
`39`	`39`	`$this->assertSame($classReflection, $methodReflection->getDeclaringClass());`
`40`	`40`	`}`
`41`	`41`
`42`		`- public function testShouldBeStatic()`
	`42`	`+ public function testShouldBeStatic(): void`
`43`	`43`	`{`
`44`	`44`	`$classReflection = $this->broker->getClass(VisibilityEnum::class);`
`45`	`45`	`$methodReflection = $this->reflectionExtension->getMethod($classReflection, 'STR');`
`46`	`46`
`47`	`47`	`$this->assertTrue($methodReflection->isStatic());`
`48`	`48`	`}`
`49`	`49`
`50`		`- public function testShouldNotBePrivate()`
	`50`	`+ public function testShouldNotBePrivate(): void`
`51`	`51`	`{`
`52`	`52`	`$classReflection = $this->broker->getClass(VisibilityEnum::class);`
`53`	`53`	`$methodReflection = $this->reflectionExtension->getMethod($classReflection, 'STR');`
`54`	`54`
`55`	`55`	`$this->assertFalse($methodReflection->isPrivate());`
`56`	`56`	`}`
`57`	`57`
`58`		`- public function testShouldBePublic()`
	`58`	`+ public function testShouldBePublic(): void`
`59`	`59`	`{`
`60`	`60`	`$classReflection = $this->broker->getClass(VisibilityEnum::class);`
`61`	`61`	`$methodReflection = $this->reflectionExtension->getMethod($classReflection, 'STR');`
`62`	`62`
`63`	`63`	`$this->assertTrue($methodReflection->isPublic());`
`64`	`64`	`}`
`65`	`65`
`66`		`- public function testGetVariants()`
	`66`	`+ public function testGetVariants(): void`
`67`	`67`	`{`
`68`	`68`	`$classReflection = $this->broker->getClass(VisibilityEnum::class);`
`69`	`69`	`$methodReflection = $this->reflectionExtension->getMethod($classReflection, 'STR');`
`@@ -72,7 +72,7 @@ public function testGetVariants()`
`72`	`72`	`$this->assertSame(VisibilityEnum::class, $parametersAcceptor->getReturnType()->describe(VerbosityLevel::value()));`
`73`	`73`	`}`
`74`	`74`
`75`		`- public function testGetDocComment()`
	`75`	`+ public function testGetDocComment(): void`
`76`	`76`	`{`
`77`	`77`	`$classReflection = $this->broker->getClass(DocCommentEnum::class);`
`78`	`78`	`$docMethodRefl = $this->reflectionExtension->getMethod($classReflection, 'WITH_DOC_BLOCK');`
`@@ -88,7 +88,7 @@ public function testGetDocComment()`
`88`	`88`	`$this->assertNotRegExp('/@var/', $docMethodRefl->getDocComment());`
`89`	`89`	`}`
`90`	`90`
`91`		`- public function testIsDeprecated()`
	`91`	`+ public function testIsDeprecated(): void`
`92`	`92`	`{`
`93`	`93`	`$classReflection = $this->broker->getClass(DeprecatedEnum::class);`
`94`	`94`	`$deprecatedRefl = $this->reflectionExtension->getMethod($classReflection, 'DEPRECATED');`
`@@ -98,7 +98,7 @@ public function testIsDeprecated()`
`98`	`98`	`$this->assertTrue($notDeprecatedRefl->isDeprecated()->no());`
`99`	`99`	`}`
`100`	`100`
`101`		`- public function testGetDeprecatedDescription()`
	`101`	`+ public function testGetDeprecatedDescription(): void`
`102`	`102`	`{`
`103`	`103`	`$classReflection = $this->broker->getClass(DeprecatedEnum::class);`
`104`	`104`	`$deprecatedRefl = $this->reflectionExtension->getMethod($classReflection, 'DEPRECATED');`
Original file line number	Diff line number	Diff line change
`@@ -22,13 +22,13 @@ class EnumMethodsClassReflectionExtensionTest extends TestCase`
`22`	`22`	`*/`
`23`	`23`	`protected $reflectionExtension;`
`24`	`24`
`25`		`- public function setUp()`
	`25`	`+ public function setUp(): void`
`26`	`26`	`{`
`27`	`27`	`$this->broker = $this->createBroker();`
`28`	`28`	`$this->reflectionExtension = new EnumMethodsClassReflectionExtension();`
`29`	`29`	`}`
`30`	`30`
`31`		`- public function testHasMethodSuccess()`
	`31`	`+ public function testHasMethodSuccess(): void`
`32`	`32`	`{`
`33`	`33`	`$classReflection = $this->broker->getClass(VisibilityEnum::class);`
`34`	`34`
`@@ -37,19 +37,19 @@ public function testHasMethodSuccess()`
`37`	`37`	`}`
`38`	`38`	`}`
`39`	`39`
`40`		`- public function testHasMethodUnknownNotFound()`
	`40`	`+ public function testHasMethodUnknownNotFound(): void`
`41`	`41`	`{`
`42`	`42`	`$classReflection = $this->broker->getClass(VisibilityEnum::class);`
`43`	`43`	`$this->assertFalse($this->reflectionExtension->hasMethod($classReflection, 'UNKNOWN'));`
`44`	`44`	`}`
`45`	`45`
`46`		`- public function testHasMethodNotSubclassOfEnumNotFound()`
	`46`	`+ public function testHasMethodNotSubclassOfEnumNotFound(): void`
`47`	`47`	`{`
`48`	`48`	`$classReflection = $this->broker->getClass(NotAnEnum::class);`
`49`	`49`	`$this->assertFalse($this->reflectionExtension->hasMethod($classReflection, 'STR'));`
`50`	`50`	`}`
`51`	`51`
`52`		`- public function testGetMethodSuccess()`
	`52`	`+ public function testGetMethodSuccess(): void`
`53`	`53`	`{`
`54`	`54`	`$classReflection = $this->broker->getClass(VisibilityEnum::class);`
`55`	`55`