Update dependencies

This will include an update of the min PHP version to PHP 8.1 to allow an update of the existing dependency on doctrine/lexer
This will also lead to an update of PHPUnit and removed an old dependency on codeclimate/php-test-reporter which is not maintained anymore
Travis CI is also not available anymore and was moved to Github Actions

Author: Marcel Thole

.github/workflows/php-ci.yml

@@ -0,0 +1,33 @@
+name: PHP CI
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+
+jobs:
+  build:
+    name: "PHP Tests"
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        php-version: [ 8.1, 8.2, 8.3 ]
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v2
+
+      - name: Set up PHP ${{ matrix.php-version }}
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php-version }}
+          coverage: none
+          tools: composer:v2
+
+      - name: Install dependencies
+        run: composer --prefer-source install
+
+      - name: Run tests
+        run: ./vendor/bin/phpunit -v

.gitignore

@@ -1,2 +1,3 @@
 vendor/
 composer.lock
+.phpunit.result.cache

.travis.yml

@@ -11,13 +11,12 @@
     ],
     "license": "MIT",
     "require": {
-        "php":            ">=7.1",
+        "php":            ">=8.1",
         "ext-SPL":        "*",
-        "doctrine/lexer": ">=1.0"
+        "doctrine/lexer": "^3.0"
     },
     "require-dev": {
-        "phpunit/phpunit":               "~5.0",
-        "codeclimate/php-test-reporter": "dev-master"
+        "phpunit/phpunit": "^9.6"
     },
     "autoload": {
         "psr-0": {

composer.json

@@ -62,7 +62,7 @@ public function __construct($input = null)
      *
      * @return int
      */
-    protected function getType(&$value)
+    protected function getType(string &$value)
     {
         if (is_numeric($value)) {
             $value += 0;
@@ -71,6 +71,8 @@ protected function getType(&$value)
                 return self::T_INTEGER;
             }
 
+            $value = rtrim(sprintf('%f', $value), '0');
+
             return self::T_FLOAT;
         }
 

lib/CrEOF/Geo/String/Lexer.php

@@ -222,7 +222,7 @@ private function degrees()
         $glimpse = $this->lexer->glimpse();
 
         // If a colon hasn't been matched, and next token is a number followed by degree symbol, when tuple separator is space instead of comma, this value is complete
-        if (Lexer::T_COLON !== $this->nextSymbol && $this->lexer->isNextTokenAny(array(Lexer::T_INTEGER, Lexer::T_FLOAT)) && Lexer::T_DEGREE === $glimpse['type']) {
+        if ($glimpse !== null && Lexer::T_COLON !== $this->nextSymbol && $this->lexer->isNextTokenAny(array(Lexer::T_INTEGER, Lexer::T_FLOAT)) && Lexer::T_DEGREE === $glimpse->type) {
             return $degrees;
         }
 
@@ -407,7 +407,7 @@ private function cardinal($value)
     {
         // If cardinal direction was not on previous coordinate it can be anything
         if (null === $this->nextCardinal) {
-            $this->nextCardinal = Lexer::T_CARDINAL_LON === $this->lexer->lookahead['type'] ? Lexer::T_CARDINAL_LON : Lexer::T_CARDINAL_LAT;
+            $this->nextCardinal = Lexer::T_CARDINAL_LON === $this->lexer->lookahead->type ? Lexer::T_CARDINAL_LON : Lexer::T_CARDINAL_LAT;
         }
 
         // Match cardinal direction
@@ -468,7 +468,7 @@ private function match($token)
         $this->lexer->moveNext();
 
         // Return the token value
-        return $this->lexer->token['value'];
+        return $this->lexer->token->value;
     }
 
     /**
@@ -482,11 +482,11 @@ private function syntaxError($expected)
     {
         $expected = sprintf('Expected %s, got', $expected);
         $token    = $this->lexer->lookahead;
-        $found    = null === $this->lexer->lookahead ? 'end of string.' : sprintf('"%s"', $token['value']);
+        $found    = null === $this->lexer->lookahead ? 'end of string.' : sprintf('"%s"', $token->value);
 
         $message = sprintf(
             '[Syntax Error] line 0, col %d: Error: %s %s in value "%s"',
-            isset($token['position']) ? $token['position'] : '-1',
+            $token->position ?? -1,
             $expected,
             $found,
             $this->input

lib/CrEOF/Geo/String/Parser.php

@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <phpunit
         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-        xsi:noNamespaceSchemaLocation="http://schema.phpunit.de/4.8/phpunit.xsd"
+        xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
         backupGlobals="false"
         colors="true"
         bootstrap="./vendor/autoload.php"
@@ -12,14 +12,4 @@
             <directory>./tests/CrEOF/Geo/String/Tests</directory>
         </testsuite>
     </testsuites>
-
-    <filter>
-        <whitelist>
-            <directory suffix=".php">lib</directory>
-            <exclude>
-                <directory>tests</directory>
-                <directory>vendor</directory>
-            </exclude>
-        </whitelist>
-    </filter>
 </phpunit>