From e4f0656a321b7c83a7caa53e7cf46577afd980c7 Mon Sep 17 00:00:00 2001 From: Leo Hemsted Date: Tue, 11 Oct 2016 14:50:06 +0100 Subject: [PATCH] remove filtering of test messages from jobs this is now down on the api side --- app/main/views/dashboard.py | 1 - app/main/views/jobs.py | 1 - tests/app/main/views/test_dashboard.py | 1 - tests/app/main/views/test_jobs.py | 1 - tests/conftest.py | 1 - 5 files changed, 5 deletions(-) diff --git a/app/main/views/dashboard.py b/app/main/views/dashboard.py index 3f966f87a..54407681f 100644 --- a/app/main/views/dashboard.py +++ b/app/main/views/dashboard.py @@ -129,7 +129,6 @@ def get_dashboard_partials(service_id): immediate_jobs = [ add_rate_to_job(job) for job in job_api_client.get_jobs(service_id, limit_days=7, statuses=statuses_to_display)['data'] - if job['original_file_name'] != current_app.config['TEST_MESSAGE_FILENAME'] ] service = service_api_client.get_detailed_service(service_id) diff --git a/app/main/views/jobs.py b/app/main/views/jobs.py index 9cdf91824..a7fd87154 100644 --- a/app/main/views/jobs.py +++ b/app/main/views/jobs.py @@ -71,7 +71,6 @@ def view_jobs(service_id): jobs_response = job_api_client.get_jobs(service_id, statuses=statuses_to_display, page=page) jobs = [ add_rate_to_job(job) for job in jobs_response['data'] - if job['original_file_name'] != current_app.config['TEST_MESSAGE_FILENAME'] ] prev_page = None diff --git a/tests/app/main/views/test_dashboard.py b/tests/app/main/views/test_dashboard.py index 83eb14148..759777938 100644 --- a/tests/app/main/views/test_dashboard.py +++ b/tests/app/main/views/test_dashboard.py @@ -231,7 +231,6 @@ def test_should_show_recent_jobs_on_dashboard( page = BeautifulSoup(response.data.decode('utf-8'), 'html.parser') table_rows = page.find_all('tbody')[2].find_all('tr') - assert "Test message" not in page.text assert len(table_rows) == 4 for index, filename in enumerate(( diff --git a/tests/app/main/views/test_jobs.py b/tests/app/main/views/test_jobs.py index 0f6d2146e..96d40c7de 100644 --- a/tests/app/main/views/test_jobs.py +++ b/tests/app/main/views/test_jobs.py @@ -27,7 +27,6 @@ def test_get_jobs_should_return_list_of_all_real_jobs( assert page.h1.string == 'Uploaded files' jobs = [x.text for x in page.tbody.find_all('a', {'class': 'file-list-filename'})] assert len(jobs) == 4 - assert 'Test message' not in jobs def test_get_jobs_shows_page_links( diff --git a/tests/conftest.py b/tests/conftest.py index d43e7d2b2..fe381eebe 100644 --- a/tests/conftest.py +++ b/tests/conftest.py @@ -914,7 +914,6 @@ def mock_get_jobs(mocker, api_user_active): job_status=job_status ) for filename, scheduled_for, job_status in ( - ('Test message', '', 'finished'), ('export 1/1/2016.xls', '', 'finished'), ('all email addresses.xlsx', '', 'pending'), ('applicants.ods', '', 'finished'),