Fixes logcat regex issue

AndroidRunner/Script.py

@@ -2,7 +2,7 @@
 import multiprocessing as mp
 import os.path as op
 import signal
-
+import time
 from . import Tests
 from .util import FileNotFoundError
 
@@ -42,7 +42,8 @@ def mp_logcat_regex(queue, device, regex):
         """The multiprocessing wrapper of Device.logcat_regex()"""
         # https://stackoverflow.com/a/21936682
         # pyadb uses subprocess.communicate(), therefore it blocks
-        device.logcat_regex(regex)
+        while not device.logcat_regex(regex):
+            time.sleep(1)
         queue.put('logcat')
 
     def run(self, device, *args, **kwargs):

tests/unit/test_scripts.py

@@ -3,7 +3,7 @@
 
 import pytest
 from mock import Mock, call, patch
-
+import time
 import paths
 from AndroidRunner.MonkeyReplay import MonkeyReplay, MonkeyReplayError
 from AndroidRunner.MonkeyRunner import MonkeyRunner
@@ -287,3 +287,14 @@ def test_script_mp_run_error(self, error_script_path):
         assert test_queue.put.call_count == 2
         assert 'NotImplementedError' in str(test_queue.put.call_args_list)
         assert 'script' in str(test_queue.put.call_args_list[1][0])
+
+    @patch("time.sleep")
+    def test_mp_logcat_regex_one_iteration(self, sleep, script_path):
+        fake_device = Mock()
+        fake_device.logcat_regex.side_effect = [False, True]
+        test_queue = Mock()
+        test_script = Python3(script_path)
+        test_script.mp_logcat_regex(test_queue, fake_device, "Test")
+
+        sleep.assert_called_once_with(1)
+        test_queue.put.assert_called_once_with("logcat")