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):