Merge pull request #21 from FriendsOfCake/ci

Add CI

.editorconfig

@@ -14,3 +14,7 @@ trim_trailing_whitespace = true
 [*.yml]
 indent_style = space
 indent_size = 2
+
+[*.neon]
+indent_style = tab
+indent_size = 4

.github/workflows/ci.yml

@@ -0,0 +1,29 @@
+name: CI
+
+on: [push, pull_request]
+
+jobs:
+  cs-stan:
+    name: Coding Standard & Static Analysis
+    runs-on: ubuntu-22.04
+
+    steps:
+    - uses: actions/checkout@v3
+
+    - name: Setup PHP
+      uses: shivammathur/setup-php@v2
+      with:
+        php-version: '7.4'
+        extensions: mbstring, intl
+        coverage: none
+        tools: phpstan:1.10, cs2pr
+
+    - name: Composer Install
+      run: composer install
+
+    - name: Run phpcs
+      run: vendor/bin/phpcs --report=checkstyle --standard=vendor/cakephp/cakephp-codesniffer/CakePHP src/ | cs2pr
+
+    - name: Run phpstan
+      if: always()
+      run: phpstan analyse --error-format=github

composer.json

@@ -43,5 +43,10 @@
         "source": "https://github.com/FriendsOfCake/fixturize",
         "issues": "https://github.com/FriendsOfCake/fixturize/issues",
         "irc": "irc://irc.freenode.org/cakephp"
+    },
+    "config": {
+        "allow-plugins": {
+            "dealerdirect/phpcodesniffer-composer-installer": true
+        }
     }
 }

phpstan.neon

@@ -0,0 +1,4 @@
+parameters:
+	level: 8
+	paths:
+		- src

src/TestSuite/Fixture/ChecksumTestFixture.php

@@ -13,14 +13,13 @@
  *
  * If no data has changed, the usual truncate/insert flow is bypassed, increasing the speed
  * of the test suite with heavy fixture usage up significantly.
- *
  */
 class ChecksumTestFixture extends TestFixture
 {
     /**
      * List of table hashes
      *
-     * @var array
+     * @var array<string, string>
      */
     protected static $_tableHashes = [];
 
@@ -113,10 +112,11 @@ protected function _hash(ConnectionInterface $db): string
         $driver = $db->getDriver();
 
         if ($driver instanceof Mysql) {
-            $sth = $db->execute("CHECKSUM TABLE " . $this->table);
+            $sth = $db->execute('CHECKSUM TABLE ' . $this->table);
 
             return $sth->fetchColumn(1);
         }
+
         // Have no better idea right now to make it always regenerate the tables
         return microtime();
     }