chore: delete commented tests and remove ignore flag in makefile

eduNEXT · Nov 7, 2024 · 5816b47 · 5816b47
1 parent 80927f6
commit 5816b47
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 18 deletions.
diff --git a/Makefile b/Makefile
@@ -50,8 +50,7 @@ python-quality-test:
 run-tests: python-test python-quality-test
 
 run-integration-tests: install-dev-dependencies
-	# pytest -rPf ./eox_core --ignore-glob='**/unit/*' --ignore-glob='**/edxapp_wrapper/*'
-	pytest -rPf ./eox_core --ignore-glob='**/unit/*' --ignore-glob='**/edxapp_wrapper/*' --ignore-glob='**/api/*'
+	pytest -rPf ./eox_core --ignore-glob='**/unit/*' --ignore-glob='**/edxapp_wrapper/*'
 
 upgrade: export CUSTOM_COMPILE_COMMAND=make upgrade
 upgrade: ## update the requirements/*.txt files with the latest packages satisfying requirements/*.in

diff --git a/eox_core/tests/integration/test_middleware.py b/eox_core/tests/integration/test_middleware.py
@@ -32,16 +32,6 @@ def test_redirect_always(self):
         self.assertEqual(response.status_code, status.HTTP_200_OK)
         self.assertEqual(response.url, f"{self.tenant_x_url}/donate")
 
-    # def test_login_required_with_logged_user(self):
-    #     """Test the login_required feature with a logged user."""
-    #     response = requests.get(
-    #         f"{self.tenant_x_url}/tos",
-    #         headers={"Authorization": f"Bearer {ACCESS_TOKEN}"},
-    #         timeout=settings["API_TIMEOUT"],
-    #     )
-    #     self.assertEqual(response.status_code, status.HTTP_200_OK)
-    #     self.assertEqual(response.url, f"{self.tenant_x_url}/tos")
-
     def test_login_required(self):
         """Test the login_required feature."""
         response = requests.get(f"{self.tenant_x_url}/tos", timeout=settings["API_TIMEOUT"])
@@ -54,12 +44,6 @@ def test_not_found(self):
         self.assertEqual(response.status_code, status.HTTP_404_NOT_FOUND)
         self.assertEqual(response.url, f"{self.tenant_x_url}/privacy")
 
-    # def test_not_found_logged_in(self):
-    #     """Test the not_found and login_required feature."""
-    #     response = requests.get(f"{self.tenant_x_url}/help", timeout=settings["API_TIMEOUT"])
-    #     self.assertEqual(response.status_code, status.HTTP_404_NOT_FOUND)
-    #     self.assertEqual(response.url, f"{self.tenant_x_url}/help")
-
     def test_not_found_logged_out(self):
         """Test the not_found and logout feature."""
         response = requests.get(f"{self.tenant_x_url}/contact", timeout=settings["API_TIMEOUT"])