Update CI setup to ensure 100% code coverage

Author: clue

.github/workflows/ci.yml

@@ -31,7 +31,14 @@ jobs:
           coverage: xdebug
           ini-file: development
       - run: composer install
-      - run: vendor/bin/phpunit --coverage-text
+      - run: docker pull busybox:latest
+      - run: vendor/bin/phpunit --coverage-text --coverage-clover=clover.xml
         if: ${{ matrix.php >= 7.3 }}
-      - run: vendor/bin/phpunit --coverage-text -c phpunit.xml.legacy
+      - run: vendor/bin/phpunit --coverage-text --coverage-clover=clover.xml -c phpunit.xml.legacy
         if: ${{ matrix.php < 7.3 }}
+      - name: Check 100% code coverage
+        shell: php {0}
+        run: |
+          <?php
+          $metrics = simplexml_load_file('clover.xml')->project->metrics;
+          exit((int) $metrics['statements'] === (int) $metrics['coveredstatements'] ? 0 : 1);

tests/FunctionalClientTest.php

@@ -90,8 +90,7 @@ public function testCreateStartAndRemoveContainer()
         // get all events between starting and removing for this container
         $promise = $this->client->events($start, $end, array('container' => array($container['Id'])));
         $ret = \React\Async\await($promise);
-
-        $this->assertIsArray($ret);
+        assert(is_array($ret));
 
         $status = array(); // array_column($ret, 'status'); // PHP 5.5+
         foreach ($ret as $one) {