fix(UX): forget password shows error on wrong email (#19235)

* fix: forget password shows error on wrong email

* test: expected test failure

[skip ci]

Co-authored-by: Ankush Menat <ankush@frappe.io>
This commit is contained in:
Yash Jane 2022-12-12 12:45:27 +05:30 committed by GitHub
parent bb5b0e5e64
commit b93022a54e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 4 additions and 3 deletions

View file

@ -288,7 +288,7 @@ class TestUser(FrappeTestCase):
c = FrappeClient(url)
res1 = c.session.post(url, data=data, verify=c.verify, headers=c.headers)
res2 = c.session.post(url, data=data, verify=c.verify, headers=c.headers)
self.assertEqual(res1.status_code, 400)
self.assertEqual(res1.status_code, 404)
self.assertEqual(res2.status_code, 417)
def test_user_rename(self):

View file

@ -891,7 +891,7 @@ def reset_password(user):
title=_("Password Email Sent"),
)
except frappe.DoesNotExistError:
frappe.local.response["http_status_code"] = 400
frappe.local.response["http_status_code"] = 404
frappe.clear_messages()
return "not found"

View file

@ -253,7 +253,8 @@ login.login_handlers = (function () {
}
},
401: get_error_handler('{{ _("Invalid Login. Try again.") }}'),
417: get_error_handler('{{ _("Oops! Something went wrong") }}')
417: get_error_handler('{{ _("Oops! Something went wrong") }}'),
404: get_error_handler('{{ _("User does not exist.")}}')
};
return login_handlers;