Merge pull request #338 from dmason30/switch-to-pcov

Add pcov coverage & experimental skip nova tests on PRs

Author: mikebronner

.travis.yml

@@ -23,16 +23,20 @@ matrix:
     - php: 7.4
 
 before_script:
-  - travis_retry composer self-update
-  - travis_retry composer config --global github-protocols https
-  - travis_retry composer config http-basic.nova.laravel.com $NOVA_USERNAME $NOVA_PASSWORD
-  - travis_retry composer install --no-interaction --prefer-source
+  - if php -v | grep -q 'Xdebug'; then phpenv config-rm xdebug.ini; fi
+  - pecl install pcov
+  - travis_retry composer self-update;
+  - travis_retry composer config --global github-protocols https;
+  - 'if [ "$TRAVIS_PULL_REQUEST" = "false" ]; then travis_retry composer config http-basic.nova.laravel.com $NOVA_USERNAME $NOVA_PASSWORD; fi'
+  - 'if [ "$TRAVIS_PULL_REQUEST" != "false" ]; then composer remove --dev laravel/nova --no-interaction --no-update; fi'
+  - travis_retry composer install --no-interaction --prefer-source;
   - psql -c "CREATE DATABASE testing;" -U postgres
   - psql -c "CREATE USER homestead WITH PASSWORD 'secret';" -U postgres
 
 script:
   - mkdir -p build/logs
-  - php vendor/bin/phpunit -c phpunit.xml.dist  --coverage-clover build/logs/clover.xml
+  - 'if [ "$TRAVIS_PULL_REQUEST" != "false" ]; then php vendor/bin/phpunit -c phpunit.xml.dist --coverage-clover build/logs/clover.xml --exclude-group nova; fi'
+  - 'if [ "$TRAVIS_PULL_REQUEST" = "false" ]; then php vendor/bin/phpunit -c phpunit.xml.dist --coverage-clover build/logs/clover.xml; fi'
 
 after_success:
   - travis_retry php vendor/bin/php-coveralls -v

composer.json

@@ -36,7 +36,8 @@
         "phpunit/phpunit": "^8.0",
         "sebastian/phpcpd": "^5.0",
         "squizlabs/php_codesniffer": "^3.4",
-        "symfony/thanks": "^1.2"
+        "symfony/thanks": "^1.2",
+        "doctrine/dbal": "^2.10"
     },
     "autoload": {
         "psr-4": {

tests/Feature/Nova/BelongsToManyTest.php

@@ -3,9 +3,9 @@
 use GeneaLabs\LaravelModelCaching\Tests\Fixtures\Book;
 use GeneaLabs\LaravelModelCaching\Tests\Fixtures\Store;
 
+/** @group nova */
 class BelongsToManyTest extends NovaTestCase
 {
-    /** @group test */
     public function testAttachRelationFlushesCache()
     {
         $beforeStore = Store::with(['books'])->get()->first();
@@ -35,7 +35,6 @@ public function testAttachRelationFlushesCache()
         $this->assertSame('new foo', $book->title);
     }
 
-    /** @group test */
     public function testDetachRelationFlushesCache()
     {
         /** @var Store $store */
@@ -66,7 +65,6 @@ public function testDetachRelationFlushesCache()
         $this->assertCount(1, $books);
     }
 
-    /** @group test */
     public function testUpdateRelationFlushesCache()
     {
         $beforeStore = Store::with(['books'])->get()->first();
@@ -86,7 +84,6 @@ public function testUpdateRelationFlushesCache()
         $this->assertSame('foo', $book->title);
     }
 
-    /** @group test */
     public function testDeleteRelationFlushesCache()
     {
         $beforeStore = Store::with(['books'])->get()->first();

tests/Feature/Nova/CreateTest.php

@@ -2,6 +2,7 @@
 
 use GeneaLabs\LaravelModelCaching\Tests\Fixtures\Author;
 
+/** @group nova */
 class CreateTest extends NovaTestCase
 {
     public function testCreateFlushesCacheForModel()

tests/Feature/Nova/DeleteTest.php

@@ -2,6 +2,7 @@
 
 use GeneaLabs\LaravelModelCaching\Tests\Fixtures\Author;
 
+/** @group nova */
 class DeleteTest extends NovaTestCase
 {
     public function testDeleteFlushesCacheForModel()

tests/Feature/Nova/UpdateTest.php

@@ -2,6 +2,7 @@
 
 use GeneaLabs\LaravelModelCaching\Tests\Fixtures\Author;
 
+/** @group nova */
 class UpdateTest extends NovaTestCase
 {
     public function testUpdateFlushesCacheForModel()