diff --git a/astrid/plugin-src/com/todoroo/astrid/actfm/sync/messages/ChangesHappened.java b/astrid/plugin-src/com/todoroo/astrid/actfm/sync/messages/ChangesHappened.java index 5b92d004f..c04a33a26 100644 --- a/astrid/plugin-src/com/todoroo/astrid/actfm/sync/messages/ChangesHappened.java +++ b/astrid/plugin-src/com/todoroo/astrid/actfm/sync/messages/ChangesHappened.java @@ -66,7 +66,7 @@ public class ChangesHappened { m.setValue(OutstandingEntry.COLUMN_STRING_PROPERTY, addedOrRemoved); m.setValue(OutstandingEntry.VALUE_STRING_PROPERTY, tagUuid); database.insert(outstandingTable.name, null, m.getSetValues()); + database.getDatabase().setTransactionSuccessful(); return true; } diff --git a/tests-sync/src/com/todoroo/astrid/sync/SyncMessageTest.java b/tests-sync/src/com/todoroo/astrid/sync/SyncMessageTest.java index d0994c451..1c3f0f5a7 100644 --- a/tests-sync/src/com/todoroo/astrid/sync/SyncMessageTest.java +++ b/tests-sync/src/com/todoroo/astrid/sync/SyncMessageTest.java @@ -25,6 +25,7 @@ public class SyncMessageTest extends NewSyncTestCase { try { ChangesHappened changes = ChangesHappened.instantiateChangesHappened(t.getId(), ModelType.TYPE_TASK); assertNotNull(changes.serializeToJSON()); + assertTrue(changes.getChanges().size() > 0); assertFalse(RemoteModel.NO_UUID.equals(changes.getUUID())); assertEquals(t.getValue(Task.UUID), changes.getUUID()); } catch (Exception e) {