123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373 |
- import logging
- import subprocess
- import sys
- import pytest
- from flexmock import flexmock
- from borgmatic import execute as module
- def test_log_outputs_logs_each_line_separately():
- flexmock(module.logger).should_receive('log').with_args(logging.INFO, 'hi').once()
- flexmock(module.logger).should_receive('log').with_args(logging.INFO, 'there').once()
- flexmock(module).should_receive('interpret_exit_code').and_return(module.Exit_status.SUCCESS)
- hi_process = subprocess.Popen(['echo', 'hi'], stdout=subprocess.PIPE)
- flexmock(module).should_receive('output_buffer_for_process').with_args(
- hi_process, ()
- ).and_return(hi_process.stdout)
- there_process = subprocess.Popen(['echo', 'there'], stdout=subprocess.PIPE)
- flexmock(module).should_receive('output_buffer_for_process').with_args(
- there_process, ()
- ).and_return(there_process.stdout)
- module.log_outputs(
- (hi_process, there_process),
- exclude_stdouts=(),
- output_log_level=logging.INFO,
- borg_local_path='borg',
- borg_exit_codes=None,
- )
- def test_log_outputs_skips_logs_for_process_with_none_stdout():
- flexmock(module.logger).should_receive('log').with_args(logging.INFO, 'hi').never()
- flexmock(module.logger).should_receive('log').with_args(logging.INFO, 'there').once()
- flexmock(module).should_receive('interpret_exit_code').and_return(module.Exit_status.SUCCESS)
- hi_process = subprocess.Popen(['echo', 'hi'], stdout=None)
- flexmock(module).should_receive('output_buffer_for_process').with_args(
- hi_process, ()
- ).and_return(hi_process.stdout)
- there_process = subprocess.Popen(['echo', 'there'], stdout=subprocess.PIPE)
- flexmock(module).should_receive('output_buffer_for_process').with_args(
- there_process, ()
- ).and_return(there_process.stdout)
- module.log_outputs(
- (hi_process, there_process),
- exclude_stdouts=(),
- output_log_level=logging.INFO,
- borg_local_path='borg',
- borg_exit_codes=None,
- )
- def test_log_outputs_returns_output_without_logging_for_output_log_level_none():
- flexmock(module.logger).should_receive('log').never()
- flexmock(module).should_receive('interpret_exit_code').and_return(module.Exit_status.SUCCESS)
- hi_process = subprocess.Popen(['echo', 'hi'], stdout=subprocess.PIPE)
- flexmock(module).should_receive('output_buffer_for_process').with_args(
- hi_process, ()
- ).and_return(hi_process.stdout)
- there_process = subprocess.Popen(['echo', 'there'], stdout=subprocess.PIPE)
- flexmock(module).should_receive('output_buffer_for_process').with_args(
- there_process, ()
- ).and_return(there_process.stdout)
- captured_outputs = module.log_outputs(
- (hi_process, there_process),
- exclude_stdouts=(),
- output_log_level=None,
- borg_local_path='borg',
- borg_exit_codes=None,
- )
- assert captured_outputs == {hi_process: 'hi', there_process: 'there'}
- def test_log_outputs_includes_error_output_in_exception():
- flexmock(module.logger).should_receive('log')
- flexmock(module).should_receive('interpret_exit_code').and_return(module.Exit_status.ERROR)
- flexmock(module).should_receive('command_for_process').and_return('grep')
- process = subprocess.Popen(['grep'], stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
- flexmock(module).should_receive('output_buffer_for_process').and_return(process.stdout)
- with pytest.raises(subprocess.CalledProcessError) as error:
- module.log_outputs(
- (process,),
- exclude_stdouts=(),
- output_log_level=logging.INFO,
- borg_local_path='borg',
- borg_exit_codes=None,
- )
- assert error.value.output
- def test_log_outputs_logs_multiline_error_output():
- '''
- Make sure that all error output lines get logged, not just (for instance) the first few lines
- of a process' traceback.
- '''
- flexmock(module.logger).should_receive('log')
- flexmock(module).should_receive('interpret_exit_code').and_return(module.Exit_status.ERROR)
- flexmock(module).should_receive('command_for_process').and_return('grep')
- process = subprocess.Popen(
- ['python', '-c', 'foopydoo'], stdout=subprocess.PIPE, stderr=subprocess.STDOUT
- )
- flexmock(module).should_receive('output_buffer_for_process').and_return(process.stdout)
- flexmock(module.logger).should_call('log').at_least().times(3)
- with pytest.raises(subprocess.CalledProcessError):
- module.log_outputs(
- (process,),
- exclude_stdouts=(),
- output_log_level=logging.INFO,
- borg_local_path='borg',
- borg_exit_codes=None,
- )
- def test_log_outputs_skips_error_output_in_exception_for_process_with_none_stdout():
- flexmock(module.logger).should_receive('log')
- flexmock(module).should_receive('interpret_exit_code').and_return(module.Exit_status.ERROR)
- flexmock(module).should_receive('command_for_process').and_return('grep')
- process = subprocess.Popen(['grep'], stdout=None)
- flexmock(module).should_receive('output_buffer_for_process').and_return(process.stdout)
- with pytest.raises(subprocess.CalledProcessError) as error:
- module.log_outputs(
- (process,),
- exclude_stdouts=(),
- output_log_level=logging.INFO,
- borg_local_path='borg',
- borg_exit_codes=None,
- )
- assert error.value.returncode == 2
- assert not error.value.output
- def test_log_outputs_kills_other_processes_and_raises_when_one_errors():
- flexmock(module.logger).should_receive('log')
- flexmock(module).should_receive('command_for_process').and_return('grep')
- process = subprocess.Popen(['grep'], stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
- flexmock(module).should_receive('interpret_exit_code').with_args(
- ['grep'],
- None,
- 'borg',
- None,
- ).and_return(module.Exit_status.SUCCESS)
- flexmock(module).should_receive('interpret_exit_code').with_args(
- ['grep'],
- 2,
- 'borg',
- None,
- ).and_return(module.Exit_status.ERROR)
- other_process = subprocess.Popen(
- ['sleep', '2'], stdout=subprocess.PIPE, stderr=subprocess.STDOUT
- )
- flexmock(module).should_receive('interpret_exit_code').with_args(
- ['sleep', '2'],
- None,
- 'borg',
- None,
- ).and_return(module.Exit_status.SUCCESS)
- flexmock(module).should_receive('output_buffer_for_process').with_args(process, ()).and_return(
- process.stdout
- )
- flexmock(module).should_receive('output_buffer_for_process').with_args(
- other_process, ()
- ).and_return(other_process.stdout)
- flexmock(other_process).should_receive('kill').once()
- with pytest.raises(subprocess.CalledProcessError) as error:
- module.log_outputs(
- (process, other_process),
- exclude_stdouts=(),
- output_log_level=logging.INFO,
- borg_local_path='borg',
- borg_exit_codes=None,
- )
- assert error.value.returncode == 2
- assert error.value.output
- def test_log_outputs_kills_other_processes_and_returns_when_one_exits_with_warning():
- flexmock(module.logger).should_receive('log')
- flexmock(module).should_receive('command_for_process').and_return('grep')
- process = subprocess.Popen(['grep'], stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
- flexmock(module).should_receive('interpret_exit_code').with_args(
- ['grep'],
- None,
- 'borg',
- None,
- ).and_return(module.Exit_status.SUCCESS)
- flexmock(module).should_receive('interpret_exit_code').with_args(
- ['grep'],
- 2,
- 'borg',
- None,
- ).and_return(module.Exit_status.WARNING)
- other_process = subprocess.Popen(
- ['sleep', '2'], stdout=subprocess.PIPE, stderr=subprocess.STDOUT
- )
- flexmock(module).should_receive('interpret_exit_code').with_args(
- ['sleep', '2'],
- None,
- 'borg',
- None,
- ).and_return(module.Exit_status.SUCCESS)
- flexmock(module).should_receive('output_buffer_for_process').with_args(process, ()).and_return(
- process.stdout
- )
- flexmock(module).should_receive('output_buffer_for_process').with_args(
- other_process, ()
- ).and_return(other_process.stdout)
- flexmock(other_process).should_receive('kill').once()
- module.log_outputs(
- (process, other_process),
- exclude_stdouts=(),
- output_log_level=logging.INFO,
- borg_local_path='borg',
- borg_exit_codes=None,
- )
- def test_log_outputs_vents_other_processes_when_one_exits():
- '''
- Execute a command to generate a longish random string and pipe it into another command that
- exits quickly. The test is basically to ensure we don't hang forever waiting for the exited
- process to read the pipe, and that the string-generating process eventually gets vented and
- exits.
- '''
- flexmock(module.logger).should_receive('log')
- flexmock(module).should_receive('command_for_process').and_return('grep')
- process = subprocess.Popen(
- [
- sys.executable,
- '-c',
- "import random, string; print(''.join(random.choice(string.ascii_letters) for _ in range(40000)))",
- ],
- stdout=subprocess.PIPE,
- stderr=subprocess.PIPE,
- )
- other_process = subprocess.Popen(
- ['true'], stdin=process.stdout, stdout=subprocess.PIPE, stderr=subprocess.STDOUT
- )
- flexmock(module).should_receive('output_buffer_for_process').with_args(
- process, (process.stdout,)
- ).and_return(process.stderr)
- flexmock(module).should_receive('output_buffer_for_process').with_args(
- other_process, (process.stdout,)
- ).and_return(other_process.stdout)
- flexmock(process.stdout).should_call('readline').at_least().once()
- module.log_outputs(
- (process, other_process),
- exclude_stdouts=(process.stdout,),
- output_log_level=logging.INFO,
- borg_local_path='borg',
- borg_exit_codes=None,
- )
- def test_log_outputs_does_not_error_when_one_process_exits():
- flexmock(module.logger).should_receive('log')
- flexmock(module).should_receive('command_for_process').and_return('grep')
- process = subprocess.Popen(
- [
- sys.executable,
- '-c',
- "import random, string; print(''.join(random.choice(string.ascii_letters) for _ in range(40000)))",
- ],
- stdout=None, # Specifically test the case of a process without stdout captured.
- stderr=None,
- )
- other_process = subprocess.Popen(
- ['true'], stdin=process.stdout, stdout=subprocess.PIPE, stderr=subprocess.STDOUT
- )
- flexmock(module).should_receive('output_buffer_for_process').with_args(
- process, (process.stdout,)
- ).and_return(process.stderr)
- flexmock(module).should_receive('output_buffer_for_process').with_args(
- other_process, (process.stdout,)
- ).and_return(other_process.stdout)
- module.log_outputs(
- (process, other_process),
- exclude_stdouts=(process.stdout,),
- output_log_level=logging.INFO,
- borg_local_path='borg',
- borg_exit_codes=None,
- )
- def test_log_outputs_truncates_long_error_output():
- flexmock(module.logger).should_receive('log')
- flexmock(module).should_receive('command_for_process').and_return('grep')
- process = subprocess.Popen(['grep'], stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
- flexmock(module).should_receive('interpret_exit_code').with_args(
- ['grep'],
- None,
- 'borg',
- None,
- ).and_return(module.Exit_status.SUCCESS)
- flexmock(module).should_receive('interpret_exit_code').with_args(
- ['grep'],
- 2,
- 'borg',
- None,
- ).and_return(module.Exit_status.ERROR)
- flexmock(module).should_receive('output_buffer_for_process').and_return(process.stdout)
- with pytest.raises(subprocess.CalledProcessError) as error:
- flexmock(module, ERROR_OUTPUT_MAX_LINE_COUNT=0).log_outputs(
- (process,),
- exclude_stdouts=(),
- output_log_level=logging.INFO,
- borg_local_path='borg',
- borg_exit_codes=None,
- )
- assert error.value.returncode == 2
- assert error.value.output.startswith('...')
- def test_log_outputs_with_no_output_logs_nothing():
- flexmock(module.logger).should_receive('log').never()
- flexmock(module).should_receive('interpret_exit_code').and_return(module.Exit_status.SUCCESS)
- process = subprocess.Popen(['true'], stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
- flexmock(module).should_receive('output_buffer_for_process').and_return(process.stdout)
- module.log_outputs(
- (process,),
- exclude_stdouts=(),
- output_log_level=logging.INFO,
- borg_local_path='borg',
- borg_exit_codes=None,
- )
- def test_log_outputs_with_unfinished_process_re_polls():
- flexmock(module.logger).should_receive('log').never()
- flexmock(module).should_receive('interpret_exit_code').and_return(module.Exit_status.SUCCESS)
- process = subprocess.Popen(['true'], stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
- flexmock(process).should_receive('poll').and_return(None).and_return(0).times(3)
- flexmock(module).should_receive('output_buffer_for_process').and_return(process.stdout)
- module.log_outputs(
- (process,),
- exclude_stdouts=(),
- output_log_level=logging.INFO,
- borg_local_path='borg',
- borg_exit_codes=None,
- )
|