Fix InProcess path update (#1449)

* update generated code

* Revert correct licensing behavior

* debug

* Move one test to other file

* update generated code

* Temporary skip of remote_operator tests

* revert

* revert prints

* revert CI

* missing print

* readd tests

* Revert "missing print"

This reverts commit ded2061.

* Revert "Revert "missing print""

This reverts commit 77455f5.

* Revert "readd tests"

This reverts commit 4d56b19.

* Fix 1

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

* Remote Operators Fix2

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

* Revert "Temporary skip of remote_operator tests"

This reverts commit 777909b.

* Remote Operators Fix3

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

* Revert PATH fix for Linux

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

* Revert "Temporary skip of remote_operator tests"

This reverts commit 777909b.

* Get the right standalone suffix when PR on a "maint/update_code_for_*" branch

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

Get the right standalone suffix when PR on a "maint/update_code_for_*" branch

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

* Unskip test_remote_operator.py tests

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

* Fix code quality

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

* Fix code quality

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

* Fix code quality

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

---------

Signed-off-by: paul.profizi <[email protected]>
Co-authored-by: rlagha <[email protected]>
Co-authored-by: Rafael Canton <[email protected]>
Co-authored-by: rafacanton <[email protected]>

Author: 3 people

.github/workflows/ci.yml

@@ -55,6 +55,10 @@ jobs:
             base_ref=${{ github.base_ref }}
             suffix=${ref_name/$sub[0-9][0-9][0-9]/''}
             suffix=${suffix/_on_$base_ref/''}
+          elif [[ "${{ github.base_ref  }}" == *"$sub"* ]]; then
+            base_ref=${{ github.base_ref }}
+            suffix=${base_ref/$sub[0-9][0-9][0-9]/''}
+            suffix=${suffix/_on_*/''}
           fi
           echo "suffix=$suffix" >> "$GITHUB_OUTPUT"
         fi

src/ansys/dpf/core/server_types.py

@@ -925,6 +925,12 @@ def __init__(
             self._context = server_context.AvailableServerContexts.premium
             pass
         self.set_as_global(as_global=as_global)
+        # Update the python os.environment
+        if not os.name == "posix":
+            new_path = subprocess.check_output(
+                ["python", "-c", r'import os; print(os.environ["PATH"])'], text=True
+            )  # pragma: no cover
+            os.environ["PATH"] = new_path
 
     @property
     def version(self):

tests/test_remote_operator.py

@@ -6,7 +6,6 @@
 import pytest
 
 
-@pytest.mark.skip
 @pytest.mark.skipif(
     not conftest.SERVERS_VERSION_GREATER_THAN_OR_EQUAL_TO_3_0,
     reason="Connecting data from different servers is " "supported starting server version 3.0",
@@ -21,7 +20,6 @@ def test_connect_remote_operators(simple_bar):
     assert np.allclose(fc[0].data, 2 * op1.outputs.fields_container()[0].data)
 
 
-@pytest.mark.skip
 @pytest.mark.skipif(
     not conftest.SERVERS_VERSION_GREATER_THAN_OR_EQUAL_TO_3_0,
     reason="Connecting data from different servers is " "supported starting server version 3.0",
@@ -36,7 +34,6 @@ def test_connect_3remote_operators(simple_bar):
     assert np.allclose(fc[0].data, 2 * op1.outputs.fields_container()[0].data)
 
 
-@pytest.mark.skip
 @pytest.mark.skipif(
     not conftest.SERVERS_VERSION_GREATER_THAN_OR_EQUAL_TO_4_0,
     reason="Connecting data from different servers is " "supported starting server version 4.0",