Min PHP version to >=8.1; add MySQL tests with charset and collation

Author: roxblnfk

.github/workflows/main.yml

@@ -8,14 +8,14 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        php-version: ['8.1', '8.2', '8.3', '8.4']
+        php-version: ['8.2', '8.3', '8.4', '8.5']
     steps:
       - name: Install ODBC driver.
         run: |
           sudo curl https://packages.microsoft.com/config/ubuntu/$(lsb_release -rs)/prod.list | sudo tee /etc/apt/sources.list.d/mssql-release.list
           sudo ACCEPT_EULA=Y apt-get install -y msodbcsql18
       - name: Checkout
-        uses: actions/checkout@v2
+        uses: actions/checkout@v5
       - name: Setup DB services
         run: |
           cd tests

.scrutinizer.yml

@@ -14,7 +14,7 @@ build:
   nodes:
     analysis:
       environment:
-        php: 8.0
+        php: 8.2
 
       tests:
         override:

composer.json

@@ -38,8 +38,8 @@
     ],
     "require": {
         "php": ">=8.1",
-        "cycle/database": "^2.15",
-        "cycle/orm": "^2.9.2",
+        "cycle/database": "^2.16",
+        "cycle/orm": "^2.15",
         "cycle/schema-builder": "^2.11.1",
         "spiral/attributes": "^2.8|^3.0",
         "spiral/tokenizer": "^2.8|^3.0",

tests/Annotated/Fixtures/Fixtures1/Label.php

@@ -36,4 +36,7 @@ class Label
 
     #[Column(type: 'tinyInteger')]
     private int $simple = 1;
+
+    #[Column(type: 'string', charset: 'ascii', collation: 'ascii_bin')]
+    private string $charsetColumn = '';
 }

tests/Annotated/Functional/Driver/Common/InheritanceTestCase.php

@@ -109,7 +109,7 @@ public function testTableInheritance(ReaderInterface $reader): void
         $this->assertSame('foo_id', $schema['executive'][SchemaInterface::PARENT_KEY]);
         $this->assertSame('executives', $schema['executive'][SchemaInterface::TABLE]);
         $this->assertSame(
-            ['bonus' => 'bonus', 'foo_id' => 'id', 'hidden' => 'hidden'],
+            ['bonus' => 'bonus', 'hidden' => 'hidden', 'foo_id' => 'id'],
             $schema['executive'][SchemaInterface::COLUMNS],
         );
 

tests/Annotated/Functional/Driver/MySQL/TableTest.php

@@ -10,6 +10,7 @@
 use Cycle\Annotated\MergeColumns;
 use Cycle\Annotated\Tests\Functional\Driver\Common\TableTestCase;
 use Cycle\Schema\Generator\RenderTables;
+use Cycle\Schema\Generator\SyncTables;
 use Cycle\Schema\Registry;
 use PHPUnit\Framework\Attributes\Group;
 use Spiral\Attributes\AttributeReader;
@@ -57,4 +58,20 @@ public function testZerofill(): void
         $this->assertTrue($this->dbal->database()->table('labels')->getSchema()->column('zerofill')->isUnsigned());
         $this->assertFalse($this->dbal->database()->table('labels')->getSchema()->column('simple')->isUnsigned());
     }
+
+    public function testColumnCustomAttributes(): void
+    {
+        $reader = new AttributeReader();
+        $r = new Registry($this->dbal);
+        (new Entities(new TokenizerEntityLocator($this->locator, $reader), $reader))->run($r);
+        (new MergeColumns($reader))->run($r);
+        (new RenderTables())->run($r);
+        (new SyncTables())->run($r);
+
+        $dbSchema = $this->dbal->database()->table('labels')->getSchema();
+        $attributes = $dbSchema->column('charset_column')->getAttributes();
+
+        $this->assertSame('ascii', $attributes['charset']);
+        $this->assertSame('ascii_bin', $attributes['collation']);
+    }
 }

tests/Annotated/Unit/Attribute/ColumnTest.php

@@ -50,6 +50,9 @@ class ColumnTest extends TestCase
     )]
     private string $column8 = 'a';
 
+    #[Column(type: 'string', length: 255, charset: 'ascii', collation: 'ascii_bin')]
+    private string $column9;
+
     public function testOneAttribute(): void
     {
         $column = $this->getColumn('column1');
@@ -119,6 +122,18 @@ public function testEnumTypeArrayBackedEnum(): void
         $this->assertArrayNotHasKey('values', $column->getAttributes());
     }
 
+    public function testCharsetAndCollationAttributes(): void
+    {
+        $column = $this->getColumn('column9');
+
+        $this->assertSame('string', $column->getType());
+        $this->assertSame([
+            'length' => 255,
+            'charset' => 'ascii',
+            'collation' => 'ascii_bin',
+        ], $column->getAttributes());
+    }
+
     private function getColumn(string $field): Column
     {
         $ref = new \ReflectionClass(static::class);