test: update test_deprecated_params to include make_block fixture to avoid plotting empty meshes error (#4168)

* test: update test_deprecated_params to include make_block fixture to avoid plotting empty meshes error

* chore: adding changelog file 4168.miscellaneous.md [dependabot-skip]

* fix: remove allow_empty_mesh setting from global_theme to prevent empty mesh plotting.

---------

Co-authored-by: pyansys-ci-bot <[email protected]>

Author: germa89

doc/changelog.d/4168.miscellaneous.md

@@ -0,0 +1 @@
+Test: update test_deprecated_params to include make_block fixture to avoid plotting empty meshes error

tests/conftest.py

@@ -218,7 +218,6 @@ def requires_dependency(dependency: str):
 
     # Necessary for CI plotting
     pyvista.OFF_SCREEN = True
-    pyvista.global_theme.allow_empty_mesh = True
 
     from ansys.mapdl.core.plotting.theme import _apply_default_theme
 

tests/test_plotting.py

@@ -34,9 +34,6 @@
         allow_module_level=True, reason="Skipping because 'pyvista' is not installed"
     )
 
-import pyvista
-
-pyvista.global_theme.allow_empty_mesh = True
 
 from ansys.mapdl.core.errors import ComponentDoesNotExits, MapdlRuntimeError
 from ansys.mapdl.core.plotting import GraphicsBackend
@@ -1339,7 +1336,7 @@ def test_plot_path_screenshoot(mapdl, cleared, tmpdir):
     mapdl.graphics("FULL")
 
 
-def test_deprecated_params(mapdl):
+def test_deprecated_params(mapdl, make_block):
     with pytest.warns(DeprecationWarning, match="'vtk' and 'use_vtk' are deprecated"):
         mapdl.eplot(vtk=True)
     with pytest.warns(DeprecationWarning, match="'vtk' and 'use_vtk' are deprecated"):