Minor cleanup, setting the set_facl_job to run by default

TACC · Jan 31, 2025 · 86b86d9 · 86b86d9
1 parent 23ffc64
commit 86b86d9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/server/portal/apps/projects/views_unit_test.py b/server/portal/apps/projects/views_unit_test.py
@@ -373,7 +373,7 @@ def test_project_instance_patch(
     }
 
 
-def test_project_change_role(client, mock_project_mgr):
+def test_project_change_role(client, mock_project_mgr, project_list):
     mock_project_mgr.change_project_role.return_value = MagicMock(
         metadata={"projectId": "PRJ-123"}
     )
@@ -395,7 +395,7 @@ def test_project_change_role(client, mock_project_mgr):
 
 
 def test_project_change_system_role(
-    client, mock_service_account, mock_tapis_client
+    client, mock_service_account, mock_tapis_client, project_list
 ):
     # USER translates to writer role
     patch_body = {
@@ -431,7 +431,7 @@ def test_project_change_system_role(
 
 @override_settings(PORTAL_PROJECTS_USE_SET_FACL_JOB=True)
 def test_project_change_system_role_setfacl_job(
-    client, mock_service_account, mock_tapis_client
+    client, mock_service_account, mock_tapis_client, project_list
 ):
     mock_rootDir = mock_service_account().systems.getSystem().rootDir
 

diff --git a/server/portal/settings/settings.py b/server/portal/settings/settings.py
@@ -601,7 +601,7 @@
 
 PORTAL_ALLOCATION = getattr(settings_custom, '_PORTAL_ALLOCATION', '')
 
-PORTAL_PROJECTS_USE_SET_FACL_JOB = getattr(settings_custom, '_PORTAL_PROJECTS_USE_SET_FACL_JOB', False)
+PORTAL_PROJECTS_USE_SET_FACL_JOB = getattr(settings_custom, '_PORTAL_PROJECTS_USE_SET_FACL_JOB', True)
 
 """
 SETTINGS: ELASTICSEARCH