From 3bd3049fd403886a9653aa7c2fbd1639926420ea Mon Sep 17 00:00:00 2001 From: Frederick Muriuki Muriithi Date: Tue, 21 Jun 2022 12:28:02 +0300 Subject: Test the progress indication feature --- tests/qc_app/test_parse.py | 15 ++--- tests/qc_app/test_progress_indication.py | 109 +++++++++++++++++++++++++++++++ 2 files changed, 115 insertions(+), 9 deletions(-) create mode 100644 tests/qc_app/test_progress_indication.py (limited to 'tests/qc_app') diff --git a/tests/qc_app/test_parse.py b/tests/qc_app/test_parse.py index 46b45d9..a4bfdd5 100644 --- a/tests/qc_app/test_parse.py +++ b/tests/qc_app/test_parse.py @@ -6,11 +6,8 @@ import pytest from qc_app.jobs import job from tests.conftest import uploadable_file_object -def module_uuid4(): - "module patch for the `uuid.uuid4()` function" - return "934c55d8-396e-4959-90e1-2698e9205758" - -def test_parse_with_existing_uploaded_file(client, redis_url, monkeypatch): +def test_parse_with_existing_uploaded_file( + client, redis_url, job_id, monkeypatch): """ GIVEN: 1. A flask application testing client 2. A valid file, and filetype @@ -19,7 +16,7 @@ def test_parse_with_existing_uploaded_file(client, redis_url, monkeypatch): 1. the system redirects to the job/parse status page 2. the job is placed on redis for processing """ - monkeypatch.setattr("qc_app.jobs.uuid4", module_uuid4) + monkeypatch.setattr("qc_app.jobs.uuid4", lambda : job_id) # Upload a file filename = "no_data_errors.tsv" filetype = "average" @@ -34,14 +31,14 @@ def test_parse_with_existing_uploaded_file(client, redis_url, monkeypatch): assert b'/parse/status/934c55d8-396e-4959-90e1-2698e9205758' in resp.data with redis.Redis.from_url(redis_url, decode_responses=True) as rconn: - the_job = job(rconn, module_uuid4()) + the_job = job(rconn, job_id) - assert the_job["job_id"] == module_uuid4() + assert the_job["job_id"] == job_id assert the_job["filename"] == filename assert the_job["command"] == " ".join([ "python3", "-m", "scripts.worker", filetype, f"{client.application.config['UPLOAD_FOLDER']}/{filename}", redis_url, - module_uuid4()]) + job_id]) @pytest.mark.parametrize( "filename,uri,error_msgs", diff --git a/tests/qc_app/test_progress_indication.py b/tests/qc_app/test_progress_indication.py new file mode 100644 index 0000000..11a412e --- /dev/null +++ b/tests/qc_app/test_progress_indication.py @@ -0,0 +1,109 @@ +"Test that the progress indication works correctly" + +def test_with_non_existing_job(client, redis_conn_with_fresh_job): # pylint: disable=[unused-argument] + """ + GIVEN: 1. A flask application testing client + 2. A redis instance with a fresh, unstarted job + WHEN: The parsing progress page is loaded for a non existing job + THEN: Ensure that the page: + 1. Has a meta tag to redirect it to the index page after 5 seconds + 2. Has text indicating that the job does not exist + """ + job_id = "non-existent-job-id" + resp = client.get(f"/parse/status/{job_id}") + assert resp.status_code == 400 + assert ( + b"No job, with the id 'non-existent-job-id' was found!" + in resp.data) + assert b'' in resp.data + +def test_with_unstarted_job(client, job_id, redis_conn_with_fresh_job): # pylint: disable=[unused-argument] + """ + GIVEN: 1. A flask application testing client + 2. A redis instance with a fresh, unstarted job + WHEN: The parsing progress page is loaded + THEN: Ensure that the page: + 1. Has a meta tag to refresh it after 5 seconds + 2. Has a progress indicator with zero progress + """ + resp = client.get(f"/parse/status/{job_id}") + assert b'' in resp.data + assert ( + b'0.0') in resp.data + +def test_with_in_progress_no_error_job( + client, job_id, redis_conn_with_in_progress_job_no_errors): # pylint: disable=[unused-argument] + """ + GIVEN: 1. A flask application testing client + 2. A redis instance with a job in progress, with no errors found in + the file so far + WHEN: The parsing progress page is loaded + THEN: Ensure that the page: + 1. Has a meta tag to refresh it after 5 seconds + 2. Has a progress indicator with the percent of the file processed + indicated + """ + resp = client.get(f"/parse/status/{job_id}") + assert b'' in resp.data + assert ( + b'32.242342') in resp.data + assert ( + b'No errors found so far' + in resp.data) + assert b"' in resp.data + assert ( + b'45.34245') in resp.data + assert ( + b'

We have found the following errors so far

' + in resp.data) + assert b'table class="reports-table">' in resp.data + assert b'Duplicate Header' in resp.data + assert b'Invalid Value' in resp.data + +def test_with_completed_job_no_errors( + client, job_id, redis_conn_with_completed_job_no_errors): # pylint: disable=[unused-argument] + """ + GIVEN: 1. A flask application testing client + 2. A redis instance with a completed job, with no errors found in + the file so far + WHEN: The parsing progress page is loaded + THEN: Ensure that the response is a redirection to the results page + """ + resp = client.get(f"/parse/status/{job_id}") + assert resp.status_code == 302 + assert f"/parse/results/{job_id}".encode("utf8") in resp.data + +def test_with_completed_job_some_errors( + client, job_id, redis_conn_with_completed_job_no_errors): # pylint: disable=[unused-argument] + """ + GIVEN: 1. A flask application testing client + 2. A redis instance with a completed job, with some errors found in + the file so far + WHEN: The parsing progress page is loaded + THEN: Ensure that the response is a redirection to the results page + """ + resp = client.get(f"/parse/status/{job_id}") + assert resp.status_code == 302 + assert f"/parse/results/{job_id}".encode("utf8") in resp.data -- cgit v1.2.3