summary refs log tree commit diff
diff options
context:
space:
mode:
authorErik Johnston <erik@matrix.org>2016-06-07 09:34:50 +0100
committerErik Johnston <erik@matrix.org>2016-06-07 09:34:50 +0100
commit310197bab5cf8ed2c26fae522f15f092dbcdff58 (patch)
treeeea9cc3468f64f8608289077febc111f341840c7
parentMerge pull request #845 from matrix-org/markjh/synchrotron_presence (diff)
downloadsynapse-310197bab5cf8ed2c26fae522f15f092dbcdff58.tar.xz
Fix AS retries
-rw-r--r--synapse/storage/appservice.py4
-rw-r--r--tests/storage/test_appservice.py6
2 files changed, 5 insertions, 5 deletions
diff --git a/synapse/storage/appservice.py b/synapse/storage/appservice.py
index ffb7d4a25b..a281571637 100644
--- a/synapse/storage/appservice.py
+++ b/synapse/storage/appservice.py
@@ -320,10 +320,10 @@ class ApplicationServiceTransactionStore(SQLBaseStore):
 
         event_ids = json.loads(entry["event_ids"])
 
-        events = yield self.get_events(event_ids)
+        event_map = yield self.get_events(event_ids)
 
         defer.returnValue(AppServiceTransaction(
-            service=service, id=entry["txn_id"], events=events
+            service=service, id=entry["txn_id"], events=event_map.values()
         ))
 
     def _get_oldest_unsent_txn(self, txn, service):
diff --git a/tests/storage/test_appservice.py b/tests/storage/test_appservice.py
index f44c4870e3..6db4b966db 100644
--- a/tests/storage/test_appservice.py
+++ b/tests/storage/test_appservice.py
@@ -353,21 +353,21 @@ class ApplicationServiceTransactionStoreTestCase(unittest.TestCase):
     @defer.inlineCallbacks
     def test_get_oldest_unsent_txn(self):
         service = Mock(id=self.as_list[0]["id"])
-        events = [Mock(event_id="e1"), Mock(event_id="e2")]
+        events = {"e1": Mock(event_id="e1"), "e2": Mock(event_id="e2")}
         other_events = [Mock(event_id="e5"), Mock(event_id="e6")]
 
         # we aren't testing store._base stuff here, so mock this out
         self.store.get_events = Mock(return_value=events)
 
         yield self._insert_txn(self.as_list[1]["id"], 9, other_events)
-        yield self._insert_txn(service.id, 10, events)
+        yield self._insert_txn(service.id, 10, events.values())
         yield self._insert_txn(service.id, 11, other_events)
         yield self._insert_txn(service.id, 12, other_events)
 
         txn = yield self.store.get_oldest_unsent_txn(service)
         self.assertEquals(service, txn.service)
         self.assertEquals(10, txn.id)
-        self.assertEquals(events, txn.events)
+        self.assertEquals(events.values(), txn.events)
 
     @defer.inlineCallbacks
     def test_get_appservices_by_state_single(self):