Add failure tests

Author: miguilimzero

src/Schema/Grammar.php

@@ -200,13 +200,13 @@ protected function columnizeWithDirection(array $columns, string $direction)
             }, $columnNames, $columnDirections));
         }
 
+        if (array_filter($columns, 'is_array') !== []) {
+            throw new InvalidArgumentException('You must set the direction for each sort key column or use the second parameter to set the direction for all sort key columns');
+        }
+
         $wrapped = array_map([$this, 'wrap'], $columns);
 
         return implode(', ', array_map(function ($column) use ($direction) {
-            if (is_array($column)) {
-                throw new InvalidArgumentException('You must set the direction for each sort key column or use the second parameter to set the direction for all sort key columns');
-            }
-
             return $column.' '.$direction;
         }, $wrapped));
     }

tests/Hybrid/CreateTable/SortKeysTest.php

@@ -5,6 +5,7 @@
 
 namespace SingleStore\Laravel\Tests\Hybrid\CreateTable;
 
+use InvalidArgumentException;
 use SingleStore\Laravel\Schema\Blueprint;
 use SingleStore\Laravel\Tests\BaseTest;
 use SingleStore\Laravel\Tests\Hybrid\HybridTestHelpers;
@@ -112,6 +113,34 @@ public function it_adds_a_dual_sort_key_with_different_directions()
         );
     }
 
+    /** @test */
+    public function it_cannot_add_a_dual_sort_key_with_only_one_direction()
+    {
+        $blueprint = $this->createTable(function (Blueprint $table) {
+            $table->string('f_name');
+            $table->string('l_name');
+            $table->sortKey(['f_name', ['l_name', 'desc']]);
+        });
+
+        $this->expectException(InvalidArgumentException::class);
+
+        $blueprint->toSql($this->getConnection(), $this->getGrammar());
+    }
+
+    /** @test */
+    public function it_cannot_add_a_dual_sort_key_with_only_one_direction_desc()
+    {
+        $blueprint = $this->createTable(function (Blueprint $table) {
+            $table->string('f_name');
+            $table->string('l_name');
+            $table->sortKey(['f_name', ['l_name', 'asc']], 'desc');
+        });
+
+        $this->expectException(InvalidArgumentException::class);
+
+        $blueprint->toSql($this->getConnection(), $this->getGrammar());
+    }
+
     /** @test */
     public function shard_and_sort_keys()
     {