Merge pull request #5 from php-api-clients/feature-error-handling

Error handling

Author: WyriHaximus

Makefile

@@ -27,6 +27,3 @@ unit-coverage:
 
 ci-coverage: init
 	composer ci-coverage
-
-mutation:
-	composer mutation

README.md

@@ -47,6 +47,8 @@ Currently the only provided locator is the `ContainerLocator` which accepts a
         return resolve($response);
     })->then(function (ResponseInterface $response) use ($runner) {
         return $runner->post($response);
+    })->otherwise(function (Throwable $throwable) use ($runner) {
+        return reject($runner->error($throwable));
     });
 ```
 

appveyor.yml

@@ -19,9 +19,13 @@ environment:
   - dependencies: highest
     php_ver_target: 7.1
 
-## Cache composer bits
+## Cache composer, chocolatey and php bits
 cache:
     - '%LOCALAPPDATA%\Composer\files -> composer.lock'
+    - composer.phar
+    - C:\ProgramData\chocolatey\bin -> .appveyor.yml
+    - C:\ProgramData\chocolatey\lib -> .appveyor.yml
+    - c:\tools\php -> .appveyor.yml
 
 ## Set up environment varriables
 init:
@@ -33,7 +37,7 @@ init:
 ## Install PHP and composer, and run the appropriate composer command
 install:
     - IF EXIST c:\tools\php (SET PHP=0)
-    - ps: appveyor-retry cinst -y php --version ((choco search php --exact --all-versions -r | select-string -pattern $Env:php_ver_target | Select-Object -first 1) -replace '[php|]','')
+    - ps: appveyor-retry cinst --params '""/InstallDir:C:\tools\php""' --ignore-checksums -y php --version ((choco search php --exact --all-versions -r | select-string -pattern $env:php_ver_target | sort { [version]($_ -split '\|' | select -last 1) } -Descending | Select-Object -first 1) -replace '[php|]','')
     - cd c:\tools\php
     - IF %PHP%==1 copy php.ini-production php.ini /Y
     - IF %PHP%==1 echo date.timezone="UTC" >> php.ini
@@ -52,4 +56,4 @@ install:
 ## Run the actual test
 test_script:
     - cd c:\projects\php-project-workspace
-    - vendor/bin/phpunit -c phpunit.xml.dist
+    - vendor/bin/phpunit -c phpunit.xml.dist

composer.json

@@ -16,7 +16,7 @@
     "react/promise": "^2.4"
   },
   "require-dev": {
-    "api-clients/test-utilities": "^1.0",
+    "api-clients/test-utilities": "^3.0",
     "container-interop/container-interop": "^1.1",
     "guzzlehttp/psr7": "^1.3"
   },
@@ -54,25 +54,19 @@
       "@ensure-installed",
       "phpunit --colors=always -c phpunit.xml.dist --coverage-text --coverage-html covHtml --coverage-clover ./build/logs/clover.xml"
     ],
-    "mutation": [
-      "@ensure-installed",
-      "if [ `php -m | grep xdebug | wc -l` -gt 0 ]; then humbug --ansi --incremental; fi"
-    ],
     "lint-php": [
       "@ensure-installed",
       "parallel-lint --exclude vendor ."
     ],
     "qa-all": [
       "@lint-php",
       "@cs",
-      "@unit",
-      "@mutation"
+      "@unit"
     ],
     "qa-all-coverage": [
       "@lint-php",
       "@cs",
-      "@unit-coverage",
-      "@mutation"
+      "@unit-coverage"
     ],
     "qa-windows": [
       "@lint-php",