From 48b4dce8486e9ccc9101f5144e01b5b44ffbb27e Mon Sep 17 00:00:00 2001 From: Imdad Ahad Date: Tue, 7 Feb 2017 13:32:20 +0000 Subject: [PATCH] Update password on user profile with new endpoint --- app/main/views/user_profile.py | 3 +-- tests/app/main/views/test_user_profile.py | 2 +- tests/conftest.py | 8 ++++++++ 3 files changed, 10 insertions(+), 3 deletions(-) diff --git a/app/main/views/user_profile.py b/app/main/views/user_profile.py index 4107e53cd..e59619dab 100644 --- a/app/main/views/user_profile.py +++ b/app/main/views/user_profile.py @@ -201,8 +201,7 @@ def user_profile_password(): form = ChangePasswordForm(_check_password) if form.validate_on_submit(): - current_user.set_password(form.new_password.data) - user_api_client.update_user(current_user) + user_api_client.update_password(current_user.id, password=form.new_password.data) return redirect(url_for('.user_profile')) return render_template( diff --git a/tests/app/main/views/test_user_profile.py b/tests/app/main/views/test_user_profile.py index a1601460b..8f1f2c6a4 100644 --- a/tests/app/main/views/test_user_profile.py +++ b/tests/app/main/views/test_user_profile.py @@ -239,7 +239,7 @@ def test_should_redirect_after_password_change( api_user_active, mock_login, mock_get_user, - mock_update_user, + mock_update_user_password, mock_verify_password, ): data = { diff --git a/tests/conftest.py b/tests/conftest.py index 0e11976e4..53a9ca0b5 100644 --- a/tests/conftest.py +++ b/tests/conftest.py @@ -801,6 +801,14 @@ def mock_update_user(mocker, api_user_active): return mocker.patch('app.user_api_client.update_user', side_effect=_update) +@pytest.fixture(scope='function') +def mock_update_user_password(mocker, api_user_active): + def _update(user_id, **kwargs): + return api_user_active + + return mocker.patch('app.user_api_client.update_password', side_effect=_update) + + @pytest.fixture(scope='function') def mock_update_user_attribute(mocker, api_user_active): def _update(user_id, **kwargs):