Basic reports page, returns 200 and is visible in PA sidebar

This commit is contained in:
Pea Tyczynska
2019-04-24 10:27:57 +01:00
parent 4b14ab748f
commit b643aa54ec
5 changed files with 40 additions and 0 deletions

View File

@@ -187,6 +187,15 @@ def platform_admin_services():
) )
@main.route("/platform-admin/reports")
@login_required
@user_is_platform_admin
def platform_admin_reports():
return render_template(
'views/platform-admin/reports.html'
)
@main.route("/platform-admin/complaints") @main.route("/platform-admin/complaints")
@login_required @login_required
@user_is_platform_admin @user_is_platform_admin

View File

@@ -89,6 +89,7 @@ class HeaderNavigation(Navigation):
'platform_admin', 'platform_admin',
'platform_admin_letter_validation_preview', 'platform_admin_letter_validation_preview',
'platform_admin_list_complaints', 'platform_admin_list_complaints',
'platform_admin_reports',
'platform_admin_returned_letters', 'platform_admin_returned_letters',
'suspend_service', 'suspend_service',
'trial_services', 'trial_services',
@@ -493,6 +494,7 @@ class MainNavigation(Navigation):
'platform_admin', 'platform_admin',
'platform_admin_letter_validation_preview', 'platform_admin_letter_validation_preview',
'platform_admin_list_complaints', 'platform_admin_list_complaints',
'platform_admin_reports',
'platform_admin_returned_letters', 'platform_admin_returned_letters',
'pricing', 'pricing',
'privacy', 'privacy',
@@ -706,6 +708,7 @@ class CaseworkNavigation(Navigation):
'organisations', 'organisations',
'platform_admin_letter_validation_preview', 'platform_admin_letter_validation_preview',
'platform_admin_list_complaints', 'platform_admin_list_complaints',
'platform_admin_reports',
'platform_admin_returned_letters', 'platform_admin_returned_letters',
'platform_admin', 'platform_admin',
'pricing', 'pricing',
@@ -960,6 +963,7 @@ class OrgNavigation(Navigation):
'platform_admin', 'platform_admin',
'platform_admin_letter_validation_preview', 'platform_admin_letter_validation_preview',
'platform_admin_list_complaints', 'platform_admin_list_complaints',
'platform_admin_reports',
'platform_admin_returned_letters', 'platform_admin_returned_letters',
'pricing', 'pricing',
'privacy', 'privacy',

View File

@@ -17,6 +17,7 @@
('Trial mode services', url_for('main.trial_services')), ('Trial mode services', url_for('main.trial_services')),
('Organisations', url_for('main.organisations')), ('Organisations', url_for('main.organisations')),
('Providers', url_for('main.view_providers')), ('Providers', url_for('main.view_providers')),
('Reports', url_for('main.platform_admin_reports')),
('Email branding', url_for('main.email_branding')), ('Email branding', url_for('main.email_branding')),
('Letter branding', url_for('main.letter_branding')), ('Letter branding', url_for('main.letter_branding')),
('Inbound SMS numbers', url_for('main.inbound_sms_admin')), ('Inbound SMS numbers', url_for('main.inbound_sms_admin')),

View File

@@ -0,0 +1,13 @@
{% extends "views/platform-admin/_base_template.html" %}
{% block per_page_title %}
Platform admin
{% endblock %}
{% block platform_admin_content %}
<h1 class="heading-large">
Reports
</h1>
{% endblock %}

View File

@@ -927,3 +927,16 @@ def test_clear_cache_requires_option(client_request, platform_admin_user, mocker
assert normalize_spaces(page.find('span', class_='error-message').text) == 'Not a valid choice' assert normalize_spaces(page.find('span', class_='error-message').text) == 'Not a valid choice'
assert not redis.delete_cache_keys_by_pattern.called assert not redis.delete_cache_keys_by_pattern.called
def test_reports_page(
client,
platform_admin_user,
mocker,
):
mock_get_user(mocker, user=platform_admin_user)
client.login(platform_admin_user)
response = client.get(url_for('main.platform_admin_reports'))
assert response.status_code == 200
response.get_data(as_text=True)