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

Fix remaining falling tests in release mode

Browse Source 
Reviewers: buda, mferencevic

Reviewed By: buda

Subscribers: pullbot

Differential Revision: https://phabricator.memgraph.io/D655
This commit is contained in:
Mislav Bradac 2017-08-09 16:10:58 +02:00
parent 1d112e1141
commit 98241f99a6
3 changed files with 7 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/database/graph_db_accessor.hpp
View File

@ -180,8 +180,8 @@ class GraphDbAccessor {
debug_assert(db_.label_property_index_.IndexExists(
LabelPropertyIndex::Key(label, property)),
"Label+property index doesn't exist.");
debug_assert(value.type() != PropertyValue::Type::Null,
"Can't query index for propery value type null.");
permanent_assert(value.type() != PropertyValue::Type::Null,
"Can't query index for propery value type null.");
return iter::imap([this, current_state](
auto vlist) { return VertexAccessor(*vlist, *this); },
db_.label_property_index_.GetVlists(

10
src/database/indexes/label_property_index.hpp
View File

@ -249,11 +249,11 @@ class LabelPropertyIndex {
debug_assert(ready_for_use_.access().contains(key), "Index not yet ready.");
auto type = [](const auto &bound) { return bound.value().value().type(); };
debug_assert(lower || upper, "At least one bound must be provided");
debug_assert(!lower || type(lower) != PropertyValue::Type::Null,
"Null value is not a valid index bound");
debug_assert(!upper || type(upper) != PropertyValue::Type::Null,
"Null value is not a valid index bound");
permanent_assert(lower || upper, "At least one bound must be provided");
permanent_assert(!lower || type(lower) != PropertyValue::Type::Null,
"Null value is not a valid index bound");
permanent_assert(!upper || type(upper) != PropertyValue::Type::Null,
"Null value is not a valid index bound");
// helper function for creating a bound with an IndexElement
auto make_index_bound = [](const auto &optional_bound, bool bottom) {

1
tests/concurrent/push_queue.cpp
View File

@ -82,7 +82,6 @@ TEST_F(IntQueue, IteratorEnd) {
EXPECT_NE(it, cpq.end());
for (int i = 0; i < 5; i++) it++;
EXPECT_EQ(it, cpq.end());
EXPECT_DEATH(*it, "Dereferencing");
}
TEST_F(IntQueue, IteratorCopy) {