Implement ToPyObject for Box<T>

newsfragments/3729.added.md

@@ -0,0 +1 @@
+Implement `IntoPy<PyObject>` and `ToPyObject` for `Box<T>`.

src/conversion.rs

@@ -243,6 +243,21 @@ impl<T: ?Sized + ToPyObject> ToPyObject for &'_ T {
     }
 }
 
+impl<T: ?Sized + ToPyObject> ToPyObject for Box<T> {
+    fn to_object(&self, py: Python<'_>) -> PyObject {
+        self.as_ref().to_object(py)
+    }
+}
+
+impl<T: ?Sized> IntoPy<PyObject> for Box<T>
+where
+    T: IntoPy<PyObject>,
+{
+    fn into_py(self, py: Python<'_>) -> PyObject {
+        (*self).into_py(py)
+    }
+}
+
 /// `Option::Some<T>` is converted like `T`.
 /// `Option::None` is converted to Python `None`.
 impl<T> ToPyObject for Option<T>
@@ -578,7 +593,7 @@ mod test_no_clone {}
 
 #[cfg(test)]
 mod tests {
-    use crate::{PyObject, Python};
+    use crate::{IntoPy, PyObject, Python, ToPyObject};
 
     #[allow(deprecated)]
     mod deprecated {
@@ -641,4 +656,22 @@ mod tests {
             assert_eq!(none.get_refcnt(), ref_cnt);
         });
     }
+
+    #[test]
+    fn test_box_topyobject() {
+        let s: Box<str> = "test".into();
+
+        let obj = Python::with_gil(|py| s.to_object(py));
+
+        assert_eq!(&obj.to_string(), "test")
+    }
+
+    #[test]
+    fn test_box_intopy() {
+        let s: Box<str> = "test".into();
+
+        let obj: PyObject = Python::with_gil(|py| s.into_py(py));
+
+        assert_eq!(&obj.to_string(), "test")
+    }
 }