Merge pull request #3 from The-3Labs-Team/add-test-coverage

Add test coverage

.github/workflows/run-tests.yml

@@ -41,12 +41,25 @@ jobs:
         with:
           php-version: ${{ matrix.php }}
           extensions: dom, curl, libxml, mbstring, zip, pcntl, pdo, sqlite, pdo_sqlite, bcmath, soap, intl, gd, exif, iconv, imagick, fileinfo
-          coverage: none
+          coverage: xdebug
 
       - name: Install dependencies
         run: |
           composer require "laravel/framework:${{ matrix.laravel }}" --no-interaction --no-update
           composer update --${{ matrix.stability }} --prefer-dist --no-interaction
 
-      - name: Execute tests
-        run: vendor/bin/pest --ci
+      - name: Setup Code Climate
+        uses: amancevice/setup-code-climate@v1
+        with:
+          cc_test_reporter_id: ${{ secrets.CC_TEST_REPORTER_ID }}
+
+      - name: Install Code Climate Test Reporter
+        run: cc-test-reporter before-build
+
+      - name: Run Tests
+        run: |
+          vendor/bin/pest --coverage-clover build/logs/clover.xml
+
+      - name: Upload coverage reports to Code Climate
+        run: |
+          cc-test-reporter after-build

src/Readability.php

@@ -9,8 +9,13 @@
 
 class Readability
 {
-    private $content;
+    private ?FFReadability $content;
 
+    /**
+     * Parse the content
+     *
+     * @throws Exception
+     */
     public function parse(string $content): self
     {
         $this->content = new FFReadability(new Configuration());