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

Fix mvcc tests

Browse Source 
Summary: Daily release build was failing because of these tests.

Reviewers: florijan, mferencevic

Reviewed By: florijan

Subscribers: pullbot

Differential Revision: https://phabricator.memgraph.io/D917
This commit is contained in:
Mislav Bradac 2017-10-18 16:03:06 +02:00
parent 19a44a7d94
commit 088a177939
2 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tests/unit/mvcc_one_transaction.cpp
View File

@ -106,7 +106,7 @@ TEST_F(Mvcc, RemoveAdvanceRemove) {
EXPECT_EQ(T2_FIND, v1);
engine.Advance(t2->id_);
EXPECT_EQ(T2_FIND, nullptr);
EXPECT_DEATH(T2_REMOVE, ".*nullptr.*");
EXPECT_FALSE(T2_FIND);
}
int main(int argc, char **argv) {

6
tests/unit/mvcc_parallel_update.cpp
View File

@ -2,6 +2,8 @@
#undef T4_FIND
#define T4_FIND version_list.find(*t4)
#undef T3_FIND
#define T3_FIND version_list.find(*t3)
// IMPORTANT: look definiton of EXPECT_CRE and EXPECT_EXP macros in
// tests/mvcc_find_update_common.hpp. Numbers in those macros represent
@ -61,7 +63,7 @@ TEST_F(Mvcc, RemCmtRemCmt1) {
T2_REMOVE;
T2_COMMIT;
T3_BEGIN;
EXPECT_DEATH(T3_REMOVE, ".*nullptr.*");
EXPECT_FALSE(T3_FIND);
}
TEST_F(Mvcc, UpdCmtUpdAbt1) {
@ -108,7 +110,7 @@ TEST_F(Mvcc, RemCmtRemAbt1) {
T2_REMOVE;
T2_COMMIT;
T3_BEGIN;
EXPECT_DEATH(T3_REMOVE, ".*nullptr.*");
EXPECT_FALSE(T3_FIND);
}
TEST_F(Mvcc, UpdAbtUpdCmt1) {