Skip to content

Commit 342f060

Browse files
author
Gaurav Singh
committed
Extracted search for users as a function
1 parent ec6c3cd commit 342f060

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

tests/people_test.py

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ def test_new_person_can_be_added():
2525
# After user is created, we read all the users and then use filter expression to find if the
2626
# created user is present in the response list
2727
peoples = requests.get(BASE_URI).json()
28-
is_new_user_created = [person for person in peoples if person['lname'] == unique_last_name]
28+
is_new_user_created = search_created_user_in(peoples, unique_last_name)
2929
assert_that(is_new_user_created).is_not_empty()
3030

3131

@@ -35,7 +35,7 @@ def test_created_person_can_be_deleted():
3535
raise AssertionError('User not created')
3636

3737
peoples = requests.get(BASE_URI).json()
38-
newly_created_user = [person for person in peoples if person['lname'] == persons_last_name][0]
38+
newly_created_user = search_created_user_in(peoples, persons_last_name)[0]
3939

4040
delete_url = f'{BASE_URI}/{newly_created_user["person_id"]}'
4141
response = requests.delete(delete_url)
@@ -60,5 +60,9 @@ def create_new_person():
6060

6161
# We use requests.post method with keyword params to make the request more readable
6262
response = requests.post(url=BASE_URI, data=payload, headers=headers)
63-
assert_that(response.status_code).is_equal_to(requests.codes.no_content)
63+
assert_that(response.status_code, description='Person not created').is_equal_to(requests.codes.no_content)
6464
return unique_last_name
65+
66+
67+
def search_created_user_in(peoples, last_name):
68+
return [person for person in peoples if person['lname'] == last_name]

0 commit comments

Comments
 (0)