Skip to content

Commit 3d8d4a6

Browse files
committed
Refactor assertions to reusable function for affiliated institutions
1 parent d7c64c1 commit 3d8d4a6

File tree

2 files changed

+15
-14
lines changed

2 files changed

+15
-14
lines changed

pages/preprints.py

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -212,6 +212,13 @@ def select_top_level_subject(self, selection):
212212
def get_preprint_institution_list_review(self) -> list:
213213
return [el.get_attribute('alt') for el in self.preprint_institution_list_review]
214214

215+
def assert_affiliated_institutions_equal(self, expected, actual, page_name):
216+
assert expected == actual, (
217+
f'Affiliated institutions on the {page_name} do not match expected values.\n'
218+
f'Expected: {expected}\n'
219+
f'Actual: {actual}'
220+
)
221+
215222
# Preprint Detail Page
216223
preprint_institution_list_detail = GroupLocator(
217224
By.CSS_SELECTOR, 'img[data-test-preprint-institution-list]'

tests/test_preprints.py

Lines changed: 8 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -211,13 +211,10 @@ def test_create_preprint_from_landing(
211211
affiliated_institutions_names_review_page = (
212212
submit_page.get_preprint_institution_list_review()
213213
)
214-
assert (
215-
affiliated_institutions_names_metadata_page
216-
== affiliated_institutions_names_review_page
217-
), (
218-
f'Affiliated institutions on the Review Page do not match expected values.\n'
219-
f'Expected: {affiliated_institutions_names_metadata_page }\n'
220-
f'Actual: {affiliated_institutions_names_review_page}'
214+
submit_page.assert_affiliated_institutions_equal(
215+
affiliated_institutions_names_metadata_page,
216+
affiliated_institutions_names_review_page,
217+
'Preprint Review Page',
221218
)
222219
submit_page.info_toast.here_then_gone()
223220
submit_page.create_preprint_button.click()
@@ -227,13 +224,10 @@ def test_create_preprint_from_landing(
227224
affiliated_institutions_names_detail_page = (
228225
submit_page.get_preprint_institution_list_detail()
229226
)
230-
assert (
231-
affiliated_institutions_names_metadata_page
232-
== affiliated_institutions_names_detail_page
233-
), (
234-
f'Affiliated institutions on the Preprint Detail Page do not match expected values.\n'
235-
f'Expected: {affiliated_institutions_names_metadata_page }\n'
236-
f'Actual: {affiliated_institutions_names_detail_page}'
227+
submit_page.assert_affiliated_institutions_equal(
228+
affiliated_institutions_names_metadata_page,
229+
affiliated_institutions_names_detail_page,
230+
'Preprint Detail Page',
237231
)
238232
# Capture guid of supplemental materials project created during workflow
239233
supplemental_url = preprint_detail.view_page.get_attribute('href')

0 commit comments

Comments
 (0)