summary refs log tree commit diff
diff options
context:
space:
mode:
authorErik Johnston <erik@matrix.org>2014-12-12 10:42:27 +0000
committerErik Johnston <erik@matrix.org>2014-12-12 10:42:27 +0000
commitfa4b610ae383bdced11068832d880d14b1df576d (patch)
treeb105b44c4c1e317d45c06b9df5eb2a4e2a7ea267
parentMerge branch 'develop' of github.com:matrix-org/synapse into events_refactor (diff)
downloadsynapse-fa4b610ae383bdced11068832d880d14b1df576d.tar.xz
Fix stream test. Make sure we add join to auth_events for invitiations
-rw-r--r--synapse/api/auth.py6
-rw-r--r--tests/rest/test_events.py4
2 files changed, 6 insertions, 4 deletions
diff --git a/synapse/api/auth.py b/synapse/api/auth.py
index cd0deeb0e6..c76f5180f6 100644
--- a/synapse/api/auth.py
+++ b/synapse/api/auth.py
@@ -61,6 +61,8 @@ class Auth(object):
             if event.type == RoomAliasesEvent.TYPE:
                 return True
 
+            logger.debug("Auth events: %s", auth_events)
+
             if event.type == RoomMemberEvent.TYPE:
                 allowed = self.is_membership_change_allowed(
                     event, auth_events
@@ -389,8 +391,12 @@ class Auth(object):
                 if join_rule_event:
                     auth_ids.append(join_rule_event.event_id)
 
+            if e_type == Membership.JOIN:
                 if member_event and not is_public:
                     auth_ids.append(member_event.event_id)
+            else:
+                if member_event:
+                    auth_ids.append(member_event.event_id)
         elif member_event:
             if member_event.content["membership"] == Membership.JOIN:
                 auth_ids.append(member_event.event_id)
diff --git a/tests/rest/test_events.py b/tests/rest/test_events.py
index 4a3234c332..d3159e2cf4 100644
--- a/tests/rest/test_events.py
+++ b/tests/rest/test_events.py
@@ -113,9 +113,6 @@ class EventStreamPermissionsTestCase(RestTestCase):
     def setUp(self):
         self.mock_resource = MockHttpResource(prefix=PATH_PREFIX)
 
-        persistence_service = Mock(spec=["get_latest_pdus_in_context"])
-        persistence_service.get_latest_pdus_in_context.return_value = []
-
         self.mock_config = NonCallableMock()
         self.mock_config.signing_key = [MockKey()]
 
@@ -127,7 +124,6 @@ class EventStreamPermissionsTestCase(RestTestCase):
             db_pool=db_pool,
             http_client=None,
             replication_layer=Mock(),
-            persistence_service=persistence_service,
             clock=Mock(spec=[
                 "call_later",
                 "cancel_call_later",