Require cancel listener availability before entering recording state
This commit is contained in:
parent
aa77dbc395
commit
2b494851a6
2 changed files with 46 additions and 3 deletions
|
|
@ -15,14 +15,17 @@ from config import Config, VocabularyReplacement
|
|||
|
||||
|
||||
class FakeDesktop:
|
||||
def __init__(self):
|
||||
def __init__(self, *, fail_cancel_listener: bool = False):
|
||||
self.inject_calls = []
|
||||
self.quit_calls = 0
|
||||
self.cancel_listener_start_calls = 0
|
||||
self.cancel_listener_stop_calls = 0
|
||||
self.cancel_listener_callback = None
|
||||
self.fail_cancel_listener = fail_cancel_listener
|
||||
|
||||
def start_cancel_listener(self, callback) -> None:
|
||||
if self.fail_cancel_listener:
|
||||
raise RuntimeError("cancel listener unavailable")
|
||||
self.cancel_listener_start_calls += 1
|
||||
self.cancel_listener_callback = callback
|
||||
|
||||
|
|
@ -93,6 +96,18 @@ class FakeAudio:
|
|||
self.size = size
|
||||
|
||||
|
||||
class FakeStream:
|
||||
def __init__(self):
|
||||
self.stop_calls = 0
|
||||
self.close_calls = 0
|
||||
|
||||
def stop(self):
|
||||
self.stop_calls += 1
|
||||
|
||||
def close(self):
|
||||
self.close_calls += 1
|
||||
|
||||
|
||||
class DaemonTests(unittest.TestCase):
|
||||
def _config(self) -> Config:
|
||||
cfg = Config()
|
||||
|
|
@ -272,6 +287,21 @@ class DaemonTests(unittest.TestCase):
|
|||
self.assertEqual(desktop.cancel_listener_stop_calls, 1)
|
||||
self.assertIsNone(desktop.cancel_listener_callback)
|
||||
|
||||
@patch("aman.start_audio_recording")
|
||||
def test_recording_does_not_start_when_cancel_listener_fails(self, start_mock):
|
||||
stream = FakeStream()
|
||||
start_mock.return_value = (stream, object())
|
||||
desktop = FakeDesktop(fail_cancel_listener=True)
|
||||
daemon = self._build_daemon(desktop, FakeModel(), verbose=False)
|
||||
|
||||
daemon.toggle()
|
||||
|
||||
self.assertEqual(daemon.get_state(), aman.State.IDLE)
|
||||
self.assertIsNone(daemon.stream)
|
||||
self.assertIsNone(daemon.record)
|
||||
self.assertEqual(stream.stop_calls, 1)
|
||||
self.assertEqual(stream.close_calls, 1)
|
||||
|
||||
|
||||
class LockTests(unittest.TestCase):
|
||||
def test_lock_rejects_second_instance(self):
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue