Stack walk tests: handle gracefully if assertions are not enabled

Author: steve-s

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

@@ -56,8 +56,17 @@
 STABILIZES_AT = 1 if sys.implementation.name != "graalpy" or __graalpython__.truffle_runtime == 'Interpreted' else 10
 STABILIZES_AT = int(os.environ.get('GRAALPY_TEST_INDIRECT_CALL_STABILIZES_AT', STABILIZES_AT))
 
+has_stack_walk_check = False
+if sys.implementation.name == "graalpy":
+    result = __graalpython__.was_stack_walk(False)
+    if result is None:
+        print("NOTE: assertions are not enabled; test_indirect_call cannot check for "
+              "repeated stack walks and will perform only basic sanity checks")
+    else:
+        has_stack_walk_check = True
+
 def was_stack_walk(new_value):
-    if sys.implementation.name == "graalpy":
+    if has_stack_walk_check:
         return __graalpython__.was_stack_walk(new_value)
     return False
 

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/GraalPythonModuleBuiltins.java

@@ -637,8 +637,9 @@ public Object doBoundary(Object value) {
             boolean assertionsEnabled = false;
             assert assertionsEnabled = true;
             if (!assertionsEnabled) {
-                // It does not make sense to run with if assertions are not enabled
-                throw PRaiseNode.raiseStatic(this, PythonBuiltinClassType.SystemError);
+                // None indicates that assertions are disabled and this functionality is not
+                // available
+                return PNone.NONE;
             }
 
             boolean prev = PythonContext.get(this).wasStackWalk;