[slim-skeleton-11.x] Test Improvements (#49111)

* Test Improvements

Signed-off-by: Mior Muhammad Zaki <crynobone@gmail.com>

* Fix MariaDB Tests with changed query

* Fix Testbench version

* Allow to override collation using DB_COLLATION

Signed-off-by: Mior Muhammad Zaki <crynobone@gmail.com>

* wip

Signed-off-by: Mior Muhammad Zaki <crynobone@gmail.com>

---------

Signed-off-by: Mior Muhammad Zaki <crynobone@gmail.com>
Co-authored-by: Julius Kiekbusch <contact@julius-kiekbusch.de>

Author: crynobone

.github/workflows/databases.yml

@@ -52,6 +52,7 @@ jobs:
         env:
           DB_CONNECTION: mysql
           DB_USERNAME: root
+          MYSQL_COLLATION: utf8mb4_unicode_ci
 
   mysql_8:
     runs-on: ubuntu-22.04

composer.json

@@ -105,7 +105,7 @@
         "league/flysystem-sftp-v3": "^3.0",
         "mockery/mockery": "^1.5.1",
         "nyholm/psr7": "^1.2",
-        "orchestra/testbench-core": "^9.0",
+        "orchestra/testbench-core": "dev-next/slim-skeleton",
         "pda/pheanstalk": "^4.0",
         "phpstan/phpstan": "^1.4.7",
         "phpunit/phpunit": "^10.1",

tests/Integration/Database/SchemaBuilderTest.php

@@ -132,9 +132,10 @@ public function testChangeTextColumnToTextColumn()
 
             $uppercase = strtoupper($type);
 
-            $expected = ["ALTER TABLE test CHANGE test_column test_column $uppercase NOT NULL"];
-
-            $this->assertEquals($expected, $queries);
+            $this->assertContains($queries, [
+                ["ALTER TABLE test CHANGE test_column test_column $uppercase NOT NULL"], // MySQL
+                ["ALTER TABLE test CHANGE test_column test_column $uppercase NOT NULL COLLATE `utf8mb4_uca1400_ai_ci`"], // MariaDB
+            ]);
         }
     }
 }

tests/Integration/Generators/CacheTableCommandTest.php

@@ -2,11 +2,13 @@
 
 namespace Illuminate\Tests\Integration\Generators;
 
+use Illuminate\Cache\Console\CacheTableCommand;
+
 class CacheTableCommandTest extends TestCase
 {
     public function testCreateMakesMigration()
     {
-        $this->artisan('cache:table')->assertExitCode(0);
+        $this->artisan(CacheTableCommand::class)->assertExitCode(0);
 
         $this->assertMigrationFileContains([
             'use Illuminate\Database\Migrations\Migration;',

tests/Integration/Generators/NotificationTableCommandTest.php

@@ -2,11 +2,13 @@
 
 namespace Illuminate\Tests\Integration\Generators;
 
+use Illuminate\Notifications\Console\NotificationTableCommand;
+
 class NotificationTableCommandTest extends TestCase
 {
     public function testCreateMakesMigration()
     {
-        $this->artisan('notifications:table')->assertExitCode(0);
+        $this->artisan(NotificationTableCommand::class)->assertExitCode(0);
 
         $this->assertMigrationFileContains([
             'use Illuminate\Database\Migrations\Migration;',

tests/Integration/Generators/QueueBatchesTableCommandTest.php

@@ -2,11 +2,13 @@
 
 namespace Illuminate\Tests\Integration\Generators;
 
+use Illuminate\Queue\Console\BatchesTableCommand;
+
 class QueueBatchesTableCommandTest extends TestCase
 {
     public function testCreateMakesMigration()
     {
-        $this->artisan('queue:batches-table')->assertExitCode(0);
+        $this->artisan(BatchesTableCommand::class)->assertExitCode(0);
 
         $this->assertMigrationFileContains([
             'use Illuminate\Database\Migrations\Migration;',

tests/Integration/Generators/QueueFailedTableCommandTest.php

@@ -2,11 +2,13 @@
 
 namespace Illuminate\Tests\Integration\Generators;
 
+use Illuminate\Queue\Console\FailedTableCommand;
+
 class QueueFailedTableCommandTest extends TestCase
 {
     public function testCreateMakesMigration()
     {
-        $this->artisan('queue:failed-table')->assertExitCode(0);
+        $this->artisan(FailedTableCommand::class)->assertExitCode(0);
 
         $this->assertMigrationFileContains([
             'use Illuminate\Database\Migrations\Migration;',

tests/Integration/Generators/QueueTableCommandTest.php

@@ -2,11 +2,13 @@
 
 namespace Illuminate\Tests\Integration\Generators;
 
+use Illuminate\Queue\Console\TableCommand;
+
 class QueueTableCommandTest extends TestCase
 {
     public function testCreateMakesMigration()
     {
-        $this->artisan('queue:table')->assertExitCode(0);
+        $this->artisan(TableCommand::class)->assertExitCode(0);
 
         $this->assertMigrationFileContains([
             'use Illuminate\Database\Migrations\Migration;',

tests/Integration/Generators/SessionTableCommandTest.php

@@ -2,11 +2,13 @@
 
 namespace Illuminate\Tests\Integration\Generators;
 
+use Illuminate\Session\Console\SessionTableCommand;
+
 class SessionTableCommandTest extends TestCase
 {
     public function testCreateMakesMigration()
     {
-        $this->artisan('session:table')->assertExitCode(0);
+        $this->artisan(SessionTableCommand::class)->assertExitCode(0);
 
         $this->assertMigrationFileContains([
             'use Illuminate\Database\Migrations\Migration;',