build: remove mentions of deprecated pyansys-tools-versioning package (#4380)

* build: remove mentions of deprecated pyansys-tools-versioning package

* chore: adding changelog file 4380.dependencies.md [dependabot-skip]

* chore: empty commit

* build: add build-test-ubuntu-minimal to job dependencies

* build: update import path for server_meets_version to use common versioning

* build: add missing dependency for tabulate and improve assertions in tests

* build: mark flaky tests in test_grpc.py and test_launcher.py

---------

Co-authored-by: pyansys-ci-bot <92810346+pyansys-ci-bot@users.noreply.github.com>

Author: germa89

.github/dependabot.yml

@@ -45,7 +45,6 @@ updates:
             - "numpy"
             - "platformdirs"
             - "psutil"
-            - "pyansys-tools-versioning"
 
        grpc-deps:
           patterns:
@@ -70,7 +69,6 @@ updates:
             - "ansys-api-mapdl"
             - "ansys-dpf-core"
             - "pyansys-tools-report"
-            - "pyansys-tools-versioning"
 
        documentation:
           patterns:

.github/workflows/ci.yml

@@ -355,7 +355,7 @@ jobs:
 
   package:
     name: "Package library"
-    needs: [build-test-remote, build-test-ubuntu-local, docs-build] # TODO: add build-test-ubuntu-minimal when re-enabled
+    needs: [build-test-remote, build-test-ubuntu-local, docs-build, build-test-ubuntu-minimal]
     runs-on: ubuntu-latest
     permissions:
       contents: read  # Needed to read repository contents for packaging
@@ -440,7 +440,7 @@ jobs:
 
   notify:
     name: "Notify failed build"
-    needs: [smoke-tests, docs-build, build-test-remote, build-test-ubuntu-local] # TODO: add build-test-ubuntu-minimal when re-enabled
+    needs: [smoke-tests, docs-build, build-test-remote, build-test-ubuntu-local, build-test-ubuntu-minimal]
     if: failure() && github.event_name == 'schedule'
     runs-on: ubuntu-latest
     permissions:
@@ -480,8 +480,8 @@ jobs:
   pytest-summary:
     name: Pytest summary for all the test jobs
     needs: [
-      build-test-remote, build-test-ubuntu-local, build-test-ubuntu-console
-      ] # TODO: add build-test-ubuntu-minimal when re-enabled
+      build-test-remote, build-test-ubuntu-local, build-test-ubuntu-console, build-test-ubuntu-minimal
+      ]
     if: always()
     runs-on: ubuntu-latest
     permissions:

doc/changelog.d/4380.dependencies.md

@@ -0,0 +1 @@
+Remove mentions of deprecated pyansys-tools-versioning package

src/ansys/mapdl/core/database/database.py

@@ -29,7 +29,7 @@
 import weakref
 
 from ansys.api.mapdl.v0 import mapdl_db_pb2_grpc
-from ansys.tools.versioning import server_meets_version
+from ansys.tools.common.versioning import server_meets_version
 import grpc
 
 from ansys.mapdl.core.errors import MapdlConnectionError

src/ansys/mapdl/core/mapdl_grpc.py

@@ -41,7 +41,7 @@
 from warnings import warn
 import weakref
 
-from ansys.tools.versioning.utils import version_string_as_tuple
+from ansys.tools.common.versioning import version_string_as_tuple
 import grpc
 from grpc._channel import _InactiveRpcError, _MultiThreadedRendezvous
 import numpy as np

src/ansys/mapdl/core/plotting/theme.py

@@ -52,7 +52,7 @@
     PyMAPDL_cmap: ListedColormap = ListedColormap(MAPDL_colorbar, name="PyMAPDL")
 
 if _HAS_PYVISTA:
-    from ansys.tools.versioning.utils import version_string_as_tuple  # type: ignore
+    from ansys.tools.common.versioning import version_string_as_tuple  # type: ignore
     from cycler import Cycler
     from pyvista import __version__ as pyvista_version
     from pyvista.plotting.colors import get_cycler  # type: ignore

tests/test_cli.py

@@ -331,6 +331,7 @@ def mock_kill_process(proc: psutil.Process):
 
 
 @requires("click")
+@requires("tabulate")
 @pytest.mark.parametrize(
     "arg,check",
     (

tests/test_database.py

@@ -23,7 +23,7 @@
 import os
 import re
 
-from ansys.tools.versioning import server_meets_version
+from ansys.tools.common.versioning import server_meets_version
 import numpy as np
 import pytest
 

tests/test_grpc.py

@@ -443,6 +443,7 @@ def test_download_project_extensions(mapdl, cleared, tmpdir):
     assert expected.intersection(files_extensions) == {"log", "err"}
 
 
+@pytest.mark.xfail(strict=False, reason="Flaky test")
 def test_download_result(mapdl, cleared, tmpdir):
     if "file.rst" not in mapdl.list_files():
         write_tmp_in_mapdl_instance(mapdl, "file", ext="rst")  # fake rst file

tests/test_krylov.py

@@ -23,7 +23,7 @@
 """Tests comparing results of krylov pymadl function with apdl macro"""
 import os
 
-from ansys.tools.versioning.utils import server_meets_version
+from ansys.tools.common.versioning import server_meets_version
 import numpy as np
 import pytest