status-im
/
realm-js
mirror of https://github.com/status-im/realm-js.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Update moves when there is another move to exactly the previous move target

This commit is contained in:
Thomas Goyne 2016-03-21 12:47:21 -07:00
parent 0a3158ce74
commit feed7c3479
2 changed files with 12 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/collection_notifications.cpp
View File

@ -226,7 +226,7 @@ void CollectionChangeBuilder::move(size_t from, size_t to)
// to the other // to the other
if (move.to >= to && move.to < from) if (move.to >= to && move.to < from)
++move.to; ++move.to;
else if (move.to < to && move.to > from) else if (move.to <= to && move.to > from)
--move.to; --move.to;
continue; continue;
} }

11
tests/collection_change_indices.cpp
View File

@ -284,6 +284,17 @@ TEST_CASE("[collection_change] move()") {
c.move(6, 12); c.move(6, 12);
REQUIRE_INDICES(c.modifications, 9); REQUIRE_INDICES(c.modifications, 9);
} }
SECTION("bumps previous moves to the same location") {
c.move(5, 10);
c.move(7, 10);
REQUIRE_MOVES(c, {5, 9}, {8, 10});
c = {};
c.move(5, 10);
c.move(15, 10);
REQUIRE_MOVES(c, {5, 11}, {15, 10});
}
} }
TEST_CASE("[collection_change] calculate() unsorted") { TEST_CASE("[collection_change] calculate() unsorted") {