diff --git a/app/main/views/jobs.py b/app/main/views/jobs.py index 390eb7325..4768275c1 100644 --- a/app/main/views/jobs.py +++ b/app/main/views/jobs.py @@ -246,7 +246,10 @@ def get_notifications_as_json(service_id, message_type=None): @main.route('/services//notifications.csv', endpoint="view_notifications_csv") -@main.route('/services//notifications/.csv', endpoint="view_notifications_csv") +@main.route( + '/services//notifications/.csv', + endpoint="view_notifications_csv" +) @user_has_permissions() def get_notifications(service_id, message_type, status_override=None): # TODO get the api to return count of pages as well. diff --git a/app/main/views/send.py b/app/main/views/send.py index ea55a50eb..a0a4b5052 100644 --- a/app/main/views/send.py +++ b/app/main/views/send.py @@ -601,7 +601,10 @@ def _check_messages(service_id, template_id, upload_id, preview_row, letters_as_ @main.route("/services///check/", methods=['GET']) -@main.route("/services///check//row-", methods=['GET']) +@main.route( + "/services///check//row-", + methods=['GET'] +) @user_has_permissions('send_messages', restrict_admin_usage=True) def check_messages(service_id, template_id, upload_id, row_index=2): diff --git a/app/main/views/service_settings.py b/app/main/views/service_settings.py index 7a845cbf4..8596a431c 100644 --- a/app/main/views/service_settings.py +++ b/app/main/views/service_settings.py @@ -425,7 +425,10 @@ def service_add_email_reply_to(service_id): first_email_address=first_email_address) -@main.route("/services//service-settings/email-reply-to//verify", methods=['GET', 'POST']) +@main.route( + "/services//service-settings/email-reply-to//verify", + methods=['GET', 'POST'] +) @user_has_permissions('manage_service') def service_verify_reply_to_address(service_id, notification_id): replace = request.args.get('replace', False) diff --git a/app/main/views/templates.py b/app/main/views/templates.py index d0a953b28..0e5fbf4f0 100644 --- a/app/main/views/templates.py +++ b/app/main/views/templates.py @@ -1,6 +1,5 @@ from datetime import datetime, timedelta from functools import partial -from string import ascii_uppercase from dateutil.parser import parse from flask import abort, flash, redirect, render_template, request, url_for @@ -10,7 +9,6 @@ from notifications_python_client.errors import HTTPError from notifications_utils import LETTER_MAX_PAGE_COUNT from notifications_utils.formatters import nl2br from notifications_utils.pdf import is_letter_too_long -from notifications_utils.recipients import first_column_headings from app import ( current_service, @@ -29,7 +27,7 @@ from app.main.forms import ( TemplateAndFoldersSelectionForm, TemplateFolderForm, ) -from app.main.views.send import get_example_csv_rows, get_sender_details +from app.main.views.send import get_sender_details from app.models.service import Service from app.models.template_list import TemplateList, TemplateLists from app.template_previews import TemplatePreview, get_page_count_for_letter @@ -110,7 +108,10 @@ def start_tour(service_id, template_id): @main.route("/services//templates/folders/", methods=['GET', 'POST']) @main.route("/services//templates/", methods=['GET', 'POST']) @main.route("/services//templates/all/folders/", methods=['GET', 'POST']) -@main.route("/services//templates//folders/", methods=['GET', 'POST']) +@main.route( + "/services//templates//folders/", + methods=['GET', 'POST'] +) @user_has_permissions() def choose_template(service_id, template_type='all', template_folder_id=None): template_folder = current_service.get_template_folder(template_folder_id) @@ -329,7 +330,9 @@ def _add_template_by_type(template_type, template_folder_id): @main.route("/services//templates/copy") @main.route("/services//templates/copy/from-folder/") @main.route("/services//templates/copy/from-service/") -@main.route("/services//templates/copy/from-service//from-folder/") +@main.route( + "/services//templates/copy/from-service//from-folder/" +) @user_has_permissions('manage_templates') def choose_template_to_copy( service_id, diff --git a/get_zendesk_tickets.py b/get_zendesk_tickets.py index 5f57b87a2..f6182eb57 100644 --- a/get_zendesk_tickets.py +++ b/get_zendesk_tickets.py @@ -2,6 +2,8 @@ This script can be used to retrieve Zendesk tickets. This can be run locally if you set the ZENDESK_API_KEY. Or the script can be run from a flask shell from a ssh session. """ +# flake8: noqa: T001 (print) + import os import requests import urllib.parse @@ -137,4 +139,3 @@ def get_tickets_with_description(): 'Tags': row.get('tags', '') }) next_page = data["next_page"] - diff --git a/tests/app/main/views/test_platform_admin.py b/tests/app/main/views/test_platform_admin.py index 352b97d12..cd22eb384 100644 --- a/tests/app/main/views/test_platform_admin.py +++ b/tests/app/main/views/test_platform_admin.py @@ -6,9 +6,8 @@ from unittest.mock import ANY, call import pyexcel import pytest -import requests_mock from bs4 import BeautifulSoup -from flask import current_app, url_for +from flask import url_for from freezegun import freeze_time from app.main.views.platform_admin import ( diff --git a/tests/app/main/views/test_uploads.py b/tests/app/main/views/test_uploads.py index cf7b6a7d3..a08e30821 100644 --- a/tests/app/main/views/test_uploads.py +++ b/tests/app/main/views/test_uploads.py @@ -68,7 +68,11 @@ def test_post_upload_letter_redirects_for_valid_file( ) mock_s3 = mocker.patch('app.main.views.uploads.upload_letter_to_s3') mocker.patch('app.main.views.uploads.get_letter_metadata', return_value={ - 'filename': 'tests/test_pdf_files/one_page_pdf.pdf', 'page_count': '1', 'status': 'valid', 'recipient': 'The Queen'}) + 'filename': 'tests/test_pdf_files/one_page_pdf.pdf', + 'page_count': '1', + 'status': 'valid', + 'recipient': 'The Queen' + }) mocker.patch('app.main.views.uploads.service_api_client.get_precompiled_template') service_one['restricted'] = False @@ -125,7 +129,11 @@ def test_post_upload_letter_shows_letter_preview_for_valid_file( mocker.patch('app.main.views.uploads.upload_letter_to_s3') mocker.patch('app.main.views.uploads.pdf_page_count', return_value=3) mocker.patch('app.main.views.uploads.get_letter_metadata', return_value={ - 'filename': 'tests/test_pdf_files/one_page_pdf.pdf', 'page_count': '3', 'status': 'valid', 'recipient': 'The Queen'}) + 'filename': 'tests/test_pdf_files/one_page_pdf.pdf', + 'page_count': '3', + 'status': 'valid', + 'recipient': 'The Queen' + }) mocker.patch('app.main.views.uploads.service_api_client.get_precompiled_template', return_value=letter_template) service_one['restricted'] = False