From 619bf2e80dd6c225a4121507f0b9374598b94ea6 Mon Sep 17 00:00:00 2001 From: Frederick Muriuki Muriithi Date: Mon, 8 Apr 2024 10:32:10 +0300 Subject: Fix issues caught by integration tests. --- qc_app/parse.py | 14 +++++++------- qc_app/templates/index.html | 2 +- tests/conftest.py | 8 ++++---- tests/qc_app/test_parse.py | 19 +++++++------------ 4 files changed, 19 insertions(+), 24 deletions(-) diff --git a/qc_app/parse.py b/qc_app/parse.py index ba2e63e..d20f6f0 100644 --- a/qc_app/parse.py +++ b/qc_app/parse.py @@ -30,7 +30,7 @@ def parse(): filename = request.args.get("filename") filetype = request.args.get("filetype") if speciesid is None: - flash("No species selected", "alert-error alert-expr-data-error") + flash("No species selected", "alert-error error-expr-data") errors = True else: try: @@ -38,30 +38,30 @@ def parse(): species = with_db_connection( lambda con: species_by_id(con, speciesid)) if not bool(species): - flash("No such species.", "alert-error alert-expr-data-error") + flash("No such species.", "alert-error error-expr-data") errors = True except ValueError: flash("Invalid speciesid provided. Expected an integer.", - "alert-error alert-expr-data-error") + "alert-error error-expr-data") errors = True if filename is None: - flash("No file provided", "alert-error alert-expr-data-error") + flash("No file provided", "alert-error error-expr-data") errors = True if filetype is None: - flash("No filetype provided", "alert-error alert-expr-data-error") + flash("No filetype provided", "alert-error error-expr-data") errors = True if filetype not in ("average", "standard-error"): - flash("Invalid filetype provided", "alert-error alert-expr-data-error") + flash("Invalid filetype provided", "alert-error error-expr-data") errors = True if filename: filepath = os.path.join(app.config["UPLOAD_FOLDER"], filename) if not os.path.exists(filepath): flash("Selected file does not exist (any longer)", - "alert-error alert-expr-data-error") + "alert-error error-expr-data") errors = True if errors: diff --git a/qc_app/templates/index.html b/qc_app/templates/index.html index b87a67f..588133a 100644 --- a/qc_app/templates/index.html +++ b/qc_app/templates/index.html @@ -59,7 +59,7 @@ enctype="multipart/form-data" id="frm-upload-expression-data"> upload expression data - {{flash_messages("error-expr-data-error")}} + {{flash_messages("error-expr-data")}}
diff --git a/tests/conftest.py b/tests/conftest.py index e4bebc6..a39acf0 100644 --- a/tests/conftest.py +++ b/tests/conftest.py @@ -135,8 +135,8 @@ def redis_conn_with_in_progress_job_some_errors(redis_url, redis_ttl, jobs_prefi "status": "Processing", "filename": "/path/to/some/file.tsv", "percent": 45.34245, "errors": jsonpickle.encode(( DuplicateHeading( - 1, (5,13,19), "DupHead", "Heading 'DupHead' is repeated"), - InvalidValue(45, 2, "ohMy", "Invalid value 'ohMy'")))} + "__test__", 1, (5,13,19), "DupHead", "Heading 'DupHead' is repeated"), + InvalidValue("__test__", 45, 2, "ohMy", "Invalid value 'ohMy'")))} } with redis.Redis.from_url(redis_url, decode_responses=True) as rconn: jobs.initialise_job(rconn, **the_job) @@ -167,8 +167,8 @@ def redis_conn_with_completed_job_some_errors(redis_url, redis_ttl, jobs_prefix, "status": "success", "filename": "/path/to/some/file.tsv", "percent": 100, "errors": jsonpickle.encode(( DuplicateHeading( - 1, (5,13,19), "DupHead", "Heading 'DupHead' is repeated"), - InvalidValue(45, 2, "ohMy", "Invalid value 'ohMy'")))}} + "__test__", 1, (5,13,19), "DupHead", "Heading 'DupHead' is repeated"), + InvalidValue("__test__", 45, 2, "ohMy", "Invalid value 'ohMy'")))}} with redis.Redis.from_url(redis_url, decode_responses=True) as rconn: jobs.initialise_job(rconn, **the_job) yield rconn diff --git a/tests/qc_app/test_parse.py b/tests/qc_app/test_parse.py index 5e55688..187af76 100644 --- a/tests/qc_app/test_parse.py +++ b/tests/qc_app/test_parse.py @@ -55,26 +55,21 @@ def test_parse_with_existing_uploaded_file(#pylint: disable=[too-many-arguments] "filename,uri,error_msgs", (("non_existent.file", "/parse/parse?filename=non_existent.file&filename=average", - [(b'Selected file does not exist ' - b'(any longer)')]), + [b'Selected file does not exist (any longer)']), ("non_existent.file", "/parse/parse?filename=non_existent.file&filename=standard-error", - [(b'Selected file does not exist ' - b'(any longer)')]), + [b'Selected file does not exist (any longer)']), ("non_existent.file", "/parse/parse?filename=non_existent.file&filename=percival", - [(b'Selected file does not exist ' - b'(any longer)'), - b'Invalid filetype provided']), + [b'Selected file does not exist (any longer)', + b'Invalid filetype provided']), ("no_data_errors.tsv", "/parse/parse?filename=no_data_errors.tsv&filename=percival", - [b'Invalid filetype provided']), + [b'Invalid filetype provided']), ("no_data_errors.tsv", "/parse/parse?filename=no_data_errors.tsv", - [b'No filetype provided']), - (None, "/parse/parse", - [b'No file provided', - b'No filetype provided']))) + [b'No filetype provided']), + (None, "/parse/parse", [b'No file provided', b'No filetype provided']))) def test_parse_with_non_uploaded_file(client, filename, uri, error_msgs): """ GIVEN: 1. A flask application testing client -- cgit v1.2.3