Skip to content

Commit

Permalink
GH-111798: skip test_super_deep() from test_call under pydebug bu…
Browse files Browse the repository at this point in the history
…ilds on WASI (GH-114010)
  • Loading branch information
brettcannon committed Jan 13, 2024
1 parent 3aa4b83 commit dac1da2
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
10 changes: 4 additions & 6 deletions Lib/test/test_call.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
import unittest
from test.support import cpython_only, requires_limited_api, skip_on_s390x
from test.support import (cpython_only, is_wasi, requires_limited_api, Py_DEBUG,
set_recursion_limit, skip_on_s390x)
try:
import _testcapi
except ImportError:
Expand Down Expand Up @@ -990,6 +991,7 @@ def case_change_over_substitution(BLuch=None, Luch = None, fluch = None):
class TestRecursion(unittest.TestCase):

@skip_on_s390x
@unittest.skipIf(is_wasi and Py_DEBUG, "requires deep stack")
def test_super_deep(self):

def recurse(n):
Expand All @@ -1010,9 +1012,7 @@ def c_py_recurse(m):
if m:
_testcapi.pyobject_vectorcall(py_recurse, (1000, m), ())

depth = sys.getrecursionlimit()
sys.setrecursionlimit(100_000)
try:
with set_recursion_limit(100_000):
recurse(90_000)
with self.assertRaises(RecursionError):
recurse(101_000)
Expand All @@ -1022,8 +1022,6 @@ def c_py_recurse(m):
c_py_recurse(90)
with self.assertRaises(RecursionError):
c_py_recurse(100_000)
finally:
sys.setrecursionlimit(depth)


class TestFunctionWithManyArgs(unittest.TestCase):
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
Disable ``test_super_deep()`` from ``test_call`` under pydebug builds on
WASI; the stack depth is too small to make the test useful.

0 comments on commit dac1da2

Please sign in to comment.