Codecov

PHPacto · Nov 6, 2017 · 07420fb · 07420fb
1 parent 714f249
commit 07420fb
Show file tree

Hide file tree

Showing 46 changed files with 25 additions and 148 deletions.
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -7,7 +7,7 @@ jobs:
   build:
     docker:
       # specify the version you desire here
-      - image: circleci/php:7.1-browsers
+      - image: circleci/php:7.1-cli
 
       # Specify service dependencies here if necessary
       # CircleCI maintains a library of pre-built images
@@ -29,9 +29,18 @@ jobs:
       - run: composer install -n --prefer-dist
 
       - save_cache:
+          key: v1-dependencies-{{ checksum "composer.json" }}
           paths:
             - ./vendor
-          key: v1-dependencies-{{ checksum "composer.json" }}
 
-      # run tests!
-      - run: ./vendor/bin/phpunit
+      - run:
+          name: Enable xdebug
+          command: sudo docker-php-ext-enable xdebug
+
+      - run:
+          name: Run Unit tests
+          command: vendor/bin/phpunit --coverage-clover=coverage.xml
+
+      - run:
+          name: Codecov
+          command: bash <(curl -s https://codecov.io/bash)
diff --git a/tests/Command/BuilderValidateContractTest.php b/tests/Command/BuilderValidateContractTest.php
@@ -30,9 +30,6 @@
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\Console\Tester\CommandTester;
 
-/**
- * @coversNothing
- */
 class BuilderValidateContractTest extends TestCase
 {
     /**

diff --git a/tests/Command/BuilderWriteContractTest.php b/tests/Command/BuilderWriteContractTest.php
@@ -30,9 +30,6 @@
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\Console\Tester\CommandTester;
 
-/**
- * @coversNothing
- */
 class BuilderWriteContractTest extends TestCase
 {
     /**
@@ -83,7 +80,7 @@ public function test_it_runs_pact_builder_and_throws_exception_if_no_pact_is_ret
             return;
         }
 
-        self::assertFalse(true, 'This test should end in the catch');
+        self::fail('This test should end in the catch');
     }
 
     public function test_it_reads_directory_and_builds_writes_contracts()

diff --git a/tests/Command/ValidateContractTest.php b/tests/Command/ValidateContractTest.php
@@ -30,9 +30,6 @@
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\Console\Tester\CommandTester;
 
-/**
- * @coversNothing
- */
 class ValidateContractTest extends TestCase
 {
     /**

diff --git a/tests/Factory/PactRequestFactoryTest.php b/tests/Factory/PactRequestFactoryTest.php
@@ -25,9 +25,6 @@
 use Zend\Diactoros\Request;
 use Zend\Diactoros\Stream;
 
-/**
- * @coversNothing
- */
 class PactRequestFactoryTest extends TestCase
 {
     public function test_it_returns_pact_request_minimal()

diff --git a/tests/Factory/PactResponseFactoryTest.php b/tests/Factory/PactResponseFactoryTest.php
@@ -25,9 +25,6 @@
 use Zend\Diactoros\Response;
 use Zend\Diactoros\Stream;
 
-/**
- * @coversNothing
- */
 class PactResponseFactoryTest extends TestCase
 {
     public function test_it_returns_pact_response_minimal()

diff --git a/tests/Factory/SerializerFactoryTest.php b/tests/Factory/SerializerFactoryTest.php
@@ -23,9 +23,6 @@
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\Serializer\Serializer;
 
-/**
- * @coversNothing
- */
 class SerializerFactoryTest extends TestCase
 {
     public function test_it_returns_serializer()

diff --git a/tests/Guzzle/ServerMock5Test.php b/tests/Guzzle/ServerMock5Test.php
@@ -29,7 +29,6 @@
 
 /**
  * @group guzzle
- * @coversNothing
  */
 class ServerMock5Test extends TestCase
 {
@@ -78,7 +77,7 @@ public function test_it_throws_mismatch_if_request_not_match()
             return;
         }
 
-        self::assertFalse(true, 'This test should end in the catch');
+        self::fail('This test should end in the catch');
     }
 
     /**

diff --git a/tests/Guzzle/ServerMock6Test.php b/tests/Guzzle/ServerMock6Test.php
@@ -29,7 +29,6 @@
 
 /**
  * @group guzzle
- * @coversNothing
  */
 class ServerMock6Test extends TestCase
 {

diff --git a/tests/Loader/FileLoaderTest.php b/tests/Loader/FileLoaderTest.php
@@ -27,9 +27,6 @@
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\Serializer\Serializer;
 
-/**
- * @coversNothing
- */
 class FileLoaderTest extends TestCase
 {
     /**

diff --git a/tests/Matcher/BodyMatcherTest.php b/tests/Matcher/BodyMatcherTest.php
@@ -25,9 +25,6 @@
 use Zend\Diactoros\Request;
 use Zend\Diactoros\Stream;
 
-/**
- * @coversNothing
- */
 class BodyMatcherTest extends TestCase
 {
     /** @var BodyMatcher */
@@ -130,7 +127,7 @@ public function test_it_throws_mismatch_if_key_is_missing_with_body_url_encoded(
             return;
         }
 
-        self::assertFalse(true, 'This test should end in the catch');
+        self::fail('This test should end in the catch');
     }
 
     /**
@@ -159,7 +156,7 @@ public function test_it_throws_mismatch_if_key_is_missing_with_body_json_encoded
             return;
         }
 
-        self::assertFalse(true, 'This test should end in the catch');
+        self::fail('This test should end in the catch');
     }
 
     /**
@@ -186,7 +183,7 @@ public function test_it_throws_mismatch_if_value_doesnt_match_with_body_plain_st
             return;
         }
 
-        self::assertFalse(true, 'This test should end in the catch');
+        self::fail('This test should end in the catch');
     }
 
     /**
@@ -215,7 +212,7 @@ public function test_it_throws_mismatch_if_value_doesnt_match_with_body_url_enco
             return;
         }
 
-        self::assertFalse(true, 'This test should end in the catch');
+        self::fail('This test should end in the catch');
     }
 
     /**
@@ -244,6 +241,6 @@ public function test_it_throws_mismatch_if_value_doesnt_match_with_body_json_enc
             return;
         }
 
-        self::assertFalse(true, 'This test should end in the catch');
+        self::fail('This test should end in the catch');
     }
 }
diff --git a/tests/Matcher/HeadersMatcherTest.php b/tests/Matcher/HeadersMatcherTest.php
@@ -24,9 +24,6 @@
 use PHPUnit\Framework\TestCase;
 use Zend\Diactoros\Request;
 
-/**
- * @coversNothing
- */
 class HeadersMatcherTest extends TestCase
 {
     /** @var HeadersMatcher */
@@ -80,7 +77,7 @@ public function test_it_throws_mismatch_if_key_is_missing()
             return;
         }
 
-        self::assertFalse(true, 'This test should end in the catch');
+        self::fail('This test should end in the catch');
     }
 
     /**
@@ -104,6 +101,6 @@ public function test_it_throws_mismatch_if_value_doesnt_match()
             return;
         }
 
-        self::assertFalse(true, 'This test should end in the catch');
+        self::fail('This test should end in the catch');
     }
 }
diff --git a/tests/Matcher/Mismatches/KeyNotFoundMismatchTest.php b/tests/Matcher/Mismatches/KeyNotFoundMismatchTest.php
@@ -22,9 +22,6 @@
 
 use PHPUnit\Framework\TestCase;
 
-/**
- * @coversNothing
- */
 class KeyNotFoundMismatchTest extends TestCase
 {
     public function test_has_message_string()

diff --git a/tests/Matcher/Mismatches/MismatchCollectionTest.php b/tests/Matcher/Mismatches/MismatchCollectionTest.php
@@ -22,9 +22,6 @@
 
 use PHPUnit\Framework\TestCase;
 
-/**
- * @coversNothing
- */
 class MismatchCollectionTest extends TestCase
 {
     public function test_it_is_countable_and_iterable()

diff --git a/tests/Matcher/Mismatches/TypeMismatchTest.php b/tests/Matcher/Mismatches/TypeMismatchTest.php
@@ -22,9 +22,6 @@
 
 use PHPUnit\Framework\TestCase;
 
-/**
- * @coversNothing
- */
 class TypeMismatchTest extends TestCase
 {
     public function test_has_message_string()

diff --git a/tests/Matcher/Mismatches/ValueMismatchTest.php b/tests/Matcher/Mismatches/ValueMismatchTest.php
@@ -22,9 +22,6 @@
 
 use PHPUnit\Framework\TestCase;
 
-/**
- * @coversNothing
- */
 class ValueMismatchTest extends TestCase
 {
     public function test_has_message_string()

diff --git a/tests/Matcher/Rules/AbstractRuleTest.php b/tests/Matcher/Rules/AbstractRuleTest.php
@@ -20,9 +20,6 @@
 
 namespace Bigfoot\PHPacto\Matcher\Rules;
 
-/**
- * @coversNothing
- */
 class AbstractRuleTest extends RuleAbstractTest
 {
     public function test_it_has_a_value_and_a_sample()

diff --git a/tests/Matcher/Rules/AbstractStringRuleTest.php b/tests/Matcher/Rules/AbstractStringRuleTest.php
@@ -20,9 +20,6 @@
 
 namespace Bigfoot\PHPacto\Matcher\Rules;
 
-/**
- * @coversNothing
- */
 class AbstractStringRuleTest extends RuleAbstractTest
 {
     public function test_it_is_not_case_sensitive_by_default()

diff --git a/tests/Matcher/Rules/AndRuleTest.php b/tests/Matcher/Rules/AndRuleTest.php
@@ -22,9 +22,6 @@
 
 use Bigfoot\PHPacto\Matcher\Mismatches;
 
-/**
- * @coversNothing
- */
 class AndRuleTest extends RuleAbstractTest
 {
     public function test_it_is_normalizable()
@@ -138,6 +135,6 @@ public function testMismatchCount()
             throw $e;
         }
 
-        self::assertFalse(true, 'This test should end in the catch');
+        self::fail('This test should end in the catch');
     }
 }
diff --git a/tests/Matcher/Rules/ContainsRuleTest.php b/tests/Matcher/Rules/ContainsRuleTest.php
@@ -22,9 +22,6 @@
 
 use Bigfoot\PHPacto\Matcher\Mismatches;
 
-/**
- * @coversNothing
- */
 class ContainsRuleTest extends RuleAbstractTest
 {
     public function test_it_is_normalizable()

diff --git a/tests/Matcher/Rules/CountRuleTest.php b/tests/Matcher/Rules/CountRuleTest.php
@@ -22,9 +22,6 @@
 
 use Bigfoot\PHPacto\Matcher\Mismatches;
 
-/**
- * @coversNothing
- */
 class CountRuleTest extends RuleAbstractTest
 {
     public function test_it_is_normalizable()

diff --git a/tests/Matcher/Rules/DateTimeRuleTest.php b/tests/Matcher/Rules/DateTimeRuleTest.php
@@ -22,9 +22,6 @@
 
 use Bigfoot\PHPacto\Matcher\Mismatches;
 
-/**
- * @coversNothing
- */
 class DateTimeRuleTest extends RuleAbstractTest
 {
     public function test_it_is_normalizable()

diff --git a/tests/Matcher/Rules/EachRuleTest.php b/tests/Matcher/Rules/EachRuleTest.php
@@ -22,9 +22,6 @@
 
 use Bigfoot\PHPacto\Matcher\Mismatches;
 
-/**
- * @coversNothing
- */
 class EachRuleTest extends RuleAbstractTest
 {
     public function test_it_is_normalizable()

diff --git a/tests/Matcher/Rules/EqualsRuleTest.php b/tests/Matcher/Rules/EqualsRuleTest.php
@@ -22,9 +22,6 @@
 
 use Bigfoot\PHPacto\Matcher\Mismatches;
 
-/**
- * @coversNothing
- */
 class EqualsRuleTest extends RuleAbstractTest
 {
     public function test_it_is_normalizable()

diff --git a/tests/Matcher/Rules/GreaterOrEqualRuleTest.php b/tests/Matcher/Rules/GreaterOrEqualRuleTest.php
@@ -22,9 +22,6 @@
 
 use Bigfoot\PHPacto\Matcher\Mismatches;
 
-/**
- * @coversNothing
- */
 class GreaterOrEqualRuleTest extends RuleAbstractTest
 {
     public function test_it_is_normalizable()

diff --git a/tests/Matcher/Rules/GreaterRuleTest.php b/tests/Matcher/Rules/GreaterRuleTest.php
@@ -22,9 +22,6 @@
 
 use Bigfoot\PHPacto\Matcher\Mismatches;
 
-/**
- * @coversNothing
- */
 class GreaterRuleTest extends RuleAbstractTest
 {
     public function test_it_is_normalizable()

diff --git a/tests/Matcher/Rules/LowerOrEqualRuleTest.php b/tests/Matcher/Rules/LowerOrEqualRuleTest.php
@@ -22,9 +22,6 @@
 
 use Bigfoot\PHPacto\Matcher\Mismatches;
 
-/**
- * @coversNothing
- */
 class LowerOrEqualRuleTest extends RuleAbstractTest
 {
     public function test_it_is_normalizable()

diff --git a/tests/Matcher/Rules/LowerRuleTest.php b/tests/Matcher/Rules/LowerRuleTest.php
@@ -22,9 +22,6 @@
 
 use Bigfoot\PHPacto\Matcher\Mismatches;
 
-/**
- * @coversNothing
- */
 class LowerRuleTest extends RuleAbstractTest
 {
     public function test_it_is_normalizable()

diff --git a/tests/Matcher/Rules/NotEqualsRuleTest.php b/tests/Matcher/Rules/NotEqualsRuleTest.php
@@ -22,9 +22,6 @@
 
 use Bigfoot\PHPacto\Matcher\Mismatches;
 
-/**
- * @coversNothing
- */
 class NotEqualsRuleTest extends RuleAbstractTest
 {
     public function test_it_is_normalizable()