Match SymbolTable repr with CPython format (RustPython#7139)

moreal · web-flow · commit e8698aa19f9e · 2026-02-14T22:59:42.000+09:00
diff --git a/Lib/test/test_symtable.py b/Lib/test/test_symtable.py
@@ -543,7 +543,6 @@ def test_symbol_repr(self):
         self.assertEqual(repr(class_A.lookup('x')),
                          "<symbol 'x': LOCAL, DEF_LOCAL|DEF_FREE_CLASS>")
 
-    @unittest.expectedFailure  # TODO: RUSTPYTHON
     def test_symtable_entry_repr(self):
         expected = f"<symtable entry top({self.top.get_id()}), line {self.top.get_lineno()}>"
         self.assertEqual(repr(self.top._table), expected)
diff --git a/crates/vm/src/stdlib/symtable.rs b/crates/vm/src/stdlib/symtable.rs
@@ -3,9 +3,10 @@ pub(crate) use _symtable::module_def;
 #[pymodule]
 mod _symtable {
     use crate::{
-        PyObjectRef, PyPayload, PyRef, PyResult, VirtualMachine,
+        Py, PyObjectRef, PyPayload, PyRef, PyResult, VirtualMachine,
         builtins::{PyDictRef, PyStrRef},
         compiler,
+        types::Representable,
     };
     use alloc::fmt;
     use rustpython_codegen::symboltable::{
@@ -132,7 +133,7 @@ mod _symtable {
     }
 
     #[pyattr]
-    #[pyclass(name = "SymbolTable")]
+    #[pyclass(name = "symtable entry")]
     #[derive(PyPayload)]
     struct PySymbolTable {
         symtable: SymbolTable,
@@ -144,7 +145,7 @@ mod _symtable {
         }
     }
 
-    #[pyclass]
+    #[pyclass(with(Representable))]
     impl PySymbolTable {
         #[pygetset]
         fn name(&self) -> String {
@@ -210,6 +211,19 @@ mod _symtable {
         }
     }
 
+    impl Representable for PySymbolTable {
+        #[inline]
+        fn repr_str(zelf: &Py<Self>, vm: &VirtualMachine) -> PyResult<String> {
+            Ok(format!(
+                "<{} {}({}), line {}>",
+                Self::class(&vm.ctx).name(),
+                zelf.symtable.name,
+                zelf.id(),
+                zelf.symtable.line_number
+            ))
+        }
+    }
+
     #[pyattr]
     #[pyclass(name = "Symbol")]
     #[derive(PyPayload)]
