test: fixed authentication tests

.github/workflows/ci.yml

@@ -71,6 +71,9 @@ jobs:
     - name: setup requirements
       working-directory: /home/runner/frappe-bench
       run: bench setup requirements --dev
+    - name: block endpoints
+      working-directory: /home/runner/frappe-bench
+      run: bench --site frappe.local set-config block_endpoints 1
     - name: allow tests
       working-directory: /home/runner/frappe-bench
       run: bench --site frappe.local set-config allow_tests true

lms/test_auth.py

@@ -1,6 +1,7 @@
 import frappe
 from frappe.tests.test_api import FrappeAPITestCase
 
+from lms.auth import authenticate
 from lms.lms.test_utils import TestUtils
 
 
@@ -11,25 +12,16 @@ class TestAuth(FrappeAPITestCase):
 		)
 
 	def test_allowed_path(self):
-		site_url = frappe.utils.get_site_url(frappe.local.site)
+		frappe.form_dict.cmd = "ping"
-		headers = {"Authorization": "Bearer set_test_example_user"}
+		frappe.session.user = self.normal_user.name
-		url = site_url + "/api/method/lms.lms.utils.get_courses"
+		authenticate()
-		response = self.get(
+		frappe.session.user = "Administrator"
-			url,
-			headers=headers,
-		)
-		self.assertNotEqual(response.json.get("exc_type"), "PermissionError")
 
 	def test_not_allowed_path(self):
-		site_url = frappe.utils.get_site_url(frappe.local.site)
+		frappe.form_dict.cmd = "frappe.auth.get_logged_user"
-		headers = {"Authorization": "Bearer set_test_example_user"}
+		frappe.session.user = self.normal_user.name
-		url = site_url + "/api/method/frappe.auth.get_logged_user"
+		self.assertRaises(frappe.PermissionError, authenticate)
-		response = self.get(
+		frappe.session.user = "Administrator"
-			url,
-			headers=headers,
-		)
-		print(response.json)
-		self.assertEqual(response.json.get("exc_type"), "PermissionError")
 
 	def tearDown(self):
 		frappe.delete_doc("User", self.normal_user.name)