tursom/memgraph
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

tests/unit: Don't ignore result of DetachRemoveVertex

Browse Source 
Reviewers: mferencevic

Reviewed By: mferencevic

Subscribers: pullbot

Differential Revision: https://phabricator.memgraph.io/D2620
This commit is contained in:
Teon Banek 2020-01-14 10:02:09 +01:00
parent 4776bea221
commit 9889b9421d
1 changed files with 2 additions and 2 deletions

4
tests/unit/query_plan_bag_semantics.cpp
View File

@ -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)));