Skip to content

Commit 6bcb2b5

Browse files
committed
Does 'stderr' need to be set if that branch of logic is not tested here?
1 parent 87e91f6 commit 6bcb2b5

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

tests/instrument_server/test_api.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -94,9 +94,7 @@ def test_upload_gain_reference(
9494
mock_response.json.return_value = mock_machine_config
9595
mock_request.get.return_value = mock_response
9696
mock_get_server_url.return_value = server_url
97-
mock_subprocess.run.return_value = Mock(
98-
returncode=0, stderr="An error has occurred."
99-
)
97+
mock_subprocess.run.return_value = Mock(returncode=0)
10098

10199
# Construct payload and pass request to function
102100
gain_ref_file = f"{gain_ref_dir}/gain.mrc"

0 commit comments

Comments
 (0)