try to fix (#637)

victimsnino · Sep 4, 2024 · e01a513 · e01a513
1 parent 6fd2da6
commit e01a513
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/src/tests/rppgrpc/test_async_server.cpp b/src/tests/rppgrpc/test_async_server.cpp
@@ -55,7 +55,8 @@ TEST_CASE("Async server")
             if constexpr (requires { reactor->get_observer(); })
                 reactor->get_observer().on_completed();
 
-            CHECK(writer->Finish().ok());
+            const auto res = writer->Finish().ok();
+            CHECK(res);
             wait(last);
         }
 
@@ -92,7 +93,8 @@ TEST_CASE("Async server")
             if constexpr (requires { reactor->get_observer(); })
                 reactor->get_observer().on_completed();
 
-            REQUIRE(writer->Finish().ok());
+            const auto res = writer->Finish().ok();
+            REQUIRE(res);
 
             wait(last);
         }
@@ -119,7 +121,8 @@ TEST_CASE("Async server")
                 REQUIRE(response.value() == i);
             }
             REQUIRE(!writer->Read(&response));
-            REQUIRE(writer->Finish().ok());
+            const auto res = writer->Finish().ok();
+            REQUIRE(res);
             wait(last);
         }
     };

diff --git a/src/tests/utils/rpp_trompeloil.hpp b/src/tests/utils/rpp_trompeloil.hpp
@@ -30,7 +30,7 @@ namespace trompeloeil
     inline void reporter<specialized>::sendOk(
         const char* trompeloeil_mock_calls_done_correctly)
     {
-        REQUIRE(trompeloeil_mock_calls_done_correctly != 0);
+        REQUIRE(trompeloeil_mock_calls_done_correctly);
     }
 } // namespace trompeloeil