Signed-off-by: Bram Leeda <[email protected]>

Add extra assertions for the ElasticSearch check result data
laminas · Jan 4, 2023 · 192bf3f · 192bf3f
1 parent 19581cd
commit 192bf3f
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/test/ElasticSearchTest.php b/test/ElasticSearchTest.php
@@ -27,7 +27,15 @@ public function testElasticSearch(string $clusterStatus, string $expectedResult)
         $mockClient  = new Client(['handler' => $mockHandler]);
         $check       = new ElasticSearch('localhost:9200', [], [], $mockClient);
 
-        static::assertInstanceOf($expectedResult, $check->check());
+        // Assert the ElasticSearch check converts the API response to the correct ResultInterface implementation
+        $checkResult = $check->check();
+        static::assertInstanceOf($expectedResult, $checkResult);
+
+        // Assert ElasticSearch check returns extra data
+        $resultData = $checkResult->getData();
+        static::assertIsArray($resultData);
+        static::assertArrayHasKey('responseTime', $resultData);
+        static::assertIsNumeric($resultData['responseTime']);
     }
 
     /**