Test housecleaning

Author: devdot

.gitattributes

@@ -0,0 +1,7 @@
+/.github export-ignore
+/tests export-ignore
+
+.gitattributes export-ignore
+.gitignore export-ignore
+.gitlab-ci.yml export-ignore
+captainhook.json export-ignore

.gitlab-ci.yml

@@ -12,11 +12,11 @@ test_phpunit:
   stage: test
   script:
     - composer install
-    - vendor/bin/phpunit tests
+    - composer test
 
 test_quality:
   stage: test
   script:
     - composer install
     - vendor/bin/phpcs --standard=PSR12 -n src
-    - vendor/bin/phpstan analyse src --level=9
+    - composer check-static

captainhook.json

@@ -3,16 +3,18 @@
         "enabled": true,
         "actions": [
             {
-                "action": "vendor/bin/phpcbf --standard=PSR12 src",
-                "options": []
-            },
+                "action": "composer format"
+            }
+        ]
+    },
+    "pre-push": {
+        "enabled": true,
+        "actions": [
             {
-                "action": "vendor/bin/phpunit tests",
-                "options": []
+                "action": "composer check-static"
             },
             {
-                "action": "vendor/bin/phpstan analyze src --level=9",
-                "options": []
+                "action": "composer test"
             }
         ]
     }

composer.json

@@ -27,6 +27,14 @@
         "monolog/monolog": "2.*",
         "phpstan/phpstan": "^1.10",
         "captainhook/captainhook": "^5.16",
-        "squizlabs/php_codesniffer": "^3.7"
+        "phpcsstandards/php_codesniffer": "^3.7"
+    },
+    "scripts": {
+        "check-static": "phpstan analyse src --level=9",
+        "format": "phpcbf --standard=PSR12 src tests",
+        "test": "phpunit tests",
+        "post-autoload-dump": [
+            "vendor/bin/captainhook install -f -s"
+        ]
     }
 }

composer.lock

@@ -3,8 +3,10 @@
 use PHPUnit\Framework\TestCase;
 use Devdot\Monolog\LogRecord;
 
-final class LogRecordTest extends TestCase {
-    public function testConstruct() {
+final class LogRecordTest extends TestCase
+{
+    public function testConstruct()
+    {
         $record = new LogRecord(
             new \DateTimeImmutable('2023-01-02 08:00:01'),
             'test-channel',
@@ -24,7 +26,8 @@ public function testConstruct() {
         $this->assertCount(0, $record->extra);
     }
 
-    public function testArrayAccess() {
+    public function testArrayAccess()
+    {
         $record = new LogRecord(
             new \DateTimeImmutable('2023-01-02 08:00:01'),
             'test-channel',
@@ -36,23 +39,21 @@ public function testArrayAccess() {
         $this->assertIsObject($record);
         $this->assertIsNotArray($record);
 
-        // make sure we can read        
+        // make sure we can read
         $this->assertIsObject($record['datetime']);
         $this->assertIsString($record['channel']);
-        
+
         // make sure we can't write or unset
         try {
             $record['level'] = 'test';
             throw new \Exception();
-        }
-        catch(\Exception $e) {
+        } catch (\Exception $e) {
             $this->assertInstanceOf(\LogicException::class, $e);
         }
         try {
             unset($record['message']);
             throw new \Exception();
-        }
-        catch(\Exception $e) {
+        } catch (\Exception $e) {
             $this->assertInstanceOf(\LogicException::class, $e);
         }
     }