[TEST] Allow JSON workaround, remove travis hackery

Author: polyfractal

.travis.yml

@@ -25,6 +25,10 @@ matrix:
   include:
     - php: 7.1
       env: ES_VERSION="5.0" TEST_BUILD_REF="origin/5.0"
+    - php: 7.1
+      env: ES_VERSION="5.1" TEST_BUILD_REF="origin/5.1"
+    - php: 7.1
+      env: ES_VERSION="5.2" TEST_BUILD_REF="origin/5.2"
   allow_failures:
     - php: hhvm
 
@@ -36,7 +40,6 @@ env:
     - JAVA_HOME="/usr/lib/jvm/java-8-oracle/jre"
 
 before_install:
-  - if [ $TRAVIS_PHP_VERSION = '5.6' ]; then pecl uninstall json && pecl install json-1.3.7; fi
   - sudo update-java-alternatives -s java-8-oracle
   - ./travis/download_and_run_es.sh
 

tests/Elasticsearch/Tests/YamlRunnerTest.php

@@ -103,7 +103,11 @@ public static function setUpBeforeClass()
     public function setUp()
     {
         $this->clean();
-        $this->client = Elasticsearch\ClientBuilder::create()->setHosts([self::getHost()])->build();
+        $builder = Elasticsearch\ClientBuilder::create()->setHosts([self::getHost()]);
+        if (version_compare(phpversion(), '5.6.6', '<') || ! defined('JSON_PRESERVE_ZERO_FRACTION')) {
+            $builder->allowBadJSONSerialization();
+        }
+        $this->client = $builder->build();
     }
 
     /**