Merge pull request #21397 from cogk/fix-test_never_render

fix(test): Fix test_never_render to get path as string, exclude .pyc files from served files

frappe/tests/test_website.py

@@ -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")
@@ -340,8 +346,9 @@ class TestWebsite(FrappeTestCase):
 		FILES_TO_SKIP = choices(list(WWW.glob("**/*.py*")), k=10)
 
 		for suffix in FILES_TO_SKIP:
-			content = get_response_content(suffix.relative_to(WWW))
-			self.assertIn("404", content)
+			path: str = suffix.relative_to(WWW).as_posix()
+			content = get_response_content(path)
+			self.assertIn("<title>Not Found</title>", content)
 
 	def test_metatags(self):
 		content = get_response_content("/_test/_test_metatags")

frappe/website/page_renderers/static_page.py

@@ -8,7 +8,18 @@ import frappe
 from frappe.website.page_renderers.base_renderer import BaseRenderer
 from frappe.website.utils import is_binary_file
 
-UNSUPPORTED_STATIC_PAGE_TYPES = ("html", "md", "js", "xml", "css", "txt", "py", "json")
+UNSUPPORTED_STATIC_PAGE_TYPES = (
+	"css",
+	"html",
+	"js",
+	"json",
+	"md",
+	"py",
+	"pyc",
+	"pyo",
+	"txt",
+	"xml",
+)
 
 
 class StaticPage(BaseRenderer):

frappe/website/path_resolver.py

@@ -51,7 +51,6 @@ class PathResolver:
 			TemplatePage,
 			ListPage,
 			PrintPage,
-			NotFoundPage,
 		]
 
 		for renderer in renderers: