untag tests not working without stdout/stderr flush

Author: timfel

graalpython/com.oracle.graal.python.test/src/tests/unittest_tags/test_warnings.txt

@@ -5,10 +5,6 @@
 *graalpython.lib-python.3.test.test_warnings.__init__.CCatchWarningTests.test_check_warnings
 *graalpython.lib-python.3.test.test_warnings.__init__.CCatchWarningTests.test_record_override_showwarning_before
 *graalpython.lib-python.3.test.test_warnings.__init__.CCatchWarningTests.test_record_override_showwarning_inside
-*graalpython.lib-python.3.test.test_warnings.__init__.CEnvironmentVariableTests.test_comma_separated_warnings
-*graalpython.lib-python.3.test.test_warnings.__init__.CEnvironmentVariableTests.test_envvar_and_command_line
-*graalpython.lib-python.3.test.test_warnings.__init__.CEnvironmentVariableTests.test_nonascii
-*graalpython.lib-python.3.test.test_warnings.__init__.CEnvironmentVariableTests.test_single_warning
 *graalpython.lib-python.3.test.test_warnings.__init__.CFilterTests.test_always
 *graalpython.lib-python.3.test.test_warnings.__init__.CFilterTests.test_always_after_default
 *graalpython.lib-python.3.test.test_warnings.__init__.CFilterTests.test_append_duplicate
@@ -48,10 +44,6 @@
 *graalpython.lib-python.3.test.test_warnings.__init__.PyCatchWarningTests.test_check_warnings
 *graalpython.lib-python.3.test.test_warnings.__init__.PyCatchWarningTests.test_record_override_showwarning_before
 *graalpython.lib-python.3.test.test_warnings.__init__.PyCatchWarningTests.test_record_override_showwarning_inside
-*graalpython.lib-python.3.test.test_warnings.__init__.PyEnvironmentVariableTests.test_comma_separated_warnings
-*graalpython.lib-python.3.test.test_warnings.__init__.PyEnvironmentVariableTests.test_envvar_and_command_line
-*graalpython.lib-python.3.test.test_warnings.__init__.PyEnvironmentVariableTests.test_nonascii
-*graalpython.lib-python.3.test.test_warnings.__init__.PyEnvironmentVariableTests.test_single_warning
 *graalpython.lib-python.3.test.test_warnings.__init__.PyFilterTests.test_always
 *graalpython.lib-python.3.test.test_warnings.__init__.PyFilterTests.test_always_after_default
 *graalpython.lib-python.3.test.test_warnings.__init__.PyFilterTests.test_append_duplicate

graalpython/lib-graalpython/pyio_patches.py

@@ -96,20 +96,19 @@ def open(*args, **kwargs):
 sys.__stderr__ = sys.stderr
 
 
-# Try to close the std streams when we exit
-import atexit
-def close_stdouts(so=sys.stdout, se=sys.stderr):
-    try:
-        so.close()
-    except:
-        pass
-    try:
-        se.close()
-    except:
-        pass
-
-
-atexit.register(close_stdouts)
+# Try to close the std streams when we exit. This currently doesn't work due to GR-25185
+# To make this work reliably, we probably have to implement the _io module in Java
+# import atexit
+# def close_stdouts(so=sys.stdout, se=sys.stderr):
+#     try:
+#         so.close()
+#     except:
+#         pass
+#     try:
+#         se.close()
+#     except:
+#         pass
+# atexit.register(close_stdouts)
 
 
 # See comment in _pyio.py. This method isn't strictly necessary and is provided