refactor: move get_mapdl_instances and get_ansys_process_from_port to helpers.py

Author: germa89

src/ansys/mapdl/core/cli/core.py src/ansys/mapdl/core/cli/helpers.pysrc/ansys/mapdl/core/cli/core.py renamed to src/ansys/mapdl/core/cli/helpers.py

@@ -25,8 +25,6 @@
 This module avoids importing heavy dependencies like pandas, numpy, etc.
 """
 
-import getpass
-import platform
 from typing import Any, Dict, List
 
 import psutil
@@ -49,6 +47,9 @@ def can_access_process(proc):
     bool
         True if we can safely access the process
     """
+    import getpass
+    import platform
+
     try:
         # Check if we can access basic process info and if it belongs to current user
         current_user = getpass.getuser()
@@ -146,10 +147,6 @@ def get_mapdl_instances() -> List[Dict[str, Any]]:
 def get_ansys_process_from_port(port: int):
     import socket
 
-    import psutil
-
-    from ansys.mapdl.core.cli.core import is_alive_status, is_valid_ansys_process_name
-
     # Filter by name first
     potential_procs = []
     for proc in psutil.process_iter(attrs=["name"]):

src/ansys/mapdl/core/cli/list_instances.py

@@ -72,7 +72,7 @@ def list_instances(instances, long, cmd, location) -> None:
 def _list_instances(instances, long, cmd, location):
     from tabulate import tabulate
 
-    from ansys.mapdl.core.cli.core import get_mapdl_instances
+    from ansys.mapdl.core.cli.helpers import get_mapdl_instances
 
     # Assuming all ansys processes have -grpc flag
     mapdl_instances = get_mapdl_instances()

tests/test_cli.py

@@ -31,8 +31,8 @@
 import psutil
 import pytest
 
-import ansys.mapdl.core.cli.core as core_module
-from ansys.mapdl.core.cli.core import get_ansys_process_from_port
+import ansys.mapdl.core.cli.helpers as core_module
+from ansys.mapdl.core.cli.helpers import get_ansys_process_from_port
 from ansys.mapdl.core.plotting import GraphicsBackend
 from conftest import VALID_PORTS, requires