From 9889b9421d823222c16529defebb9cfe772fd45b Mon Sep 17 00:00:00 2001
From: Teon Banek <teon.banek@memgraph.io>
Date: Tue, 14 Jan 2020 10:02:09 +0100
Subject: [PATCH] tests/unit: Don't ignore result of DetachRemoveVertex
Reviewers: mferencevic
Reviewed By: mferencevic
Subscribers: pullbot
Differential Revision: https://phabricator.memgraph.io/D2620
---
tests/unit/query_plan_bag_semantics.cpp | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/unit/query_plan_bag_semantics.cpp b/tests/unit/query_plan_bag_semantics.cpp
index c6427bf72..a8abe83a9 100644
--- a/tests/unit/query_plan_bag_semantics.cpp
+++ b/tests/unit/query_plan_bag_semantics.cpp
@@ -144,7 +144,7 @@ TEST(QueryPlan, OrderBy) {
for (const auto &v : order_value_pair.second) values.emplace_back(v);
// empty database
for (auto vertex : dba.Vertices(storage::View::OLD))
- dba.DetachRemoveVertex(&vertex);
+ ASSERT_TRUE(dba.DetachRemoveVertex(&vertex).HasValue());
dba.AdvanceCommand();
ASSERT_EQ(0, CountIterable(dba.Vertices(storage::View::OLD)));
@@ -268,7 +268,7 @@ TEST(QueryPlan, OrderByExceptions) {
for (const auto &pair : exception_pairs) {
// empty database
for (auto vertex : dba.Vertices(storage::View::OLD))
- dba.DetachRemoveVertex(&vertex);
+ ASSERT_TRUE(dba.DetachRemoveVertex(&vertex).HasValue());
dba.AdvanceCommand();
ASSERT_EQ(0, CountIterable(dba.Vertices(storage::View::OLD)));