]> granicus.if.org Git - graphviz/commitdiff
move some pytest skip logic into a decorator
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 25 Oct 2020 02:53:06 +0000 (19:53 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 25 Oct 2020 18:47:27 +0000 (11:47 -0700)
For more consistency with other tests.

rtest/test_examples.py

index 364b1a314f0d85f3677b542bf24b1bde0e303292..1d6de61ce66323e534d6042deae8b4be929b0c94 100644 (file)
@@ -70,13 +70,10 @@ def test_compile_example(src):
   'addranks', 'anon', 'bb', 'chkclusters', 'cliptree', 'col', 'color',
   'dechain', 'deledges', 'delnodes', 'dijkstra', 'get-layers-list', 'knbhd',
   'maxdeg', 'rotate', 'scalexy', 'topon'])
+@pytest.mark.skipif(shutil.which('gvpr') is None, reason='GVPR not available')
 def test_gvpr_example(src):
     '''check GVPR can parse the given example'''
 
-    # skip this test if GVPR is unavailable
-    if shutil.which('gvpr') is None:
-      pytest.skip('GVPR not available')
-
 # FIXME: remove when https://gitlab.com/graphviz/graphviz/-/issues/1784 is fixed
     if os.environ.get('build_system') == 'msbuild' and \
       os.environ.get('configuration') == 'Debug' and \