Skip to content
Snippets Groups Projects
Unverified Commit 74b74462 authored by Erik Johnston's avatar Erik Johnston Committed by GitHub
Browse files

Fix `/events/:event_id` deprecated API. (#6731)

parent 0f6e525b
No related branches found
No related tags found
No related merge requests found
Fix `/events/:event_id` deprecated API.
...@@ -70,7 +70,6 @@ class EventStreamRestServlet(RestServlet): ...@@ -70,7 +70,6 @@ class EventStreamRestServlet(RestServlet):
return 200, {} return 200, {}
# TODO: Unit test gets, with and without auth, with different kinds of events.
class EventRestServlet(RestServlet): class EventRestServlet(RestServlet):
PATTERNS = client_patterns("/events/(?P<event_id>[^/]*)$", v1=True) PATTERNS = client_patterns("/events/(?P<event_id>[^/]*)$", v1=True)
...@@ -78,6 +77,7 @@ class EventRestServlet(RestServlet): ...@@ -78,6 +77,7 @@ class EventRestServlet(RestServlet):
super(EventRestServlet, self).__init__() super(EventRestServlet, self).__init__()
self.clock = hs.get_clock() self.clock = hs.get_clock()
self.event_handler = hs.get_event_handler() self.event_handler = hs.get_event_handler()
self.auth = hs.get_auth()
self._event_serializer = hs.get_event_client_serializer() self._event_serializer = hs.get_event_client_serializer()
async def on_GET(self, request, event_id): async def on_GET(self, request, event_id):
......
...@@ -134,3 +134,30 @@ class EventStreamPermissionsTestCase(unittest.HomeserverTestCase): ...@@ -134,3 +134,30 @@ class EventStreamPermissionsTestCase(unittest.HomeserverTestCase):
# someone else set topic, expect 6 (join,send,topic,join,send,topic) # someone else set topic, expect 6 (join,send,topic,join,send,topic)
pass pass
class GetEventsTestCase(unittest.HomeserverTestCase):
servlets = [
events.register_servlets,
room.register_servlets,
synapse.rest.admin.register_servlets_for_client_rest_resource,
login.register_servlets,
]
def prepare(self, hs, reactor, clock):
# register an account
self.user_id = self.register_user("sid1", "pass")
self.token = self.login(self.user_id, "pass")
self.room_id = self.helper.create_room_as(self.user_id, tok=self.token)
def test_get_event_via_events(self):
resp = self.helper.send(self.room_id, tok=self.token)
event_id = resp["event_id"]
request, channel = self.make_request(
"GET", "/events/" + event_id, access_token=self.token,
)
self.render(request)
self.assertEquals(channel.code, 200, msg=channel.result)
...@@ -463,7 +463,7 @@ class HomeserverTestCase(TestCase): ...@@ -463,7 +463,7 @@ class HomeserverTestCase(TestCase):
# Create the user # Create the user
request, channel = self.make_request("GET", "/_matrix/client/r0/admin/register") request, channel = self.make_request("GET", "/_matrix/client/r0/admin/register")
self.render(request) self.render(request)
self.assertEqual(channel.code, 200) self.assertEqual(channel.code, 200, msg=channel.result)
nonce = channel.json_body["nonce"] nonce = channel.json_body["nonce"]
want_mac = hmac.new(key=b"shared", digestmod=hashlib.sha1) want_mac = hmac.new(key=b"shared", digestmod=hashlib.sha1)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment