Fix a bug where a new server is run on object creation (#1592)

* Add test

Signed-off-by: paul.profizi <[email protected]>

* Fix for Scoping

Signed-off-by: paul.profizi <[email protected]>

* Fix for all

Signed-off-by: paul.profizi <[email protected]>

---------

Signed-off-by: paul.profizi <[email protected]>

Author: PProfizi

src/ansys/dpf/core/any.py

@@ -33,7 +33,9 @@ class Any:
 
     def __init__(self, any_dpf=None, server=None):
         # step 1: get server
-        self._server = server_module.get_or_create_server(server)
+        self._server = server_module.get_or_create_server(
+            any_dpf._server if isinstance(any_dpf, Any) else server
+        )
 
         if any_dpf is None and not self._server.meet_version("7.0"):
             raise errors.DpfVersionNotSupported("7.0")

src/ansys/dpf/core/collection.py

@@ -33,7 +33,9 @@ class Collection(CollectionBase[TYPE]):
 
     def __init__(self, collection=None, server=None, entries_type: type = None):
         # step 1: get server
-        self._server = server_module.get_or_create_server(server)
+        self._server = server_module.get_or_create_server(
+            collection._server if isinstance(collection, Collection) else server
+        )
         if not self._server.meet_version("8.1"):
             raise errors.DpfVersionNotSupported("8.1")
 

src/ansys/dpf/core/collection_base.py

@@ -46,7 +46,9 @@ class CollectionBase(Generic[TYPE]):
 
     def __init__(self, collection=None, server: BaseServer = None):
         # step 1: get server
-        self._server = server_module.get_or_create_server(server)
+        self._server = server_module.get_or_create_server(
+            collection._server if isinstance(collection, CollectionBase) else server
+        )
 
         # step2: if object exists, take the instance, else create it
         self._internal_obj = None

src/ansys/dpf/core/config.py

@@ -57,7 +57,9 @@ class Config:
 
     def __init__(self, operator_name=None, config=None, server=None, spec=None):
         # step 1: get server
-        self._server = server_module.get_or_create_server(server)
+        self._server = server_module.get_or_create_server(
+            config._server if isinstance(config, Config) else server
+        )
 
         # step 2: get api
         self._api_instance = None  # see _api property

src/ansys/dpf/core/custom_type_field.py

@@ -93,7 +93,9 @@ def __init__(
         """Initialize the field either with an optional field message or
         by connecting to a stub.
         """
-        self._server = server_module.get_or_create_server(server)
+        self._server = server_module.get_or_create_server(
+            field._server if isinstance(field, CustomTypeField) else server
+        )
         if field is None and not self._server.meet_version("5.0"):
             raise errors.DpfVersionNotSupported("5.0")
         if unitary_type is not None:

src/ansys/dpf/core/cyclic_support.py

@@ -45,7 +45,9 @@ class CyclicSupport:
     def __init__(self, cyclic_support, server=None):
         """Initialize time frequency support with its `TimeFreqSupport` message (if possible)."""
         # step 1: get server
-        self._server = server_module.get_or_create_server(server)
+        self._server = server_module.get_or_create_server(
+            cyclic_support._server if isinstance(cyclic_support, CyclicSupport) else server
+        )
 
         # step 2: get api
         self._api = self._server.get_api_for_type(

src/ansys/dpf/core/data_sources.py

@@ -55,7 +55,9 @@ class DataSources:
     def __init__(self, result_path=None, data_sources=None, server=None):
         """Initialize a connection with the server."""
         # step 1: get server
-        self._server = server_module.get_or_create_server(server)
+        self._server = server_module.get_or_create_server(
+            data_sources._server if isinstance(data_sources, DataSources) else server
+        )
 
         # step 2: get api
         self._api = self._server.get_api_for_type(

src/ansys/dpf/core/data_tree.py

@@ -84,7 +84,9 @@ def __init__(self, data=None, data_tree=None, server=None):
             "_holds_server",
         ]
         # step 1: get server
-        self._server_instance = server_module.get_or_create_server(server)
+        self._server_instance = server_module.get_or_create_server(
+            data_tree._server_instance if isinstance(data_tree, DataTree) else server
+        )
 
         if data_tree is None and not self._server.meet_version("4.0"):
             raise errors.DpfVersionNotSupported("4.0")

src/ansys/dpf/core/dpf_operator.py

@@ -103,7 +103,9 @@ def __init__(self, name, config=None, server=None):
         self._inputs = None
 
         # step 1: get server
-        self._server = server_module.get_or_create_server(server)
+        self._server = server_module.get_or_create_server(
+            config._server if isinstance(config, Config) else server
+        )
 
         # step 2: get api
         self._api_instance = None  # see _api property

src/ansys/dpf/core/field_definition.py

@@ -32,7 +32,9 @@ class FieldDefinition:
 
     def __init__(self, field_definition=None, server=None):
         # step 1: get server
-        self._server = server_module.get_or_create_server(server)
+        self._server = server_module.get_or_create_server(
+            field_definition._server if isinstance(field_definition, FieldDefinition) else server
+        )
 
         # step 2: get api
         self._api = self._server.get_api_for_type(