diff options
author | Richard van der Hoff <richard@matrix.org> | 2015-10-26 18:32:49 +0000 |
---|---|---|
committer | Richard van der Hoff <richard@matrix.org> | 2015-10-26 18:44:03 +0000 |
commit | f69a5c9134a3e4bba929dc76d561d9cc42cadeac (patch) | |
tree | 24915683079b7031654130f3aa1985f84b5b6a7f /tests | |
parent | Merge pull request #328 from matrix-org/erikj/search (diff) | |
download | synapse-f69a5c9134a3e4bba929dc76d561d9cc42cadeac.tar.xz |
Fix a 500 error resulting from empty room_ids
POST /_matrix/client/api/v1/rooms//send/a.b.c gave a 500 error, because we assumed that rooms always had at least one character.
Diffstat (limited to 'tests')
-rw-r--r-- | tests/test_types.py | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/tests/test_types.py b/tests/test_types.py index b29a8415b1..495cd20f02 100644 --- a/tests/test_types.py +++ b/tests/test_types.py @@ -15,13 +15,14 @@ from tests import unittest +from synapse.api.errors import SynapseError from synapse.server import BaseHomeServer from synapse.types import UserID, RoomAlias mock_homeserver = BaseHomeServer(hostname="my.domain") -class UserIDTestCase(unittest.TestCase): +class UserIDTestCase(unittest.TestCase): def test_parse(self): user = UserID.from_string("@1234abcd:my.domain") @@ -29,6 +30,11 @@ class UserIDTestCase(unittest.TestCase): self.assertEquals("my.domain", user.domain) self.assertEquals(True, mock_homeserver.is_mine(user)) + def test_pase_empty(self): + with self.assertRaises(SynapseError): + UserID.from_string("") + + def test_build(self): user = UserID("5678efgh", "my.domain") @@ -44,7 +50,6 @@ class UserIDTestCase(unittest.TestCase): class RoomAliasTestCase(unittest.TestCase): - def test_parse(self): room = RoomAlias.from_string("#channel:my.domain") |