vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 4

fix(test): Remove frappe.local.request between requests

Browse source 
`frappe.local.request` was not cleared between tests, which would not be a problem if all tests did set it to another Request object. But, some tests directly fetch the response content using get_response_content without first setting the frappe.local.request object (using set_request).
This commit is contained in:
Corentin Flr 2023-06-16 13:27:49 +02:00 • committed by GitHub
parent bcdc483a13
commit 9afedfae25
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
frappe/tests/test_website.py
View file

@ -11,10 +11,16 @@ from frappe.website.utils import build_response, clear_website_cache, get_home_p
class TestWebsite(FrappeTestCase):
def setUp(self):
frappe.set_user("Guest")
self._clearRequest()
def tearDown(self):
frappe.db.delete("Access Log")
frappe.set_user("Administrator")
self._clearRequest()
def _clearRequest(self):
if hasattr(frappe.local, "request"):
delattr(frappe.local, "request")
def test_home_page(self):
frappe.set_user("Administrator")