Use TestCase::fail instead of failing assertion

ruflin · Jun 3, 2022 · 1cd8980 · 1cd8980
1 parent 0beea19
commit 1cd8980
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/tests/ReindexTest.php b/tests/ReindexTest.php
@@ -208,7 +208,8 @@ public function testReindexWithRemote(): void
 
         try {
             $reindex->run();
-            $this->assertFalse(true, 'Elasticsearch should have thrown an Exception, maybe the remote option has not been sent.');
+
+            $this->fail('Elasticsearch should have thrown an Exception, maybe the remote option has not been sent.');
         } catch (ResponseException $exception) {
             $this->assertStringContainsString('reindex.remote.whitelist', $exception->getMessage());
         }