@@ -232,8 +232,9 @@ def test_new_template_cloned(mock_error, mock_subprocess,
232232
233233
234234@patch ('subprocess.check_output' , side_effect = mock_output )
235+ @patch ('subprocess.check_call' )
235236@patch ('qubes_config.new_qube.new_qube_app.show_error' )
236- def test_new_standalone (mock_error , mock_subprocess ,
237+ def test_new_standalone (mock_error , mock_check_call , mock_subprocess ,
237238 test_qapp , new_qube_builder ):
238239 # the builder fixture must be called to register needed signals and
239240 # only do it once
@@ -284,7 +285,7 @@ def test_new_standalone(mock_error, mock_subprocess,
284285 assert mock_dialog .mock_calls # called to tell us about the success
285286
286287 assert call (['qubes-vm-boot-from-device' , 'test' ]) \
287- in mock_popen .mock_calls # called install system to qube
288+ in mock_check_call .mock_calls # called install system to qube
288289
289290 # but no apps were added
290291 assert call (['qvm-appmenus' , '--set-whitelist' , '-' ,
@@ -359,8 +360,9 @@ def test_new_disposable(mock_error, mock_subprocess,
359360
360361
361362@patch ('subprocess.check_output' , side_effect = mock_output )
363+ @patch ('subprocess.check_call' )
362364@patch ('qubes_config.new_qube.new_qube_app.show_error' )
363- def test_advanced_new_qube (mock_error , mock_subprocess ,
365+ def test_advanced_new_qube (mock_error , _mock_check_call , mock_subprocess ,
364366 test_qapp , new_qube_builder ):
365367 # the builder fixture must be called to register needed signals and
366368 # only do it once
0 commit comments