internal docs and coverage

Author: davidhewitt

src/impl_/pyclass.rs

@@ -1236,7 +1236,7 @@ impl<
         const IMPLEMENTS_TOPYOBJECT: bool,
     > PyClassGetterGenerator<ClassT, Py<U>, Offset, true, IMPLEMENTS_TOPYOBJECT>
 {
-    /// Py<T> fields have a potential optimization to use Python's "struct members" to read
+    /// `Py<T>` fields have a potential optimization to use Python's "struct members" to read
     /// the field directly from the struct, rather than using a getter function.
     ///
     /// This is the most efficient operation the Python interpreter could possibly do to
@@ -1261,7 +1261,7 @@ impl<
     }
 }
 
-/// Field is not Py<T>; try to use `ToPyObject` to avoid potentially expensive clones of containers like `Vec`
+/// Field is not `Py<T>`; try to use `ToPyObject` to avoid potentially expensive clones of containers like `Vec`
 impl<ClassT: PyClass, FieldT: ToPyObject, Offset: OffsetCalculator<ClassT, FieldT>>
     PyClassGetterGenerator<ClassT, FieldT, Offset, false, true>
 {

tests/test_getter_setter.rs

@@ -258,3 +258,25 @@ fn borrowed_value_with_lifetime_of_self() {
         py_run!(py, inst, "assert inst.value == 'value'");
     });
 }
+
+#[test]
+fn frozen_py_field_get() {
+    #[pyclass(frozen)]
+    struct FrozenPyField {
+        #[pyo3(get)]
+        value: Py<PyAny>,
+    }
+
+    Python::with_gil(|py| {
+        let inst = Py::new(
+            py,
+            FrozenPyField {
+                value: "value".into_py(py),
+            },
+        )
+        .unwrap()
+        .to_object(py);
+
+        py_run!(py, inst, "assert inst.value == 'value'");
+    });
+}