fix: make non-sqlsrv tests all green again

Author: tpetry

composer.json

@@ -18,23 +18,14 @@
     ],
     "require": {
         "php": "^8.1",
-        "illuminate/contracts": "^10.13.1|^11.0|^12.0",
-        "illuminate/database": "^10.13.1|^11.0|^12.0",
-        "illuminate/support": "^10.0|^11.0|^12.0"
+        "laravel/framework": "^10.13.1|^11.0|^12.0"
     },
     "require-dev": {
-        "larastan/larastan": "^2.7.0|^3.0",
         "laravel/pint": "^1.0",
-        "nunomaduro/collision": "^7.0|^8.0",
         "orchestra/testbench": "^8.0|^9.0|^10.0",
         "pestphp/pest": "^2.28.1|^3.0.0",
         "pestphp/pest-plugin-laravel": "^2.2.0|^3.0.0",
-        "phpstan/extension-installer": "^1.1",
-        "phpstan/phpstan": "^1.11|^2.0",
-        "phpstan/phpstan-deprecation-rules": "^1.0|^2.0",
-        "phpstan/phpstan-phpunit": "^1.0|^2.0",
-        "phpunit/phpunit": "^10.5.3|^11.0.0",
-        "spatie/laravel-ray": "^1.26"
+        "phpstan/phpstan": "^1.11|^2.0"
     },
     "autoload": {
         "psr-4": {
@@ -54,6 +45,9 @@
         "format": "vendor/bin/pint"
     },
     "config": {
+        "audit": {
+            "block-insecure": false
+        },
         "sort-packages": true,
         "allow-plugins": {
             "pestphp/pest-plugin": true,

src/Function/Conditional/ManyArgumentsExpression.php

@@ -16,7 +16,7 @@ abstract class ManyArgumentsExpression implements Expression
     use StringizeExpression;
 
     /**
-     * @param  non-empty-array<int, string|\Illuminate\Contracts\Database\Query\Expression>  $expressions
+     * @param  non-empty-array<int, string|Expression>  $expressions
      */
     public function __construct(
         private readonly array $expressions,

src/Language/Cast.php

@@ -45,8 +45,8 @@ private function castMysql(float|int|string $expression): string
         // float one, but it will be silently downsized to a float when stored in a table.
         return match ($this->type) {
             'bigint', 'int' => "cast({$expression} as signed)",
-            'float', 'double' => "(({$expression})*1.0)",
-            default => throw new RuntimeException("Unknown cast type '{$this->type}'."), // @phpstan-ignore match.unreachable
+            'float', 'double' => "(({$expression})*1.0)", // @phpstan-ignore match.alwaysTrue
+            default => throw new RuntimeException("Unknown cast type '{$this->type}'."),
         };
     }
 
@@ -56,17 +56,17 @@ private function castPgsql(float|int|string $expression): string
             'bigint' => "cast({$expression} as bigint)",
             'float' => "cast({$expression} as real)",
             'double' => "cast({$expression} as double precision)",
-            'int' => "cast({$expression} as int)",
-            default => throw new RuntimeException("Unknown cast type '{$this->type}'."), // @phpstan-ignore match.unreachable
+            'int' => "cast({$expression} as int)", // @phpstan-ignore match.alwaysTrue
+            default => throw new RuntimeException("Unknown cast type '{$this->type}'."),
         };
     }
 
     private function castSqlite(float|int|string $expression): string
     {
         return match ($this->type) {
             'bigint', 'int' => "cast({$expression} as integer)",
-            'float', 'double' => "cast({$expression} as real)",
-            default => throw new RuntimeException("Unknown cast type '{$this->type}'."), // @phpstan-ignore match.unreachable
+            'float', 'double' => "cast({$expression} as real)", // @phpstan-ignore match.alwaysTrue
+            default => throw new RuntimeException("Unknown cast type '{$this->type}'."),
         };
     }
 
@@ -76,8 +76,8 @@ private function castSqlsrv(float|int|string $expression): string
             'bigint' => "cast({$expression} as bigint)",
             'float' => "cast({$expression} as float(24))",
             'double' => "cast({$expression} as float(53))",
-            'int' => "(({$expression})*1)",
-            default => throw new RuntimeException("Unknown cast type '{$this->type}'."), // @phpstan-ignore match.unreachable
+            'int' => "(({$expression})*1)", // @phpstan-ignore match.alwaysTrue
+            default => throw new RuntimeException("Unknown cast type '{$this->type}'."),
         };
     }
 }

src/Operator/Arithmetic/ArithmeticExpression.php

@@ -34,7 +34,7 @@ public function getValue(Grammar $grammar): string
     }
 
     /**
-     * @return array<int, float|int|string>
+     * @return array<array-key, float|int|string>
      */
     protected function expressions(Grammar $grammar): array
     {

tests/Pest.php

@@ -2,6 +2,7 @@
 
 declare(strict_types=1);
 
+use Illuminate\Database\Connection;
 use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Facades\Schema;
 use Pest\Expectation;
@@ -13,7 +14,7 @@
 )->in(__DIR__);
 
 expect()->extend('toBeExecutable', function (?Closure $migration = null, array $options = []): Expectation {
-    /** @var \Illuminate\Database\Connection $connection */
+    /** @var Connection $connection */
     $connection = DB::connection();
 
     $table = null;
@@ -67,10 +68,10 @@
 
 function skipOnMariaBefore(string $version)
 {
-    /** @var \Illuminate\Database\Connection $connection */
+    /** @var Connection $connection */
     $connection = DB::connection();
 
-    if ($connection->getDriverName() !== 'mysql') {
+    if (! in_array($connection->getDriverName(), ['mariadb', 'mysql'])) {
         return;
     }
 
@@ -82,7 +83,7 @@ function skipOnMariaBefore(string $version)
 
 function skipOnMysqlBefore(string $version): void
 {
-    /** @var \Illuminate\Database\Connection $connection */
+    /** @var Connection $connection */
     $connection = DB::connection();
 
     if ($connection->getDriverName() !== 'mysql') {
@@ -97,7 +98,7 @@ function skipOnMysqlBefore(string $version): void
 
 function skipOnPgsqlBefore(string $version): void
 {
-    /** @var \Illuminate\Database\Connection $connection */
+    /** @var Connection $connection */
     $connection = DB::connection();
 
     if ($connection->getDriverName() !== 'pgsql') {
@@ -112,7 +113,7 @@ function skipOnPgsqlBefore(string $version): void
 
 function skipOnSqliteBefore(string $version): void
 {
-    /** @var \Illuminate\Database\Connection $connection */
+    /** @var Connection $connection */
     $connection = DB::connection();
 
     if ($connection->getDriverName() !== 'sqlite') {
@@ -127,7 +128,7 @@ function skipOnSqliteBefore(string $version): void
 
 function skipOnSqlsrvBefore(string $version): void
 {
-    /** @var \Illuminate\Database\Connection $connection */
+    /** @var Connection $connection */
     $connection = DB::connection();
 
     if ($connection->getDriverName() !== 'sqlsrv') {