diff --git a/app/main/views/manage_users.py b/app/main/views/manage_users.py index 9896130f3..d2c0ce5e4 100644 --- a/app/main/views/manage_users.py +++ b/app/main/views/manage_users.py @@ -229,7 +229,15 @@ def edit_user_mobile_number(service_id, user_id): @login_required @user_has_permissions('manage_service') def confirm_edit_user_mobile_number(service_id, user_id): - return True + user = user_api_client.get_user(user_id) + new_number = session['team_member_mobile_change'] + + return render_template( + 'views/manage-users/confirm-edit-user-mobile-number.html', + user=user, + service_id=service_id, + new_mobile_number=new_number + ) @main.route("/services//cancel-invited-user/", methods=['GET']) diff --git a/app/templates/views/manage-users/confirm-edit-user-mobile-number.html b/app/templates/views/manage-users/confirm-edit-user-mobile-number.html new file mode 100644 index 000000000..99265137e --- /dev/null +++ b/app/templates/views/manage-users/confirm-edit-user-mobile-number.html @@ -0,0 +1,30 @@ +{% extends "withnav_template.html" %} +{% from "components/page-footer.html" import page_footer %} +{% from "components/form.html" import form_wrapper %} + +{% block per_page_title %} + Confirm change of mobile number +{% endblock %} + +{% block maincolumn_content %} + +

Confirm change of mobile number

+ +
+
+ {% call form_wrapper() %} +

New mobile number:

+
+

{{ new_mobile_number }}

+
+

We will send {{ user.name }} an email to tell them about the change.

+ {{ page_footer( + 'Confirm', + destructive=destructive, + back_link=url_for('.edit_user_mobile_number', service_id=service_id, user_id=user.id) + ) }} + {% endcall %} +
+
+ +{% endblock %} diff --git a/tests/app/main/views/test_manage_users.py b/tests/app/main/views/test_manage_users.py index ff997a563..7f558c85f 100644 --- a/tests/app/main/views/test_manage_users.py +++ b/tests/app/main/views/test_manage_users.py @@ -1079,8 +1079,31 @@ def test_edit_user_mobile_number_redirects_to_confirmation( ) -def test_confirm_edit_user_mobile_number_page(): - pass +def test_confirm_edit_user_mobile_number_page( + logged_in_client, + active_user_with_permissions, + service_one, + mocker, + mock_get_user, +): + new_number = '07554080636' + with logged_in_client.session_transaction() as session: + session['team_member_mobile_change'] = new_number + response = logged_in_client.get(url_for( + 'main.confirm_edit_user_mobile_number', + service_id=service_one['id'], + user_id=active_user_with_permissions.id + )) + + assert response.status_code == 200 + assert 'Confirm change of mobile number' in response.get_data(as_text=True) + for text in [ + 'New mobile number:', + new_number, + 'We will send {} an email to tell them about the change.'.format(active_user_with_permissions.name) + ]: + assert text in response.get_data(as_text=True) + assert 'Confirm' in response.get_data(as_text=True) def test_confirm_edit_user_mobile_number_changes_user_mobile_number():