Skip to content

Commit a03f609

Browse files
committed
Fix test overwriting a test configuration file
1 parent bd21957 commit a03f609

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

tests/test_file_operations.py

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -545,11 +545,12 @@ def test_configure_op(test_dir, fileutils, param_dict, error_type):
545545
assert filecmp.cmp(written, correct)
546546

547547

548-
def test_configure_invalid_tags(fileutils):
548+
def test_configure_invalid_tags(fileutils, test_dir):
549549
"""Test configure operation with an invalid tag"""
550-
tagged_file = fileutils.get_test_conf_path(
551-
osp.join("generator_files", "easy", "marked", "invalidtag.txt")
552-
)
550+
generator_files = pathlib.Path(fileutils.get_test_conf_path("generator_files"))
551+
tagged_file = generator_files / "easy/marked/invalidtag.txt"
552+
correct_file = generator_files / "easy/correct/invalidtag.txt"
553+
target_file = pathlib.Path(test_dir, "target.txt")
553554

554555
tag = ";"
555556
param_dict = {"VALID": "valid"}
@@ -560,11 +561,12 @@ def test_configure_invalid_tags(fileutils):
560561
# Encode the pickled dictionary with Base64
561562
encoded_dict = base64.b64encode(pickled_dict).decode("ascii")
562563
parser = get_parser()
563-
cmd = f"configure {tagged_file} {tagged_file} {tag} {encoded_dict}"
564+
cmd = f"configure {tagged_file} {target_file} {tag} {encoded_dict}"
564565
args = cmd.split()
565566
ns = parser.parse_args(args)
566567

567568
file_operations.configure(ns)
569+
assert filecmp.cmp(correct_file, target_file)
568570

569571

570572
def test_configure_not_absolute():

0 commit comments

Comments
 (0)