Added mutation testing.

Author: Bilge

.gitignore

@@ -1,3 +1,4 @@
-/vendor/
-/.*/
-/composer.lock
+/vendor/
+/.*/
+/composer.lock
+/*.log

.travis.yml

@@ -23,12 +23,13 @@ cache:
     - vendor
 
 install:
-  - alias composer=composer\ -n && composer selfupdate
   - composer validate
-  - composer update $DEPENDENCIES
+  - composer --no-interaction update $DEPENDENCIES
 
 script:
-  - composer test -- --coverage-clover=build/logs/clover.xml
+  - composer test -- --coverage-clover=build/logs/clover.xml --coverage-xml=build/coverage/coverage-xml
+    --log-junit=build/coverage/phpunit.junit.xml
+  - vendor/bin/infection --min-msi=95 --threads=$(nproc) --coverage=build/coverage
 
 after_success:
   - bash <(curl -s https://codecov.io/bash)

README.md

@@ -4,6 +4,7 @@ Porter <img src="https://github.com/ScriptFUSION/Porter/wiki/images/porter%20222
 [![Latest version][Version image]][Releases]
 [![Total downloads][Downloads image]][Downloads]
 [![Build status][Build image]][Build]
+[![Mutation score][MSI image]][Build]
 [![Test coverage][Coverage image]][Coverage]
 [![Code style][Style image]][Style]
 
@@ -580,6 +581,7 @@ Porter is published under the open source GNU Lesser General Public License v3.0
   [Downloads image]: https://poser.pugx.org/scriptfusion/porter/downloads "Total downloads"
   [Build]: http://travis-ci.org/ScriptFUSION/Porter
   [Build image]: https://travis-ci.org/ScriptFUSION/Porter.svg?branch=master "Build status"
+  [MSI image]: https://badge.stryker-mutator.io/github.com/ScriptFUSION/Porter/master
   [Coverage]: https://codecov.io/gh/ScriptFUSION/Porter
   [Coverage image]: https://codecov.io/gh/ScriptFUSION/Porter/branch/master/graphs/badge.svg "Test coverage"
   [Style]: https://styleci.io/repos/49824895

composer.json

@@ -19,8 +19,9 @@
   },
   "require-dev": {
     "amphp/phpunit-util": "^1.1",
-    "phpunit/phpunit": "^7.1.3",
-    "mockery/mockery": "^1.1"
+    "infection/infection": "^0.13",
+    "mockery/mockery": "^1.1",
+    "phpunit/phpunit": "^7.1.3"
   },
   "suggest" : {
     "connectors/http": "Provides an HTTP connector for Porter providers.",
@@ -38,5 +39,8 @@
   },
   "scripts": {
     "test": "phpunit -c test"
+  },
+  "config": {
+    "sort-packages": true
   }
 }

infection.json

@@ -0,0 +1,20 @@
+{
+    "timeout": 10,
+    "source": {
+        "directories": [
+            "src"
+        ]
+    },
+    "phpUnit": {
+        "configDir": "test"
+    },
+    "logs": {
+        "text": "infection.log",
+        "badge": {
+            "branch": "5.0"
+        }
+    },
+    "mutators": {
+        "@default": true
+    }
+}

src/Porter.php

@@ -104,7 +104,7 @@ public function importOne(ImportSpecification $specification): ?array
     private function fetch(ImportSpecification $specification): \Iterator
     {
         $resource = $specification->getResource();
-        $provider = $this->getProvider($specification->getProviderName() ?: $resource->getProviderClassName());
+        $provider = $this->getProvider($specification->getProviderName() ?? $resource->getProviderClassName());
 
         if ($resource->getProviderClassName() !== \get_class($provider)) {
             throw new ForeignResourceException(sprintf(
@@ -175,7 +175,7 @@ public function importOneAsync(AsyncImportSpecification $specification): Promise
     private function fetchAsync(AsyncImportSpecification $specification): Iterator
     {
         $resource = $specification->getAsyncResource();
-        $provider = $this->getProvider($specification->getProviderName() ?: $resource->getProviderClassName());
+        $provider = $this->getProvider($specification->getProviderName() ?? $resource->getProviderClassName());
 
         if (!$provider instanceof AsyncProvider) {
             // TODO: Specific exception type.

src/Specification/Specification.php

@@ -102,7 +102,7 @@ final public function getTransformers(): array
      *
      * @return $this
      */
-    public function addTransformer(AnysyncTransformer $transformer): self
+    protected function addTransformer(AnysyncTransformer $transformer): self
     {
         if ($this->hasTransformer($transformer)) {
             throw new DuplicateTransformerException('Transformer already added.');

src/Transform/AsyncTransformer.php

@@ -5,7 +5,16 @@
 
 use ScriptFUSION\Porter\Collection\AsyncRecordCollection;
 
+/**
+ * Provides a method to asynchronously transform imported data.
+ */
 interface AsyncTransformer extends AnysyncTransformer
 {
+    /**
+     * Transforms the specified asynchronous record collection, decorated with the specified context data.
+     *
+     * @param AsyncRecordCollection $records Asynchronous Record collection.
+     * @param mixed $context Context data.
+     */
     public function transformAsync(AsyncRecordCollection $records, $context): AsyncRecordCollection;
 }

src/Transform/Transformer.php

@@ -11,7 +11,7 @@
 interface Transformer extends AnysyncTransformer
 {
     /**
-     * Transforms the specified record collection decorated with the specified context data.
+     * Transforms the specified record collection, decorated with the specified context data.
      *
      * @param RecordCollection $records Record collection.
      * @param mixed $context Context data.

test/Integration/Porter/PorterTest.php

@@ -66,7 +66,7 @@ protected function setUp(): void
 
     protected function registerProvider(Provider $provider, string $name = null): void
     {
-        $name = $name ?: \get_class($provider);
+        $name = $name ?? \get_class($provider);
 
         $this->container
             ->shouldReceive('has')->with($name)->andReturn(true)

test/phpunit.xml

@@ -1,7 +1,7 @@
 <phpunit
     beStrictAboutOutputDuringTests="true"
 >
-    <testsuite>
+    <testsuite name="all">
         <directory>.</directory>
     </testsuite>
     <filter>