Fix Workflow.view and add graphviz to the testing environment (#2099)

* Fix workflow.view()

* Add graphviz to requirements_test.txt

* Install graphviz in test pipelines

* Fix skip message in test_workflow_view

Author: PProfizi

.github/workflows/test_docker.yml

@@ -51,6 +51,9 @@ jobs:
         with:
           python-version: ${{ matrix.python-version }}
 
+      - name: "Setup Graphviz"
+        uses: ts-graphviz/setup-graphviz@v2
+
       - name: "Install requirements"
         run: pip install -r requirements/requirements_build.txt
 

.github/workflows/tests.yml

@@ -114,6 +114,9 @@ jobs:
         with:
           python-version: ${{ matrix.python-version }}
 
+      - name: "Setup Graphviz"
+        uses: ts-graphviz/setup-graphviz@v2
+
       - name: "Update pip to the latest version and install tox"
         shell: pwsh
         run: |

requirements/requirements_test.txt

@@ -1,5 +1,6 @@
 ansys-platform-instancemanagement==1.1.2
 coverage==7.6.11
+graphviz==0.20.1
 imageio==2.36.0
 imageio-ffmpeg==0.6.0
 pytest==8.3.4

src/ansys/dpf/core/workflow.py

@@ -934,10 +934,10 @@ def view(
 
         if save_as:
             image_path = Path(save_as)
-            dot_path = image_path.parent / image_path.stem / ".dot"
+            dot_path = image_path.parent / f"{image_path.stem}.dot"
         else:
             image_path = Path.cwd() / f"{name}.png"
-            dot_path = image_path.parent / image_path.stem / ".dot"
+            dot_path = image_path.parent / f"{image_path.stem}.dot"
 
         # Create graphviz file of workflow
         self.to_graphviz(dot_path)

tests/test_workflow.py

@@ -63,7 +63,7 @@ def remove_files():
     request.addfinalizer(remove_files)
 
 
-@pytest.mark.skipif(not HAS_GRAPHVIZ, reason="Please install pyvista")
+@pytest.mark.skipif(not HAS_GRAPHVIZ, reason="Please install graphviz")
 def test_workflow_view(server_in_process, remove_dot_file):
     pre_wf = dpf.core.Workflow(server=server_in_process)
     pre_op = dpf.core.operators.utility.forward(server=server_in_process)