mirror of
https://github.com/GSA/notifications-admin.git
synced 2026-08-18 21:49:37 -04:00
fix test get_immediate_jobs for download availability check
This commit is contained in:
@@ -50,6 +50,9 @@ def test_all_activity(
|
|||||||
mock_get_page_of_jobs = mocker.patch(
|
mock_get_page_of_jobs = mocker.patch(
|
||||||
"app.job_api_client.get_page_of_jobs", return_value=MOCK_JOBS
|
"app.job_api_client.get_page_of_jobs", return_value=MOCK_JOBS
|
||||||
)
|
)
|
||||||
|
mocker.patch(
|
||||||
|
"app.job_api_client.get_immediate_jobs", return_value=[]
|
||||||
|
)
|
||||||
|
|
||||||
response = client_request.get_response(
|
response = client_request.get_response(
|
||||||
"main.all_jobs_activity",
|
"main.all_jobs_activity",
|
||||||
@@ -60,7 +63,11 @@ def test_all_activity(
|
|||||||
assert response.data is not None, "Response data is None"
|
assert response.data is not None, "Response data is None"
|
||||||
|
|
||||||
assert "All activity" in response.text
|
assert "All activity" in response.text
|
||||||
mock_get_page_of_jobs.assert_called_with(SERVICE_ONE_ID, page=current_page)
|
|
||||||
|
assert any(
|
||||||
|
call[0][0] == SERVICE_ONE_ID and call[1].get('page') == current_page
|
||||||
|
for call in mock_get_page_of_jobs.call_args_list
|
||||||
|
)
|
||||||
page = BeautifulSoup(response.data, "html.parser")
|
page = BeautifulSoup(response.data, "html.parser")
|
||||||
table = page.find("table")
|
table = page.find("table")
|
||||||
assert table is not None, "Table not found in the response"
|
assert table is not None, "Table not found in the response"
|
||||||
@@ -115,7 +122,6 @@ def test_all_activity(
|
|||||||
|
|
||||||
failed_cell = cells[6].get_text(strip=True)
|
failed_cell = cells[6].get_text(strip=True)
|
||||||
assert failed_cell == "5", f"Expected failed count '5', but got '{failed_cell}'"
|
assert failed_cell == "5", f"Expected failed count '5', but got '{failed_cell}'"
|
||||||
mock_get_page_of_jobs.assert_called_with(SERVICE_ONE_ID, page=current_page)
|
|
||||||
|
|
||||||
|
|
||||||
def test_all_activity_no_jobs(client_request, mocker):
|
def test_all_activity_no_jobs(client_request, mocker):
|
||||||
@@ -133,6 +139,9 @@ def test_all_activity_no_jobs(client_request, mocker):
|
|||||||
"total": 0,
|
"total": 0,
|
||||||
},
|
},
|
||||||
)
|
)
|
||||||
|
mocker.patch(
|
||||||
|
"app.job_api_client.get_immediate_jobs", return_value=[]
|
||||||
|
)
|
||||||
response = client_request.get_response(
|
response = client_request.get_response(
|
||||||
"main.all_jobs_activity",
|
"main.all_jobs_activity",
|
||||||
service_id=SERVICE_ONE_ID,
|
service_id=SERVICE_ONE_ID,
|
||||||
@@ -152,7 +161,10 @@ def test_all_activity_no_jobs(client_request, mocker):
|
|||||||
assert (
|
assert (
|
||||||
expected_message == actual_message
|
expected_message == actual_message
|
||||||
), f"Expected message '{expected_message}', but got '{actual_message}'"
|
), f"Expected message '{expected_message}', but got '{actual_message}'"
|
||||||
mock_get_page_of_jobs.assert_called_with(SERVICE_ONE_ID, page=current_page)
|
assert any(
|
||||||
|
call[0][0] == SERVICE_ONE_ID and call[1].get('page') == current_page
|
||||||
|
for call in mock_get_page_of_jobs.call_args_list
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
def test_all_activity_pagination(client_request, mocker):
|
def test_all_activity_pagination(client_request, mocker):
|
||||||
@@ -182,13 +194,19 @@ def test_all_activity_pagination(client_request, mocker):
|
|||||||
"total": 100,
|
"total": 100,
|
||||||
},
|
},
|
||||||
)
|
)
|
||||||
|
mocker.patch(
|
||||||
|
"app.job_api_client.get_immediate_jobs", return_value=[]
|
||||||
|
)
|
||||||
|
|
||||||
response = client_request.get_response(
|
response = client_request.get_response(
|
||||||
"main.all_jobs_activity",
|
"main.all_jobs_activity",
|
||||||
service_id=SERVICE_ONE_ID,
|
service_id=SERVICE_ONE_ID,
|
||||||
page=current_page,
|
page=current_page,
|
||||||
)
|
)
|
||||||
mock_get_page_of_jobs.assert_called_with(SERVICE_ONE_ID, page=current_page)
|
assert any(
|
||||||
|
call[0][0] == SERVICE_ONE_ID and call[1].get('page') == current_page
|
||||||
|
for call in mock_get_page_of_jobs.call_args_list
|
||||||
|
)
|
||||||
|
|
||||||
page = BeautifulSoup(response.data, "html.parser")
|
page = BeautifulSoup(response.data, "html.parser")
|
||||||
pagination_controls = page.find_all("li", class_="usa-pagination__item")
|
pagination_controls = page.find_all("li", class_="usa-pagination__item")
|
||||||
|
|||||||
Reference in New Issue
Block a user