Browse Source

Fix timing-related test error in Alpine Edge.

Dan Helfman 4 years ago
parent
commit
ccd1627175
1 changed files with 3 additions and 1 deletions
  1. 3 1
      tests/integration/test_execute.py

+ 3 - 1
tests/integration/test_execute.py

@@ -88,13 +88,15 @@ def test_log_outputs_skips_error_output_in_exception_for_process_with_none_stdou
 
 def test_log_outputs_kills_other_processes_when_one_errors():
     flexmock(module.logger).should_receive('log')
-    flexmock(module).should_receive('exit_code_indicates_error').and_return(True)
     flexmock(module).should_receive('command_for_process').and_return('grep')
 
     process = subprocess.Popen(['grep'], stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
+    flexmock(module).should_receive('exit_code_indicates_error').with_args(process, None, 'borg').and_return(False)
+    flexmock(module).should_receive('exit_code_indicates_error').with_args(process, 2, 'borg').and_return(True)
     other_process = subprocess.Popen(
         ['watch', 'true'], stdout=subprocess.PIPE, stderr=subprocess.STDOUT
     )
+    flexmock(module).should_receive('exit_code_indicates_error').with_args(other_process, None, 'borg').and_return(False)
     flexmock(module).should_receive('output_buffer_for_process').with_args(process, ()).and_return(
         process.stdout
     )