Add StopWords tests and improve CI workflow

Author: danielebarbaro

.github/workflows/test.yml

@@ -9,13 +9,29 @@ on:
 permissions:
   contents: read
 
-jobs:
-  build:
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
 
+jobs:
+  test:
     runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        php: ['8.1', '8.3', '8.4', '8.5']
+
+    name: PHP ${{ matrix.php }}
 
     steps:
-    - uses: actions/checkout@v4
+    - name: Checkout
+      uses: actions/checkout@v5
+
+    - name: Setup PHP
+      uses: shivammathur/setup-php@v2
+      with:
+        php-version: ${{ matrix.php }}
+        coverage: pcov
 
     - name: Validate composer.json and composer.lock
       run: composer validate --strict
@@ -37,3 +53,11 @@ jobs:
 
     - name: Run test suite
       run: composer run-script test
+
+    - name: Run test suite with coverage
+      run: ./vendor/bin/phpunit --coverage-clover coverage.xml
+
+    - name: Upload coverage to Codecov
+      uses: codecov/codecov-action@v4
+      with:
+        files: coverage.xml

.gitignore

@@ -4,4 +4,7 @@ composer.lock
 
 # PHPUnit
 .phpunit.result.cache
-.phpunit.cache/
+.phpunit.cache/
+
+# Coverage
+.coverage/

composer.json

@@ -25,6 +25,9 @@
         "test" : [
             "vendor/bin/phpunit"
         ],
+        "coverage": [
+            "vendor/bin/phpunit --coverage-html .coverage/"
+        ],
         "phpstan": [
             "vendor/bin/phpstan analyse -l 5 src tests benchmark"
         ]

src/BM25/StopWords/FileStopWords.php

@@ -59,7 +59,7 @@ public function getStopWords(): array
      */
     private function loadFromFile(): array
     {
-        $content = file_get_contents($this->filePath);
+        $content = @file_get_contents($this->filePath);
         if ($content === false) {
             throw new \RuntimeException(
                 sprintf('Failed to read stop words file: %s', $this->filePath)