From f08faf151c48a3f65b0dabb94ec2e986cf5fb515 Mon Sep 17 00:00:00 2001
From: Gavin D'souza <gavin18d@gmail.com>
Date: Fri, 22 Oct 2021 18:18:14 +0530
Subject: [PATCH] test(website): Clear db.value_cache on tearDown

---
 frappe/tests/test_website.py | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/frappe/tests/test_website.py b/frappe/tests/test_website.py
index 688679a80f..25aa7b31ce 100644
--- a/frappe/tests/test_website.py
+++ b/frappe/tests/test_website.py
@@ -4,14 +4,13 @@ import frappe
 from frappe.utils import set_request
 from frappe.website.serve import get_response, get_response_content
 from frappe.website.utils import (build_response, clear_website_cache, get_home_page)
-from tenacity import retry, stop_after_attempt, retry_if_exception_type
-
 
 class TestWebsite(unittest.TestCase):
 	def setUp(self):
 		frappe.set_user('Guest')
 
 	def tearDown(self):
+		frappe.db.value_cache = {}
 		frappe.set_user('Administrator')
 
 	def test_home_page(self):
@@ -197,13 +196,8 @@ class TestWebsite(unittest.TestCase):
 		delattr(frappe.hooks, 'page_renderer')
 		frappe.cache().delete_key('app_hooks')
 
-	# TODO: Get rid of this retry logic
-	# Added since test is flaky and we can't figure out why at this point
-	@retry(
-		stop=stop_after_attempt(5), retry=retry_if_exception_type(AssertionError),
-	)
 	def test_printview_page(self):
-		content = get_response_content('/Language/en')
+		content = get_response_content('/Language/ru')
 		self.assertIn('<div class="print-format">', content)
 		self.assertIn('<div>Language</div>', content)