]> granicus.if.org Git - graphviz/commitdiff
move #1786 test case into CI-only tests
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Wed, 9 Feb 2022 11:50:23 +0000 (22:50 +1100)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Tue, 15 Feb 2022 20:38:41 +0000 (07:38 +1100)
In retrospect, this test case does not make sense in the regular test suite. It
is validating a particular plugin feature is enabled. It is perfectly valid to
build a version of Graphviz without png:gd support, yet it will fail this test
case.

ci/tests.py
rtest/test_regression.py

index d1c21000e00928268215b3de0be6245637642c69..cd6626b99953d64564cfd0272efbe4d9088629db 100644 (file)
@@ -7,8 +7,12 @@ test cases that are only relevant to run in CI
 import os
 import platform
 import shutil
+import sys
 import pytest
 
+sys.path.append(os.path.join(os.path.dirname(__file__), "../rtest"))
+from gvtest import dot #pylint: disable=C0413
+
 @pytest.mark.parametrize("binary", [
   "acyclic",
   "bcomps",
@@ -138,3 +142,13 @@ def check_that_tool_does_not_exist(tool, os_id):
   """
   assert shutil.which(tool) is None, f"{tool} has been resurrected in the " \
     f'{os.getenv("build_system")} build on {os_id}. Please remove skip.'
+
+@pytest.mark.xfail(strict=os.getenv("build_system") == "cmake") # FIXME
+def test_1786():
+  """
+  png:gd format should be supported
+  https://gitlab.com/graphviz/graphviz/-/issues/1786
+  """
+
+  # run a trivial graph through Graphviz
+  dot("png:gd", source="digraph { a -> b; }")
index 01128fa9f28347fe3d202d1ac92fd43031a944b6..f85396fca1dd5e86404fc2701e1d7dcb65e8d33f 100644 (file)
@@ -627,16 +627,6 @@ def test_1783():
 
   assert ret != -signal.SIGSEGV, "Graphviz segfaulted"
 
-@pytest.mark.xfail(strict=os.getenv("build_system") == "cmake") # FIXME
-def test_1786():
-  """
-  png:gd format should be supported
-  https://gitlab.com/graphviz/graphviz/-/issues/1786
-  """
-
-  # run a trivial graph through Graphviz
-  dot("png:gd", source="digraph { a -> b; }")
-
 @pytest.mark.skipif(shutil.which("gvedit") is None, reason="Gvedit not available")
 def test_1813():
   """