Merge pull request #28 from tomplus/reset_stop
fix: reset _stop when new stream is called
This commit is contained in:
commit
4127e1afa9
@ -109,6 +109,7 @@ class Watch(object):
|
||||
watch.stop()
|
||||
"""
|
||||
|
||||
self._stop = False
|
||||
return_type = self.get_return_type(func)
|
||||
kwargs['watch'] = True
|
||||
kwargs['_preload_content'] = False
|
||||
|
||||
@ -58,6 +58,33 @@ class WatchTests(unittest.TestCase):
|
||||
fake_resp.close.assert_called_once()
|
||||
fake_resp.release_conn.assert_called_once()
|
||||
|
||||
def test_watch_stream_twice(self):
|
||||
w = Watch(float)
|
||||
for step in ['first', 'second']:
|
||||
fake_resp = Mock()
|
||||
fake_resp.close = Mock()
|
||||
fake_resp.release_conn = Mock()
|
||||
fake_resp.read_chunked = Mock(
|
||||
return_value=['{"type": "ADDED", "object": 1}\n'] * 4)
|
||||
|
||||
fake_api = Mock()
|
||||
fake_api.get_namespaces = Mock(return_value=fake_resp)
|
||||
fake_api.get_namespaces.__doc__ = ':return: V1NamespaceList'
|
||||
|
||||
count = 1
|
||||
for e in w.stream(fake_api.get_namespaces):
|
||||
count += 1
|
||||
if count == 3:
|
||||
w.stop()
|
||||
|
||||
self.assertEqual(count, 3)
|
||||
fake_api.get_namespaces.assert_called_once_with(
|
||||
_preload_content=False, watch=True)
|
||||
fake_resp.read_chunked.assert_called_once_with(
|
||||
decode_content=False)
|
||||
fake_resp.close.assert_called_once()
|
||||
fake_resp.release_conn.assert_called_once()
|
||||
|
||||
def test_unmarshal_with_float_object(self):
|
||||
w = Watch()
|
||||
event = w.unmarshal_event('{"type": "ADDED", "object": 1}', 'float')
|
||||
|
||||
Loading…
Reference in New Issue
Block a user