Fix cron-ci failures: ctypes set_attr, missing features, __func__ AttributeError

- Use cls.set_attr() instead of cls.as_object().set_attr() in ctypes
  to ensure modified() is called and type cache stays valid
- Add host_env feature to cron-ci.yaml for frozen_origname_matches test
- Add stdio feature to cron-ci.yaml for encodings initialization
- Fix __func__ AttributeError in custom_text_test_runner.py

Author: youknowone

.github/workflows/cron-ci.yaml

@@ -12,7 +12,7 @@ on:
 name: Periodic checks/tasks
 
 env:
-  CARGO_ARGS: --no-default-features --features stdlib,importlib,encodings,ssl-rustls,jit
+  CARGO_ARGS: --no-default-features --features stdlib,importlib,stdio,encodings,ssl-rustls,jit,host_env
   PYTHON_VERSION: "3.14.3"
 
 jobs:
@@ -35,7 +35,7 @@ jobs:
           python-version: ${{ env.PYTHON_VERSION }}
       - run: sudo apt-get update && sudo apt-get -y install lcov
       - name: Run cargo-llvm-cov with Rust tests.
-        run: cargo llvm-cov --no-report --workspace --exclude rustpython_wasm --exclude rustpython-compiler-source --exclude rustpython-venvlauncher --verbose --no-default-features --features stdlib,importlib,encodings,ssl-rustls,jit
+        run: cargo llvm-cov --no-report --workspace --exclude rustpython_wasm --exclude rustpython-compiler-source --exclude rustpython-venvlauncher --verbose --no-default-features --features stdlib,importlib,stdio,encodings,ssl-rustls,jit
       - name: Run cargo-llvm-cov with Python snippets.
         run: python scripts/cargo-llvm-cov.py
         continue-on-error: true
@@ -170,12 +170,12 @@ jobs:
       - name: restructure generated files
         run: |
           cd ./target/criterion/reports
-          find -type d -name cpython | xargs rm -rf
-          find -type d -name rustpython | xargs rm -rf
-          find -mindepth 2 -maxdepth 2 -name violin.svg | xargs rm -rf
-          find -type f -not -name violin.svg | xargs rm -rf
-          for file in $(find -type f -name violin.svg); do mv $file $(echo $file | sed -E "s_\./([^/]+)/([^/]+)/violin\.svg_./\1/\2.svg_"); done
-          find -mindepth 2 -maxdepth 2 -type d | xargs rm -rf
+          find . -type d -name cpython | xargs rm -rf
+          find . -type d -name rustpython | xargs rm -rf
+          find . -mindepth 2 -maxdepth 2 -name violin.svg | xargs rm -rf
+          find . -type f -not -name violin.svg | xargs rm -rf
+          for file in $(find . -type f -name violin.svg); do mv "$file" $(echo "$file" | sed -E "s_\./([^/]+)/([^/]+)/violin\.svg_./\1/\2.svg_"); done
+          find . -mindepth 2 -maxdepth 2 -type d | xargs rm -rf
           cd ..
           mv reports/* .
           rmdir reports

crates/vm/src/stdlib/_ctypes/base.rs

@@ -2554,11 +2554,10 @@ fn make_fields(
         }
 
         let new_descr = super::PyCField::new_from_field(fdescr, index, offset);
-        cls.as_object().set_attr(
+        cls.set_attr(
             vm.ctx.intern_str(fname.as_wtf8()),
             new_descr.to_pyobject(vm),
-            vm,
-        )?;
+        );
     }
 
     Ok(())
@@ -2597,11 +2596,10 @@ pub(super) fn make_anon_fields(cls: &Py<PyType>, vm: &VirtualMachine) -> PyResul
 
         let mut new_descr = super::PyCField::new_from_field(descr, 0, 0);
         new_descr.set_anonymous(true);
-        cls.as_object().set_attr(
+        cls.set_attr(
             vm.ctx.intern_str(fname.as_wtf8()),
             new_descr.to_pyobject(vm),
-            vm,
-        )?;
+        );
 
         make_fields(cls, descr, descr.index, descr.offset, vm)?;
     }

crates/vm/src/stdlib/_ctypes/structure.rs

@@ -498,11 +498,7 @@ impl PyCStructType {
             };
 
             // Set the CField as a class attribute
-            cls.as_object().set_attr(
-                vm.ctx.intern_str(name.clone()),
-                c_field.to_pyobject(vm),
-                vm,
-            )?;
+            cls.set_attr(vm.ctx.intern_str(name.clone()), c_field.to_pyobject(vm));
 
             // Update tracking - don't advance offset for packed bitfields
             if field_advances_offset {

crates/vm/src/stdlib/_ctypes/union.rs

@@ -312,8 +312,7 @@ impl PyCUnionType {
                 PyCField::new(name.clone(), field_type_ref, 0, size as isize, index)
             };
 
-            cls.as_object()
-                .set_attr(vm.ctx.intern_str(name), c_field.to_pyobject(vm), vm)?;
+            cls.set_attr(vm.ctx.intern_str(name), c_field.to_pyobject(vm));
         }
 
         // Calculate total_align and aligned_size

extra_tests/custom_text_test_runner.py

@@ -38,6 +38,16 @@
 from unittest.runner import registerResult, result
 
 
+def _get_method_dict(test):
+    """Get the __dict__ of the underlying function for a test method.
+
+    Works for both bound methods (__func__.__dict__) and plain functions.
+    """
+    method = getattr(test, test._testMethodName)
+    func = getattr(method, '__func__', method)
+    return func.__dict__
+
+
 class TablePrinter(object):
     # Modified from https://github.com/agramian/table-printer, same license as above
     "Print a list of dicts as a table"
@@ -325,7 +335,7 @@ def startTest(self, test):
                 self.stream.writeln("TEST SUITE: %s" % self.suite)
                 self.stream.writeln("Description: %s" % self.getSuiteDescription(test))
         try:
-            name_override = getattr(test, test._testMethodName).__func__.__dict__[
+            name_override = _get_method_dict(test)[
                 "test_case_name"
             ]
         except:
@@ -382,30 +392,30 @@ def startTest(self, test):
             ).__func__.__dict__ and set([s.lower() for s in self.test_types]) == set(
                 [
                     s.lower()
-                    for s in getattr(test, test._testMethodName).__func__.__dict__[
+                    for s in _get_method_dict(test)[
                         "test_type"
                     ]
                 ]
             ):
                 pass
             else:
-                getattr(test, test._testMethodName).__func__.__dict__[
+                _get_method_dict(test)[
                     "__unittest_skip_why__"
                 ] = 'Test run specified to only run tests of type "%s"' % ",".join(
                     self.test_types
                 )
-                getattr(test, test._testMethodName).__func__.__dict__[
+                _get_method_dict(test)[
                     "__unittest_skip__"
                 ] = True
-        if "skip_device" in getattr(test, test._testMethodName).__func__.__dict__:
-            for device in getattr(test, test._testMethodName).__func__.__dict__[
+        if "skip_device" in _get_method_dict(test):
+            for device in _get_method_dict(test)[
                 "skip_device"
             ]:
                 if self.config and device.lower() in self.config["device_name"].lower():
-                    getattr(test, test._testMethodName).__func__.__dict__[
+                    _get_method_dict(test)[
                         "__unittest_skip_why__"
                     ] = "Test is marked to be skipped on %s" % device
-                    getattr(test, test._testMethodName).__func__.__dict__[
+                    _get_method_dict(test)[
                         "__unittest_skip__"
                     ] = True
                     break