Upgraded PHPUnit 9 -> 10 and fixed tests.

Removed Travis config in lieu of GHA.
Dropped support for PHP < 8.1.

Author: Bilge

Diff for: .github/workflows/Tests.yaml

@@ -0,0 +1,55 @@
+name: Tests
+
+on:
+  push:
+  pull_request:
+  workflow_dispatch:
+  schedule:
+    - cron: 0 6 * * *
+
+jobs:
+  Test:
+    runs-on: ubuntu-latest
+
+    strategy:
+      fail-fast: false
+      matrix:
+        php:
+          - 8.1
+          - 8.2
+          - 8.3
+        dependencies:
+          - hi
+          - lo
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Setup PHP ${{ matrix.php }}
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php }}
+          coverage: xdebug
+
+      - name: Validate composer.json
+        run: composer validate
+
+      - name: Cache dependencies
+        id: composer-cache
+        uses: actions/cache@v4
+        with:
+          path: vendor
+          key: php-${{ matrix.php }}-${{ matrix.dependencies }}-${{ hashFiles('composer.json') }}
+          restore-keys: php-${{ matrix.php }}-${{ matrix.dependencies }}-
+
+      - name: Install dependencies ${{ matrix.dependencies == 'lo' && '(lowest)' || '' }}
+        run: composer update --no-interaction --no-progress
+          ${{ matrix.dependencies == 'lo' && '--prefer-lowest' || '' }}
+
+      - name: Run test suite with coverage
+        run: composer test -- --coverage-clover=build/logs/clover.xml
+
+      - name: Upload test coverage
+        uses: codecov/codecov-action@v4
+        env:
+          CODECOV_TOKEN: ${{ secrets.CODECOV_ORG_TOKEN }}

Diff for: .gitignore

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

Diff for: .travis.yml

@@ -926,7 +926,7 @@ Debug(Strategy|Mapping|array|mixed $expression)
 Requirements
 ------------
 
- - [PHP 5.6](http://php.net/)
+ - [PHP 8.1](http://php.net/)
  - [Composer](https://getcomposer.org/)
 
 Limitations
@@ -947,8 +947,8 @@ in this document can be found in `DocumentationTest`.
   [Version image]: https://poser.pugx.org/scriptfusion/mapper/version "Latest version"
   [Downloads]: https://packagist.org/packages/scriptfusion/mapper
   [Downloads image]: https://poser.pugx.org/scriptfusion/mapper/downloads "Total downloads"
-  [Build]: http://travis-ci.org/ScriptFUSION/Mapper
-  [Build image]: https://travis-ci.org/ScriptFUSION/Mapper.svg?branch=master "Build status"
+  [Build]: https://github.com/ScriptFUSION/Mapper/actions/workflows/Tests.yaml
+  [Build image]: https://github.com/ScriptFUSION/Mapper/actions/workflows/Tests.yaml/badge.svg "Build status"
   [Coverage]: https://coveralls.io/github/ScriptFUSION/Mapper
   [Coverage image]: https://coveralls.io/repos/ScriptFUSION/Mapper/badge.svg "Test coverage"
   [Style]: https://styleci.io/repos/59734709

Diff for: README.md

@@ -9,13 +9,13 @@
   ],
   "license": "LGPL-3.0",
   "require": {
-    "php": "^7.2|^8",
+    "php": "^8.1",
     "scriptfusion/array-walker": "^1",
     "eloquent/enumeration": "^5|^6"
   },
   "require-dev": {
     "scriptfusion/static-class": "^1",
-    "phpunit/phpunit": "^8.5|^9",
+    "phpunit/phpunit": "^10.5",
     "mockery/mockery": "^1.3.3"
   },
   "autoload": {

Diff for: composer.json

@@ -44,7 +44,7 @@ public function __invoke($record, $context = null)
         // Resolve the path expression. Path will always be an array after this block.
         if (!is_array($path = parent::__invoke($record, $context))) {
             // If it's not an array treat it as a delimited string; implicitly casts other scalar types.
-            $path = explode(self::PATH_SEPARATOR, $path);
+            $path = explode(self::PATH_SEPARATOR, (string)$path);
         }
 
         // Overwrite record with resolved data expression if set and ensure it is an array.

Diff for: src/Strategy/Copy.php

@@ -19,7 +19,7 @@ class Replace extends Delegate
      * Any number of searches and replacements can be specified. Searches and replacements are parsed in pairs. If no
      * replacements are specified, all matches are removed instead of replaced. If fewer replacements than searches are
      * specified, the last replacement will be used for the remaining searches. If more replacements than searches are
-     * specified, the extra replacements are ignored.
+     * specified, the extra replacements will be ignored.
      *
      * @param Strategy|Mapping|array|mixed $expression Expression to search in.
      * @param $searches string|Expression|array Search string(s).
@@ -40,7 +40,7 @@ public function __invoke($data, $context = null)
         $replace = null;
 
         foreach ($this->searches as $search) {
-            $replace = count($replacements) ? array_shift($replacements) : $replace;
+            $replace = count($replacements) ? array_shift($replacements) : (string)$replace;
 
             if ($search instanceof Expression) {
                 $output = preg_replace($search, $replace, $output);

Diff for: src/Strategy/Replace.php

@@ -53,7 +53,7 @@ public function testMapScalars($scalar)
         self::assertSame($scalar, $mapped);
     }
 
-    public function provideScalars()
+    public static function provideScalars()
     {
         return [
             'string empty' => [''],

Diff for: test/Integration/Mapper/MapperTest.php

@@ -21,7 +21,7 @@ public function testReplace($input, $search, $replace, $output)
         self::assertSame($output, $replace([]));
     }
 
-    public function provideReplacements()
+    public static function provideReplacements()
     {
         return [
             'Single removal' => ['foo', 'o', null, 'f'],

Diff for: test/Unit/Mapper/Strategy/ReplaceTest.php

@@ -26,7 +26,7 @@ public function testMap()
             ]
         ));
 
-        self::assertSame([$map], $this->toList($map));
+        self::assertSame([$map], $this->toList());
     }
 
     public function testList()
@@ -38,18 +38,18 @@ public function testList()
             ]
         ));
 
-        self::assertSame($map, $this->toList($map));
+        self::assertSame($map, $this->toList());
     }
 
     public function testScalar()
     {
         $this->toList->setMapper(MockFactory::mockMapper('foo'));
 
-        $this->toList(['foo']);
+        self::assertSame(['foo'], $this->toList());
     }
 
     private function toList()
     {
-        return call_user_func_array($this->toList, func_get_args());
+        return ($this->toList)([]);
     }
 }

Diff for: test/Unit/Mapper/Strategy/ToListTest.php

@@ -1,12 +1,13 @@
 <phpunit
-    beStrictAboutOutputDuringTests="true"
+  beStrictAboutOutputDuringTests="true"
 >
-    <testsuite>
-        <directory>.</directory>
-    </testsuite>
-    <filter>
-        <whitelist processUncoveredFilesFromWhitelist="true">
-            <directory>../src</directory>
-        </whitelist>
-    </filter>
+  <testsuite name="default">
+    <directory>.</directory>
+  </testsuite>
+
+  <source>
+    <include>
+      <directory>../src</directory>
+    </include>
+  </source>
 </phpunit>