Skip to content

Commit 5a93afd

Browse files
Apply suggestions from code review
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
1 parent 7fb7297 commit 5a93afd

File tree

1 file changed

+9
-4
lines changed

1 file changed

+9
-4
lines changed

scripts/r.pack/tests/r_pack_test.py

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,9 @@ def test_output_exists_without_overwrite(
4545
assert not output.exists()
4646
tools.r_pack(input="rows_raster", output=output)
4747
assert output.exists()
48-
with pytest.raises(CalledModuleError, match=rf"(?s){re.escape(str(output))}.*[Oo]verwrite"):
48+
with pytest.raises(
49+
CalledModuleError, match=rf"(?s){re.escape(str(output))}.*[Oo]verwrite"
50+
):
4951
# Same call repeated again.
5052
tools.r_pack(input="rows_raster", output=output)
5153
assert output.exists()
@@ -70,7 +72,8 @@ def test_output_dir_does_not_exist(xy_raster_dataset_session_for_module, tmp_pat
7072
assert not output.exists()
7173
assert not output.parent.exists()
7274
with pytest.raises(
73-
CalledModuleError, match=rf"(?s)'{re.escape(str(output.parent))}'.*does not exist"
75+
CalledModuleError,
76+
match=rf"(?s)'{re.escape(str(output.parent))}'.*does not exist",
7477
):
7578
tools.r_pack(input="rows_raster", output=output)
7679

@@ -85,7 +88,8 @@ def test_output_dir_is_file(xy_raster_dataset_session_for_module, tmp_path):
8588
assert output.parent.exists()
8689
assert output.parent.is_file()
8790
with pytest.raises(
88-
CalledModuleError, match=rf"(?s)'{re.escape(str(output.parent))}'.*is not.*directory"
91+
CalledModuleError,
92+
match=rf"(?s)'{re.escape(str(output.parent))}'.*is not.*directory",
8993
):
9094
tools.r_pack(input="rows_raster", output=output)
9195

@@ -101,7 +105,8 @@ def test_output_dir_is_not_writable(xy_raster_dataset_session_for_module, tmp_pa
101105
assert output.parent.exists()
102106
assert output.parent.is_dir()
103107
with pytest.raises(
104-
CalledModuleError, match=rf"(?s)'{re.escape(str(output.parent))}'.*is not.*writable"
108+
CalledModuleError,
109+
match=rf"(?s)'{re.escape(str(output.parent))}'.*is not.*writable",
105110
):
106111
tools.r_pack(input="rows_raster", output=output)
107112

0 commit comments

Comments
 (0)