Skip to content

Commit

Permalink
[pre-commit.ci] auto fixes from pre-commit.com hooks
Browse files Browse the repository at this point in the history
for more information, see https://pre-commit.ci
  • Loading branch information
pre-commit-ci[bot] committed Oct 14, 2024
1 parent 8deb5d0 commit a1b4eff
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/streaming/test_downloader.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ def test_s3_downloader_with_s5cmd_no_storage_options(popen_mock, system_mock, tm
downloader.download_file(remote_filepath, local_filepath)

# Assertion: Verify subprocess.Popen was called with correct arguments and no env variables
popen_mock.assert_called_once_with( # noqa: S604
popen_mock.assert_called_once_with(
f"s5cmd cp {remote_filepath} {local_filepath}",
shell=True,
stdout=subprocess.PIPE,
Expand Down Expand Up @@ -68,7 +68,7 @@ def test_s3_downloader_with_s5cmd_with_storage_options(popen_mock, system_mock,
expected_env.update(storage_options)

# Assertion: Verify subprocess.Popen was called with the correct arguments and environment variables
popen_mock.assert_called_once_with( # noqa: S604
popen_mock.assert_called_once_with(
f"s5cmd cp {remote_filepath} {local_filepath}",
shell=True,
stdout=subprocess.PIPE,
Expand Down

0 comments on commit a1b4eff

Please sign in to comment.