Merge pull request #2 from tyron098/master

fix tests, php 7.3 instead of 7.0 support

Author: sartor

.travis.yml

@@ -1,8 +1,8 @@
 language: php
 php:
-    - 7
     - 7.1
     - 7.2
+    - 7.3
 
 env:
   - CLICKHOUSE_VERSION=latest
@@ -18,4 +18,4 @@ before_script:
     - docker run -d -p 127.0.0.1:8123:8123 --name test-clickhouse-server --ulimit nofile=262144:262144 yandex/clickhouse-server:$CLICKHOUSE_VERSION
     - docker logs test-clickhouse-server
 
-script: ./vendor/bin/phpunit
+script: ./vendor/bin/phpunit

composer.json

@@ -5,9 +5,9 @@
     "type": "yii2-extension",
     "license": "MIT",
     "require": {
-        "php": ">=7.0",
+        "php": "^7.1",
         "yiisoft/yii2": "~2.0.13",
-        "smi2/phpClickHouse": "^1.2"
+        "smi2/phpclickhouse": "^1.3.3"
     },
     "require-dev": {
         "phpunit/phpunit": "~5.7"

tests/CommandTest.php

@@ -114,7 +114,9 @@ public function testArrays()
             'Array_UInt8' => [5],
             'Array_Float64' => [5.5],
             'Array_String' => ['asdasd'],
-            'Array_DateTime' => [date('Y-m-d H:i:s')]
+            'Array_DateTime' => [date('Y-m-d H:i:s')],
+            'Array_Nullable_Decimal' => [null, null],
+            'Array_FixedString_empty' => [],
         ];
 
         $this->assertEquals(1, $db->createCommand()->insert('{{arrays}}', $dataForInsert)->execute());

tests/data/clickhouse.sql

@@ -46,5 +46,7 @@ CREATE TABLE `types` (
   `Array_UInt8` Array(UInt8),
   `Array_Float64` Array(Float64),
   `Array_String` Array(String),
-  `Array_DateTime` Array(DateTime)
+  `Array_DateTime` Array(DateTime),
+  `Array_Nullable_Decimal` Array(Nullable(Decimal32(4))),
+  `Array_FixedString_empty` Array(FixedString(2))
  ) ENGINE=Memory;