From 11235f6b3a58548ff24736a6cdef9930663d8557 Mon Sep 17 00:00:00 2001 From: Sagar Vora <16315650+sagarvora@users.noreply.github.com> Date: Fri, 21 Nov 2025 17:01:06 +0530 Subject: [PATCH] fix: allow whitelist_for_tests in CI environment The CI check was incorrectly nested inside the dev_server condition, preventing test endpoints from being accessible during CI runs using bench execute. Now CI is checked independently. --- frappe/tests/utils/__init__.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/frappe/tests/utils/__init__.py b/frappe/tests/utils/__init__.py index 8e559405c8..ded61c6860 100644 --- a/frappe/tests/utils/__init__.py +++ b/frappe/tests/utils/__init__.py @@ -25,7 +25,7 @@ def whitelist_for_tests(**whitelist_kwargs): @wraps(fn) def wrapper(*args, **kwargs): if not ( - frappe.in_test or (frappe._dev_server and (frappe.conf.allow_tests or os.environ.get("CI"))) + frappe.in_test or (frappe._dev_server and frappe.conf.allow_tests) or os.environ.get("CI") ): frappe.throw( # nosemgrep: frappe-missing-translate-function-python 'Test endpoints are only available when running in test mode or running a development server ("bench start") with the "allow_tests" site config enabled'