Issue 48499: Use preferred SecurityPolicyManager.savePolicy() variant (#243)

Author: labkey-jeckels

OpenLdapSync/src/org/labkey/openldapsync/ldap/LdapSyncRunner.java

@@ -893,7 +893,7 @@ public void doSetup() throws Exception
             Container project = ContainerManager.getForPath(PROJECT_NAME);
             if (project == null)
             {
-                project = ContainerManager.createContainer(ContainerManager.getRoot(), PROJECT_NAME);
+                project = ContainerManager.createContainer(ContainerManager.getRoot(), PROJECT_NAME, TestContext.get().getUser());
                 Set<Module> modules = new HashSet<>();
                 modules.addAll(project.getActiveModules());
                 modules.add(ModuleLoader.getInstance().getModule(OpenLdapSyncModule.NAME));

SequenceAnalysis/src/org/labkey/sequenceanalysis/SequenceIntegrationTests.java

@@ -244,7 +244,7 @@ protected static void doInitialSetUp(String projectName) throws Exception
             Container project = ContainerManager.getForPath(projectName);
             if (project == null)
             {
-                project = ContainerManager.createContainer(ContainerManager.getRoot(), projectName);
+                project = ContainerManager.createContainer(ContainerManager.getRoot(), projectName, TestContext.get().getUser());
 
                 //disable search so we dont get conflicts when deleting folder quickly
                 ContainerManager.updateSearchable(project, false, TestContext.get().getUser());

SequenceAnalysis/src/org/labkey/sequenceanalysis/run/analysis/BamIterator.java

@@ -263,7 +263,7 @@ public static void before() throws Exception
             Container project = ContainerManager.getForPath(PROJECT_NAME);
             if (project == null)
             {
-                project = ContainerManager.createContainer(ContainerManager.getRoot(), PROJECT_NAME);
+                project = ContainerManager.createContainer(ContainerManager.getRoot(), PROJECT_NAME, TestContext.get().getUser());
                 Set<Module> modules = new HashSet<>();
                 modules.addAll(project.getActiveModules());
                 modules.add(ModuleLoader.getInstance().getModule(SequenceAnalysisModule.NAME));

cluster/src/org/labkey/cluster/pipeline/TestCase.java

@@ -52,7 +52,7 @@ public static void initialSetUp() throws Exception
         Container project = ContainerManager.getForPath(PROJECT_NAME);
         if (project == null)
         {
-            project = ContainerManager.createContainer(ContainerManager.getRoot(), PROJECT_NAME);
+            project = ContainerManager.createContainer(ContainerManager.getRoot(), PROJECT_NAME, TestContext.get().getUser());
             Set<Module> modules = new HashSet<>();
             modules.addAll(project.getActiveModules());
             modules.add(ModuleLoader.getInstance().getModule(ClusterModule.class));
@@ -102,19 +102,19 @@ protected static void doCleanup(String projectName)
     public void basicTest() throws Exception
     {
         Container c = ContainerManager.getForPath(PROJECT_NAME);
-        for (RemoteExecutionEngine engine : PipelineJobService.get().getRemoteExecutionEngines())
+        for (RemoteExecutionEngine<?> engine : PipelineJobService.get().getRemoteExecutionEngines())
         {
             _log.info("testing engine: " + engine.getType());
-            if (engine instanceof AbstractClusterExecutionEngine)
+            if (engine instanceof AbstractClusterExecutionEngine<?> acee)
             {
-                runTestJob(c, TestContext.get().getUser(), (AbstractClusterExecutionEngine) engine);
+                runTestJob(c, TestContext.get().getUser(), acee);
             }
         }
     }
 
     public static class TestRunner implements ClusterService.ClusterRemoteTask
     {
-        public long _sleep = 0;
+        public long _sleep = 0l;
 
         public TestRunner(long sleep)
         {