Fix for #1012

Author: ezimuel

src/Elasticsearch/Common/Exceptions/Serializer/JsonErrorException.php

@@ -10,7 +10,7 @@
  * Class JsonErrorException
  *
  * @category Elasticsearch
- * @package  Elasticsearch\Common\Exceptions\Curl
+ * @package  Elasticsearch\Common\Exceptions\Serializer
  * @author   Bez Hermoso <[email protected]>
  * @license  http://www.apache.org/licenses/LICENSE-2.0 Apache2
  * @link     http://elastic.co

src/Elasticsearch/Serializers/SmartSerializer.php

@@ -81,7 +81,7 @@ private function decode(?string $data): array
                 return $result;
             } catch (\JsonException $e) {
                 $result = $result ?? [];
-                throw new JsonErrorException($e->getMessage(), $data, $result);
+                throw new JsonErrorException($e->getCode(), $data, $result);
             }
         }
 

tests/Elasticsearch/Tests/Serializers/SmartSerializerTest.php

@@ -0,0 +1,35 @@
+<?php
+
+declare(strict_types = 1);
+
+namespace Elasticsearch\Tests\Serializers;
+
+use Elasticsearch\Common\Exceptions\Serializer\JsonErrorException;
+use Elasticsearch\Serializers\SmartSerializer;
+use Mockery as m;
+use PHPUnit\Framework\TestCase;
+
+/**
+ * Class SmartSerializerTest
+ *
+ * @package Elasticsearch\Tests\Serializers
+ */
+class SmartSerializerTest extends TestCase
+{
+    public function setUp(): void
+    {
+        $this->serializer = new SmartSerializer();
+    }
+
+    /**
+     * @requires PHP 7.3
+     * @see https://github.com/elastic/elasticsearch-php/issues/1012
+     */
+    public function testThrowJsonErrorException()
+    {
+        $this->expectException(JsonErrorException::class);
+        $this->expectExceptionCode(JSON_ERROR_SYNTAX);
+
+        $result = $this->serializer->deserialize('{ "foo" : bar" }', []);
+    }
+}