Adds phpcs and resolves phpcs errors

.github/workflows/pull-request.yml

@@ -35,7 +35,7 @@ jobs:
 
       - name: Test Suite
         run: |
-          composer test
+          composer check
 
   #
   # CakePHP version compatability

tests/AuditLogsTable.php

@@ -0,0 +1,33 @@
+<?php
+/**
+ * @copyright Copyright (c) 2024, Canadian Mortgages Inc.
+ */
+declare(strict_types=1);
+
+namespace AuditStash\Test;
+
+use Cake\ORM\Table;
+
+class AuditLogsTable extends Table
+{
+    public function initialize(array $config): void
+    {
+        parent::initialize($config);
+
+        $this->setTable('audit_logs');
+        $this->setPrimaryKey('id');
+
+        $this->setSchema([
+            'id' => 'integer',
+            'transaction' => 'string',
+            'type' => 'string',
+            'primary_key' => 'integer',
+            'source' => 'string',
+            'parent_source' => 'string',
+            'original' => 'string',
+            'changed' => 'string',
+            'meta' => 'string',
+            'created' => 'datetime',
+        ]);
+    }
+}

tests/DebugPersister.php

@@ -0,0 +1,13 @@
+<?php
+declare(strict_types=1);
+
+namespace AuditStash\Test;
+
+use AuditStash\PersisterInterface;
+
+class DebugPersister implements PersisterInterface
+{
+    public function logEvents(array $events): void
+    {
+    }
+}

tests/TestCase/Meta/ApplicationMetadataTest.php

@@ -22,7 +22,7 @@ public function testDataIsAdded()
         $listener = new ApplicationMetadata('my_app', ['extra' => 'thing']);
         $this->getEventManager()->on($listener);
         $logs[] = new AuditDeleteEvent('1234', 1, 'articles');
-        $event = $this->dispatchEvent('AuditStash.beforeLog', ['logs' => $logs]);
+        $this->dispatchEvent('AuditStash.beforeLog', ['logs' => $logs]);
 
         $expected = [
             'app_name' => 'my_app',

tests/TestCase/Meta/RequestMetadataTest.php

@@ -27,7 +27,7 @@ public function testRequestDataIsAdded()
         $request->expects($this->once())->method('clientIp')->will($this->returnValue('12345'));
         $request->expects($this->once())->method('getRequestTarget')->will($this->returnValue('/things?a=b'));
         $logs[] = new AuditDeleteEvent('1234', 1, 'articles');
-        $event = $this->dispatchEvent('AuditStash.beforeLog', ['logs' => $logs]);
+        $this->dispatchEvent('AuditStash.beforeLog', ['logs' => $logs]);
 
         $expected = [
             'ip' => '12345',

tests/TestCase/Model/Behavior/AuditIntegrationTest.php

@@ -7,18 +7,11 @@
 use AuditStash\Event\AuditDeleteEvent;
 use AuditStash\Event\AuditUpdateEvent;
 use AuditStash\Model\Behavior\AuditLogBehavior;
-use AuditStash\PersisterInterface;
+use AuditStash\Test\DebugPersister;
 use Cake\ORM\Locator\LocatorAwareTrait;
 use Cake\ORM\Table;
 use Cake\TestSuite\TestCase;
 
-class DebugPersister implements PersisterInterface
-{
-    public function logEvents(array $events): void
-    {
-    }
-}
-
 class AuditIntegrationTest extends TestCase
 {
     use LocatorAwareTrait;

tests/TestCase/Persister/TablePersisterTest.php

@@ -5,6 +5,7 @@
 
 use AuditStash\Event\AuditCreateEvent;
 use AuditStash\Persister\TablePersister;
+use AuditStash\Test\AuditLogsTable;
 use Cake\Datasource\EntityInterface;
 use Cake\ORM\Entity;
 use Cake\ORM\Table;
@@ -13,30 +14,6 @@
 use InvalidArgumentException;
 use PHPUnit\Framework\MockObject\MockObject;
 
-class AuditLogsTable extends Table
-{
-    public function initialize(array $config): void
-    {
-        parent::initialize($config);
-
-        $this->setTable('audit_logs');
-        $this->setPrimaryKey('id');
-
-        $this->setSchema([
-            'id' => 'integer',
-            'transaction' => 'string',
-            'type' => 'string',
-            'primary_key' => 'integer',
-            'source' => 'string',
-            'parent_source' => 'string',
-            'original' => 'string',
-            'changed' => 'string',
-            'meta' => 'string',
-            'created' => 'datetime',
-        ]);
-    }
-}
-
 class TablePersisterTest extends TestCase
 {
     /**