]> granicus.if.org Git - graphviz/commitdiff
Skip test_large_graphs on win32.
authorMark Hansen <mark@markhansen.co.nz>
Fri, 22 May 2020 01:52:36 +0000 (11:52 +1000)
committerMark Hansen <mark@markhansen.co.nz>
Fri, 22 May 2020 08:07:12 +0000 (18:07 +1000)
This test is expected to fail on win32 due to Window's smaller stack
size defaults:

- Windows by default uses 1MB stacks:
  https://docs.microsoft.com/en-us/windows/win32/procthread/thread-stack-size

- Linux x86_64 seems to usually have a default of 8MB (architecture
dependent):

  ```
  mark@x86_64_linux $ ulimit -a | grep stack
  Maximum stack size (kB, -s) 8192

  mark@arm32v6 $ ulimit -a | grep stack
  Maximum stack size (kB, -s) 8192
  ```

This test failure is not a regression (it would have been failing before
81c3a3af), and it's blocking other submissions. This change gets the CI
green again to unblock forward progress.

tests/regression_tests/large/test_large_graphs.py

index 7294d89cc40a8fb57a111ce67d16c057e1e46b2a..5a94fbb957a697aef7c282588969f8b44acaed0a 100644 (file)
@@ -1,13 +1,15 @@
-from unittest import TestCase\r
+from unittest import TestCase, skipIf\r
 \r
 import subprocess\r
 import os\r
+import sys\r
 \r
 \r
 TEST_DIR = os.path.dirname(os.path.realpath(__file__))\r
 \r
 \r
 class TestLargeGraphs(TestCase):\r
+       @skipIf(sys.platform == 'win32', reason='https://gitlab.com/graphviz/graphviz/-/issues/1710')\r
        def test_long_chain(self):\r
                """\r
                This test will fail on 32bit Windows machine if compiled with stack size < 16MB.\r