Fix wild import warnings in system tests

Author: robertapplin

Framework/PythonInterface/plugins/algorithms/WorkflowAlgorithms/SANSDarkRunBackgroundCorrection.py

@@ -5,8 +5,16 @@
 #   Institut Laue - Langevin & CSNS, Institute of High Energy Physics, CAS
 # SPDX - License - Identifier: GPL - 3.0 +
 # pylint: disable=no-init,invalid-name,too-many-locals,too-many-branches
-from mantid.api import AlgorithmFactory, AlgorithmManager, IEventWorkspace, MatrixWorkspaceProperty, Progress, PythonAlgorithm
-from mantid.kernel import CommonBinsValidator, Direction, IntArrayBoundedValidator, IntArrayProperty, Logger
+from mantid.api import (
+    AlgorithmFactory,
+    AlgorithmManager,
+    CommonBinsValidator,
+    IEventWorkspace,
+    MatrixWorkspaceProperty,
+    Progress,
+    PythonAlgorithm,
+)
+from mantid.kernel import Direction, IntArrayBoundedValidator, IntArrayProperty, Logger
 import numpy as np
 
 

Framework/PythonInterface/plugins/algorithms/WorkflowAlgorithms/SimulatedDensityOfStates.py

@@ -13,8 +13,8 @@
 
 import scipy.constants
 
-from mantid.api import AlgorithmFactory, Progress, PythonAlgorithm, WorkspaceProperty
-from mantid.kernel import logger, Direction, FileAction, FileProperty, StringArrayProperty, StringListValidator
+from mantid.api import AlgorithmFactory, FileAction, FileProperty, Progress, PythonAlgorithm, WorkspaceProperty
+from mantid.kernel import logger, Direction, StringArrayProperty, StringListValidator
 import mantid.simpleapi as s_api
 
 

Framework/PythonInterface/plugins/algorithms/WorkflowAlgorithms/SofQWMomentsScan.py

@@ -15,12 +15,10 @@
     IntArrayMandatoryValidator,
     IntArrayProperty,
     Property,
-    SofQWMoments,
     StringArrayProperty,
     StringListValidator,
-    Rebin,
 )
-from mantid.simpleapi import Fit, LoadLog
+from mantid.simpleapi import Fit, LoadLog, Rebin, SofQWMoments
 
 import os
 import numpy as np

Framework/PythonInterface/plugins/algorithms/WorkflowAlgorithms/TimeSlice.py

@@ -8,19 +8,21 @@
 from mantid.api import mtd, AlgorithmFactory, Progress, PropertyMode, PythonAlgorithm, WorkspaceGroupProperty, WorkspaceProperty
 from mantid.kernel import (
     logger,
+    Direction,
+    FloatArrayProperty,
+    IntArrayProperty,
+    StringArrayProperty,
+)
+from mantid.simpleapi import (
     CalculateFlatBackground,
     CropWorkspace,
     DeleteWorkspace,
-    Direction,
     Divide,
     GroupWorkspaces,
     Integration,
-    FloatArrayProperty,
-    IntArrayProperty,
-    StringArrayProperty,
+    Load,
     Transpose,
 )
-from mantid.simpleapi import Load
 
 from IndirectCommon import check_hist_zero
 

Framework/PythonInterface/test/python/plugins/algorithms/WorkflowAlgorithms/SANSFitShiftScaleTest.py

@@ -5,7 +5,7 @@
 #   Institut Laue - Langevin & CSNS, Institute of High Energy Physics, CAS
 # SPDX - License - Identifier: GPL - 3.0 +
 import unittest
-from mantid.api import AlgorithmManager
+from mantid.api import AlgorithmManager, FrameworkManager
 import numpy as np
 from SANSFitShiftScale import ErrorTransferFromModelToData
 
@@ -293,4 +293,5 @@ def test_that_error_is_transferred(self):
 
 
 if __name__ == "__main__":
+    FrameworkManager.Instance()
     unittest.main()

Framework/PythonInterface/test/python/plugins/algorithms/WorkflowAlgorithms/SANSStitchTest.py

@@ -5,7 +5,7 @@
 #   Institut Laue - Langevin & CSNS, Institute of High Energy Physics, CAS
 # SPDX - License - Identifier: GPL - 3.0 +
 import unittest
-from mantid.api import AlgorithmManager, MatrixWorkspace
+from mantid.api import AlgorithmManager, FrameworkManager, MatrixWorkspace
 import numpy as np
 
 from SANSStitch import QErrorCorrectionForMergedWorkspaces
@@ -890,4 +890,5 @@ def test_correct_x_error_is_produced(self):
 
 
 if __name__ == "__main__":
+    FrameworkManager.Instance()
     unittest.main()

Testing/SystemTests/tests/framework/ARCSReductionTest.py

@@ -11,7 +11,9 @@
 
 import os
 import systemtesting
-from mantid.simpleapi import *
+from mantid.api import mtd
+from mantid.kernel import config
+from mantid.simpleapi import DeleteWorkspace, DgsReduction, ExtractMask, Load, LoadNXSPE, SaveNXSPE
 
 
 class ARCSReductionTest(systemtesting.MantidSystemTest):

Testing/SystemTests/tests/framework/AlignAndFocusPowderFromFilesTest.py

@@ -6,8 +6,27 @@
 # SPDX - License - Identifier: GPL - 3.0 +
 # pylint: disable=no-init,invalid-name,attribute-defined-outside-init
 import systemtesting
-from mantid.simpleapi import *
-from mantid.api import WorkspaceFactory
+from mantid.api import mtd, AnalysisDataService, WorkspaceFactory
+from mantid.kernel import PropertyManagerDataService
+from mantid.simpleapi import (
+    AlignAndFocusPowder,
+    AlignAndFocusPowderFromFiles,
+    ConvertUnits,
+    CreateGroupingWorkspace,
+    CylinderAbsorption,
+    DeleteWorkspace,
+    Divide,
+    FilterBadPulses,
+    LoadDiffCal,
+    LoadEventAndCompress,
+    LoadEventNexus,
+    NormaliseByCurrent,
+    PDDetermineCharacterizations,
+    PDLoadCharacterizations,
+    RebinRagged,
+    SetSample,
+    SortEvents,
+)
 import numpy as np
 import os
 import time
@@ -418,7 +437,7 @@ def runTest(self):
         for i in range(absorptionWS.getNumberHistograms()):
             absorptionWS.setX(i, xaxis)
         absorptionWS.getAxis(0).setUnit("Wavelength")
-        mantid.api.AnalysisDataService.addOrReplace("V_abs", absorptionWS)
+        AnalysisDataService.addOrReplace("V_abs", absorptionWS)
         SetSample(
             InputWorkspace="V_abs",
             Material={"ChemicalFormula": "V", "SampleNumberDensity": 0.0721},

Testing/SystemTests/tests/framework/BuildSQWTest.py

@@ -16,7 +16,19 @@
 
 import systemtesting
 import os
-from mantid.simpleapi import *
+from mantid.kernel import config, logger
+from mantid.simpleapi import (
+    CompareMDWorkspaces,
+    ConvertToMD,
+    DeleteWorkspace,
+    LoadMD,
+    LoadNXSPE,
+    LoadSQW,
+    MergeMDFiles,
+    SaveMD,
+    SetGoniometer,
+    SetUB,
+)
 
 # allow for multiple locations
 FILE_LOCATIONS = ["/isis/mantid/localtestdata/"]  # ,"d:/Data/MantidSystemTests/BigData/Dropbox/LoadSQW"]
@@ -109,7 +121,7 @@ def cleanup(self):
             try:
                 os.remove(filename)
             except OSError:
-                mantid.logger.warning("Unable to remove created file '%s'" % filename)
+                logger.warning("Unable to remove created file '%s'" % filename)
 
 
 class LoadSQW_FileBasedTest(BuildSQWTest):

Testing/SystemTests/tests/framework/CNCSReductionTest.py

@@ -11,7 +11,9 @@
 
 import os
 import systemtesting
-from mantid.simpleapi import *
+from mantid.api import mtd
+from mantid.kernel import config
+from mantid.simpleapi import CNCSSuggestTIB, DeleteWorkspace, DgsReduction, GenerateGroupingPowder, Load, LoadNXSPE, SaveNXSPE
 
 
 class CNCSReductionTest(systemtesting.MantidSystemTest):