Fixed tests

Author: AdalbertMemSQL

src/Query/Grammar.php

@@ -26,7 +26,7 @@ public function compileOptions(array $options): string
             if (! empty($optionString)) {
                 $optionString .= ',';
             }
-            $optionString .= $key.'='.$value;
+            $optionString .= $key . '=' . $value;
         }
 
         return "OPTION ({$optionString})";
@@ -122,18 +122,6 @@ public function prepareBindingsForUpdate(array $bindings, array $values)
         return parent::prepareBindingsForUpdate($bindings, $values);
     }
 
-    protected function whereNull(Builder $query, $where)
-    {
-        $columnValue = (string) $this->getValue($where['column']);
-        if ($this->isJsonSelector($columnValue)) {
-            [$field, $path] = $this->wrapJsonFieldAndPath($where['column']);
-
-            return '(JSON_EXTRACT_JSON('.$field.$path.') IS NULL OR JSON_GET_TYPE(JSON_EXTRACT_JSON('.$field.$path.')) = \'NULL\')';
-        }
-
-        return parent::whereNull($query, $where);
-    }
-
     /**
      * Transforms expressions to their scalar types.
      *
@@ -149,16 +137,28 @@ public function getValue($expression)
         return $expression;
     }
 
+    protected function whereNull(Builder $query, $where)
+    {
+        $columnValue = (string) $this->getValue($where['column']);
+        if ($this->isJsonSelector($columnValue)) {
+            [$field, $path] = $this->wrapJsonFieldAndPath($where['column']);
+
+            return '(JSON_EXTRACT_JSON(' . $field . $path . ') IS NULL OR JSON_GET_TYPE(JSON_EXTRACT_JSON(' . $field . $path . ')) = \'NULL\')';
+        }
+
+        return $this->wrap($where['column']) . ' is null';
+    }
+
     protected function whereNotNull(Builder $query, $where)
     {
         $columnValue = (string) $this->getValue($where['column']);
         if ($this->isJsonSelector($columnValue)) {
             [$field, $path] = $this->wrapJsonFieldAndPath($where['column']);
 
-            return '(JSON_EXTRACT_JSON('.$field.$path.') IS NOT NULL AND JSON_GET_TYPE(JSON_EXTRACT_JSON('.$field.$path.')) != \'NULL\')';
+            return '(JSON_EXTRACT_JSON(' . $field . $path . ') IS NOT NULL AND JSON_GET_TYPE(JSON_EXTRACT_JSON(' . $field . $path . ')) != \'NULL\')';
         }
 
-        return parent::whereNotNull($query, $where);
+        return $this->wrap($where['column']) . ' is not null';;
     }
 
     protected function wrapJsonSelector($value)
@@ -207,7 +207,7 @@ protected function wrapJsonFieldAndPath($column)
             return "'$part'";
         }, $parts);
 
-        $path = count($parts) ? ', '.implode(', ', $parts) : '';
+        $path = count($parts) ? ', ' . implode(', ', $parts) : '';
 
         return [$field, $path];
     }
@@ -220,7 +220,7 @@ protected function wrapJsonFieldAndPath($column)
      */
     protected function wrapUnion($sql)
     {
-        return 'SELECT * FROM ('.$sql.')';
+        return 'SELECT * FROM (' . $sql . ')';
     }
 
     /**
@@ -248,7 +248,7 @@ protected function compileUnion(array $union)
     {
         $conjunction = $union['all'] ? ' union all ' : ' union ';
 
-        return $conjunction.'('.$union['query']->toSql().')';
+        return $conjunction . '(' . $union['query']->toSql() . ')';
     }
 
     /**
@@ -267,18 +267,18 @@ public function compileSelect(Builder $query)
         }
 
         if (! empty($query->unionOrders) || isset($query->unionLimit) || isset($query->unionOffset)) {
-            $sql = 'SELECT * FROM ('.$sql.') ';
+            $sql = 'SELECT * FROM (' . $sql . ') ';
 
             if (! empty($query->unionOrders)) {
-                $sql .= ' '.$this->compileOrders($query, $query->unionOrders);
+                $sql .= ' ' . $this->compileOrders($query, $query->unionOrders);
             }
 
             if (isset($query->unionLimit)) {
-                $sql .= ' '.$this->compileLimit($query, $query->unionLimit);
+                $sql .= ' ' . $this->compileLimit($query, $query->unionLimit);
             }
 
             if (isset($query->unionOffset)) {
-                $sql .= ' '.$this->compileUnionOffset($query, $query->unionOffset);
+                $sql .= ' ' . $this->compileUnionOffset($query, $query->unionOffset);
             }
         }
 
@@ -312,10 +312,10 @@ private function compileOffsetWithLimit($offset, $limit): string
         // Add a huge LIMIT clause
         if (! isset($limit)) {
             // 9223372036854775807 - max 64-bit integer
-            return ' LIMIT 9223372036854775807 OFFSET '.(int) $offset;
+            return ' LIMIT 9223372036854775807 OFFSET ' . (int) $offset;
         }
 
-        return ' OFFSET '.(int) $offset;
+        return ' OFFSET ' . (int) $offset;
     }
 
     /**