[GR-38700] GraalPy Python unit tests: dedicated helpers to ignore Bytecode DSL/Manual interpreter tests.

ci/python-gate.libsonnet

@@ -467,6 +467,7 @@
             // logging
             ["mx"] + self.mx_parameters + self.dy + ["sversions"],
         ],
+        run+: [ ["sleep", "30m"] ],
         on_success+: [
             ["rm", "-rf", "graal_dumps"],
         ],

graalpython/com.oracle.graal.python.test/src/tests/__init__.py

@@ -78,8 +78,8 @@ def get_setuptools(setuptools='setuptools==67.6.1'):
         else:
             py_executable = setuptools_path / 'bin' / 'python3'
         extra_args = []
-        if sys.implementation.name == "graalpy" and not system_python and __graalpython__.is_bytecode_dsl_interpreter:
-            extra_args = ['--vm.Dpython.EnableBytecodeDSLInterpreter=true']
+        if sys.implementation.name == "graalpy" and not system_python and not __graalpython__.is_native:
+            extra_args += [f'--vm.Dpython.EnableBytecodeDSLInterpreter={str(__graalpython__.is_bytecode_dsl_interpreter).lower()}']
         subprocess.run([py_executable, *extra_args, "-m", "pip", "install", "--target", str(setuptools_path), setuptools], check=True)
         print('setuptools is installed in %s' % setuptools_path)
 

graalpython/com.oracle.graal.python.test/src/tests/cpyext/test_shutdown.py

@@ -1,4 +1,4 @@
-# Copyright (c) 2023, 2024, Oracle and/or its affiliates. All rights reserved.
+# Copyright (c) 2023, 2025, Oracle and/or its affiliates. All rights reserved.
 # DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
 #
 # The Universal Permissive License (UPL), Version 1.0
@@ -51,8 +51,8 @@
 ARGS = []
 if sys.implementation.name == 'graalpy':
     ARGS = ['--experimental-options', '--python.EnableDebuggingBuiltins']
-    if not __graalpython__.is_native and __graalpython__.is_bytecode_dsl_interpreter:
-        ARGS += ['--vm.Dpython.EnableBytecodeDSLInterpreter=true']
+    if not __graalpython__.is_native:
+        ARGS += [f'--vm.Dpython.EnableBytecodeDSLInterpreter={str(__graalpython__.is_bytecode_dsl_interpreter).lower()}']
 COMMAND = [sys.executable, *ARGS, str(MODULE_PATH)]
 
 

graalpython/com.oracle.graal.python.test/src/tests/test_frame_tests.py

@@ -38,15 +38,15 @@
 # SOFTWARE.
 import os
 import sys
-
+from tests import util
 
 # IMPORTANT: DO NOT MOVE!
 # This test checks that lineno works on frames,
 # it MUST stay on this line!
 def test_lineno():
     assert sys._getframe(0).f_lineno == 47
 
-if not os.environ.get('BYTECODE_DSL_INTERPRETER'): # Blocked by GR-61955
+if not util.IS_BYTECODE_DSL: # Blocked by GR-61955
     # IMPORTANT: DO NOT MOVE!
     def test_nested_lineno():
         def test_nested():

graalpython/com.oracle.graal.python.test/src/tests/test_generators.py

@@ -6,6 +6,7 @@
 import sys
 import os
 import unittest
+from tests import util
 
 
 class ExceptionTest(unittest.TestCase):
@@ -557,7 +558,7 @@ def gen():
         assert f.f_globals is globals()
         assert f.f_locals == {'a': 1, 'b': 2, 'c': 3}
         # TODO GR-61955
-        if sys.implementation.name != "graalpy" or not __graalpython__.is_bytecode_dsl_interpreter:
+        if not util.IS_BYTECODE_DSL:
             assert f.f_lineno == gen.__code__.co_firstlineno + 5
     assert not g.gi_frame
 
@@ -591,7 +592,7 @@ def gen():
     assert f.f_globals is globals()
     assert f.f_locals == {'a': 1, 'b': 2}
     # TODO GR-61955
-    if sys.implementation.name != "graalpy" or not __graalpython__.is_bytecode_dsl_interpreter:
+    if not util.IS_BYTECODE_DSL:
         assert f.f_lineno == gen.__code__.co_firstlineno + 3
 
 

graalpython/com.oracle.graal.python.test/src/tests/test_pdb.py

@@ -41,6 +41,7 @@
 import doctest
 import sys
 import unittest
+from tests import util
 
 
 # Copied from test_doctest
@@ -69,8 +70,6 @@ def __exit__(self, *exc):
         if self.orig_trace:
             sys.settrace(self.orig_trace)
 
-
-@unittest.skipIf(os.environ.get('BYTECODE_DSL_INTERPRETER'), "TODO: FrameSlotTypeException with reparsing")
 def doctest_pdb_locals():
     """
     Test that locals get synced after breakpoint
@@ -101,63 +100,61 @@ def doctest_pdb_locals():
     """
 
 
-@unittest.skipIf(os.environ.get('BYTECODE_DSL_INTERPRETER'), "TODO: FrameSlotTypeException with reparsing")
-def doctest_pdb_locals_generator():
-    """
-    Test that locals get synced after breakpoint in a generator
-
-    >>> def test_function():
-    ...     a = 1
-    ...     import pdb; pdb.Pdb(nosigint=True, readrc=False).set_trace()
-    ...     a = 2
-    ...     yield
-
-    >>> with PdbTestInput([
-    ...     'p a',
-    ...     'next',
-    ...     'p a',
-    ...     'continue',
-    ... ]):
-    ...     next(test_function())
-    > <doctest tests.test_pdb.doctest_pdb_locals_generator[0]>(4)test_function()
-    -> a = 2
-    (Pdb) p a
-    1
-    (Pdb) next
-    > <doctest tests.test_pdb.doctest_pdb_locals_generator[0]>(5)test_function()
-    -> yield
-    (Pdb) p a
-    2
-    (Pdb) continue
-    """
-
-
-@unittest.skipIf(os.environ.get('BYTECODE_DSL_INTERPRETER'), "TODO: FrameSlotTypeException with reparsing")
-def doctest_pdb_locals_sync_back():
-    """
-    Test that locals set by debugger get propagated back into the frame.
-
-    >>> def test_function():
-    ...     foo = 1
-    ...     import pdb; pdb.Pdb(nosigint=True, readrc=False).set_trace()
-    ...     return foo
-
-    >>> with PdbTestInput([
-    ...     'p foo',
-    ...     'foo = 5',
-    ...     'continue',
-    ... ]):
-    ...     print(test_function())
-    > <doctest tests.test_pdb.doctest_pdb_locals_sync_back[0]>(4)test_function()
-    -> return foo
-    (Pdb) p foo
-    1
-    (Pdb) foo = 5
-    (Pdb) continue
-    5
-    """
+if not util.IS_BYTECODE_DSL:
+    def doctest_pdb_locals_generator():
+        """
+        Test that locals get synced after breakpoint in a generator
+
+        >>> def test_function():
+        ...     a = 1
+        ...     import pdb; pdb.Pdb(nosigint=True, readrc=False).set_trace()
+        ...     a = 2
+        ...     yield
+
+        >>> with PdbTestInput([
+        ...     'p a',
+        ...     'next',
+        ...     'p a',
+        ...     'continue',
+        ... ]):
+        ...     next(test_function())
+        > <doctest tests.test_pdb.doctest_pdb_locals_generator[0]>(4)test_function()
+        -> a = 2
+        (Pdb) p a
+        1
+        (Pdb) next
+        > <doctest tests.test_pdb.doctest_pdb_locals_generator[0]>(5)test_function()
+        -> yield
+        (Pdb) p a
+        2
+        (Pdb) continue
+        """
+
+
+    def doctest_pdb_locals_sync_back():
+        """
+        Test that locals set by debugger get propagated back into the frame.
+
+        >>> def test_function():
+        ...     foo = 1
+        ...     import pdb; pdb.Pdb(nosigint=True, readrc=False).set_trace()
+        ...     return foo
+
+        >>> with PdbTestInput([
+        ...     'p foo',
+        ...     'foo = 5',
+        ...     'continue',
+        ... ]):
+        ...     print(test_function())
+        > <doctest tests.test_pdb.doctest_pdb_locals_sync_back[0]>(4)test_function()
+        -> return foo
+        (Pdb) p foo
+        1
+        (Pdb) foo = 5
+        (Pdb) continue
+        5
+        """
 
 
-@unittest.skipIf(os.environ.get('BYTECODE_DSL_INTERPRETER'), "TODO: FrameSlotTypeException with reparsing")
 def test_run_doctests():
     doctest.testmod(sys.modules[__name__], raise_on_error=True)

graalpython/com.oracle.graal.python.test/src/tests/test_repl.py

@@ -45,6 +45,7 @@
 import sys
 import tempfile
 import unittest
+from tests import util
 from dataclasses import dataclass
 from textwrap import dedent
 
@@ -191,7 +192,7 @@ def test_continuation():
         '''))
 
 
-    @unittest.skipIf(os.environ.get('BYTECODE_DSL_INTERPRETER'), "TODO: <interactive> vs <module> in traceback")
+    @util.skipIfBytecodeDSL("TODO: <interactive> vs <module> in traceback")
     def test_exceptions():
         validate_repl(dedent("""\
             >>> 1 / 0