From 72b21687d41c268cb31f1a0f953094c332c7529d Mon Sep 17 00:00:00 2001 From: Sven Tennie Date: Tue, 5 Nov 2024 15:58:41 +0100 Subject: [PATCH] fix: Delete confusing assertion in test The assertion implied that a version conflict can lead to a Right value (maybe for either ElasticSearch or OpenSearch.) However, the parsing logic would force this to be a Left value. --- tests/Test/DocumentsSpec.hs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/Test/DocumentsSpec.hs b/tests/Test/DocumentsSpec.hs index d55fa388..2c173ace 100644 --- a/tests/Test/DocumentsSpec.hs +++ b/tests/Test/DocumentsSpec.hs @@ -35,7 +35,7 @@ spec = liftIO $ isCreated res `shouldBe` True insertedConflict <- tryEsError $ insertData' cfg case insertedConflict of - Right (res', _) -> liftIO $ isVersionConflict res' `shouldBe` True + Right _ -> liftIO $ expectationFailure "This should never not happen." Left e -> liftIO $ errorStatus e `shouldBe` Just 409 it "indexes two documents in a parent/child relationship and checks that the child exists" $