Skip to content

Commit 2e37244

Browse files
committed
Addresses review comments
1 parent ef6f0c3 commit 2e37244

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

RATapi/inputs.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,7 @@ def make_problem(project: RATapi.Project) -> ProblemDefinition:
163163
for field in contrast_fields:
164164
if getattr(contrast, field) == "":
165165
raise ValueError(
166-
f'In the input project, the {field} of contrast "{contrast.name}" does not have a '
166+
f'In the input project, the "{field}" field of contrast "{contrast.name}" does not have a '
167167
f"value defined. A value must be supplied before running the project."
168168
)
169169

@@ -399,7 +399,7 @@ def get_layer_details(project: RATapi.Project) -> list[int]:
399399
for field in layer_fields:
400400
if getattr(layer, field) == "":
401401
raise ValueError(
402-
f"In the input project, the {field} field of layer {layer.name} does not have a value "
402+
f'In the input project, the "{field}" field of layer {layer.name} does not have a value '
403403
f"defined. A value must be supplied before running the project."
404404
)
405405

tests/test_inputs.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -581,7 +581,7 @@ def test_undefined_contrast_fields(test_project, field, request):
581581

582582
with pytest.raises(
583583
ValueError,
584-
match=f"In the input project, the {field} of contrast "
584+
match=f'In the input project, the "{field}" field of contrast '
585585
f'"{test_project.contrasts[0].name}" does not have a value defined. '
586586
f"A value must be supplied before running the project.",
587587
):
@@ -629,7 +629,7 @@ def test_undefined_layers(test_project, field, request):
629629

630630
with pytest.raises(
631631
ValueError,
632-
match=f"In the input project, the {field} field of layer {test_project.layers[0].name} "
632+
match=f'In the input project, the "{field}" field of layer {test_project.layers[0].name} '
633633
f"does not have a value defined. A value must be supplied before running the project.",
634634
):
635635
make_problem(test_project)

0 commit comments

Comments
 (0)