Skip to content

Commit b998a02

Browse files
committed
additional test suite stylistic changes
- DRYed up fixture prefixes - normalized variable names
1 parent a7920a3 commit b998a02

File tree

1 file changed

+7
-8
lines changed

1 file changed

+7
-8
lines changed

tests/test_springshare.py

+7-8
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,10 @@
44

55
SPRINGSHARE_FIXTURES_PREFIX = "tests/fixtures/oai_dc/springshare"
66

7-
LIBGUIDES_FIXTURES_PREFIX = "tests/fixtures/oai_dc/springshare/libguides"
7+
LIBGUIDES_FIXTURES_PREFIX = f"{SPRINGSHARE_FIXTURES_PREFIX}/libguides"
8+
9+
RESEARCHDATABASES_FIXTURES_PREFIX = f"{SPRINGSHARE_FIXTURES_PREFIX}/research_databases"
810

9-
RESEARCHDATABASES_FIXTURES_PREFIX = (
10-
"tests/fixtures/oai_dc/springshare/research_databases"
11-
)
1211

1312
LIBGUIDES_BLANK_OR_MISSING_OPTIONAL_FIELDS_TIMDEX = timdex.TimdexRecord(
1413
source="LibGuides",
@@ -59,8 +58,8 @@ def test_springshare_get_dates_valid():
5958
)
6059
transformer_instance = SpringshareOaiDc("libguides", input_records)
6160
for xml in transformer_instance.input_records:
62-
date_field_val = transformer_instance.get_dates("test_get_dates", xml)
63-
assert date_field_val == [
61+
date_field_value = transformer_instance.get_dates("test_get_dates", xml)
62+
assert date_field_value == [
6463
timdex.Date(kind=None, note=None, range=None, value="2000-01-01T00:00:00")
6564
]
6665

@@ -82,8 +81,8 @@ def test_springshare_get_links_missing_identifier_logged_and_skipped(caplog):
8281
)
8382
transformer_instance = SpringshareOaiDc("libguides", input_records)
8483
for xml in transformer_instance.input_records:
85-
links_field_val = transformer_instance.get_links("test_get_links", xml)
86-
assert links_field_val is None
84+
links_field_value = transformer_instance.get_links("test_get_links", xml)
85+
assert links_field_value is None
8786
assert "has links that cannot be generated" in caplog.text
8887

8988

0 commit comments

Comments
 (0)