Merge branch '3410-replication-attrs' closes #3410 refs #5011

[arvados.git] / sdk / python / tests / test_websockets.py

diff --git a/sdk/python/tests/test_websockets.py b/sdk/python/tests/test_websockets.py
index 45dd28a1575f416989b42ee18cc53e35d826f3b4..d879ebe1f8062c02d965bd9c845e5e00c57d1e76 100644 (file)
--- a/sdk/python/tests/test_websockets.py
+++ b/sdk/python/tests/test_websockets.py
@@ -3,34 +3,38 @@ import run_test_server
 import unittest
 import arvados
 import arvados.events
+import mock
 import threading
 
-class EventTestBase(object):
-    def runTest(self):
-        run_test_server.authorize_with("admin")
+class WebsocketTest(run_test_server.TestCaseWithServers):
+    MAIN_SERVER = {}
+
+    def setUp(self):
+        self.ws = None
+
+    def tearDown(self):
+        if self.ws:
+            self.ws.close()
+        super(WebsocketTest, self).tearDown()
+
+    def _test_subscribe(self, poll_fallback, expect_type):
+        run_test_server.authorize_with('active')
         events = Queue.Queue(3)
         self.ws = arvados.events.subscribe(
             arvados.api('v1'), [['object_uuid', 'is_a', 'arvados#human']],
-            events.put, poll_fallback=2)
-        self.assertIsInstance(self.ws, self.WS_TYPE)
+            events.put, poll_fallback=poll_fallback)
+        self.assertIsInstance(self.ws, expect_type)
         self.assertEqual(200, events.get(True, 10)['status'])
         human = arvados.api('v1').humans().create(body={}).execute()
         self.assertEqual(human['uuid'], events.get(True, 10)['object_uuid'])
         self.assertTrue(events.empty(), "got more events than expected")
 
-    def tearDown(self):
-        try:
-            self.ws.close()
-        except AttributeError:
-            pass
-        super(EventTestBase, self).tearDown()
-
+    def test_subscribe_websocket(self):
+        self._test_subscribe(
+            poll_fallback=False, expect_type=arvados.events.EventClient)
 
-class WebsocketTest(EventTestBase, run_test_server.TestCaseWithServers):
-    MAIN_SERVER = {'websockets': True}
-    WS_TYPE = arvados.events.EventClient
-
-
-class PollClientTest(EventTestBase, run_test_server.TestCaseWithServers):
-    MAIN_SERVER = {}
-    WS_TYPE = arvados.events.PollClient
+    @mock.patch('arvados.events.EventClient.__init__')
+    def test_subscribe_poll(self, event_client_constr):
+        event_client_constr.side_effect = Exception('All is well')
+        self._test_subscribe(
+            poll_fallback=1, expect_type=arvados.events.PollClient)