pythongh-117968: Make the test for closed file more safe in the C API tests (pythonGH-118230)

serhiy-storchaka · miss-islington · commit 5b2b6912f016 · 2024-04-25T05:00:49.000Z
The behavior of fileno() after fclose() is undefined, but it is the only practical way to check whether the file was closed. Only test this on the known platforms (Linux, Windows, macOS), where we already tested that it works. (cherry picked from commit 546cbcf) Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
diff --git a/Modules/_testcapi/run.c b/Modules/_testcapi/run.c
@@ -75,8 +75,10 @@ run_fileexflags(PyObject *mod, PyObject *pos_args)
 
     result = PyRun_FileExFlags(fp, filename, start, globals, locals, closeit, pflags);
 
-#if !defined(__wasi__)
-    /* The behavior of fileno() after fclose() is undefined. */
+#if defined(__linux__) || defined(MS_WINDOWS) || defined(__APPLE__)
+    /* The behavior of fileno() after fclose() is undefined, but it is
+     * the only practical way to check whether the file was closed.
+     * Only test this on the known platforms. */
     if (closeit && result && fileno(fp) >= 0) {
         PyErr_SetString(PyExc_AssertionError, "File was not closed after excution");
         Py_DECREF(result);
