Test PR.

Author: ManuelSchneid3r

tests/test_class_sh_virtual_py_cpp_mix.cpp

@@ -29,6 +29,34 @@ int get_from_cpp_plainc_ptr(const Base *b) { return b->get() + 4000; }
 
 int get_from_cpp_unique_ptr(std::unique_ptr<Base> b) { return b->get() + 5000; }
 
+
+// >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
+
+struct VirtualItem
+{
+    VirtualItem() = default;
+    virtual ~VirtualItem() = default;
+    virtual int get() const = 0;
+};
+
+struct VirtualItemTrampoline : public VirtualItem, py::trampoline_self_life_support
+{
+    int get() const override { PYBIND11_OVERRIDE_PURE(int, VirtualItem, get, ); }
+};
+
+struct VirtualItemContainer
+{
+    std::shared_ptr<VirtualItem> held;
+    // ...
+};
+
+void cpp_space_test(std::shared_ptr<VirtualItemContainer> container_vec)
+{
+    auto i = container_vec->held->get();  // segfaults
+}
+
+// <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<
+
 struct BaseVirtualOverrider : Base, py::trampoline_self_life_support {
     using Base::Base;
 
@@ -55,4 +83,19 @@ TEST_SUBMODULE(class_sh_virtual_py_cpp_mix, m) {
 
     m.def("get_from_cpp_plainc_ptr", get_from_cpp_plainc_ptr, py::arg("b"));
     m.def("get_from_cpp_unique_ptr", get_from_cpp_unique_ptr, py::arg("b"));
+
+    // >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
+
+    py::classh<VirtualItem, VirtualItemTrampoline>(m, "VirtualItem")
+        .def(py::init<>())
+        .def("get", &VirtualItem::get)
+        ;
+
+    py::classh<VirtualItemContainer>(m, "VirtualItemContainer")
+        .def(py::init<std::shared_ptr<VirtualItem>>())
+        ;
+
+    m.def("cpp_space_test", cpp_space_test);
+
+    // <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<
 }

tests/test_class_sh_virtual_py_cpp_mix.py

@@ -64,3 +64,14 @@ def test_get_from_cpp_plainc_ptr(ctor, expected):
 def test_get_from_cpp_unique_ptr(ctor, expected):
     obj = ctor()
     assert m.get_from_cpp_unique_ptr(obj) == expected
+
+#  >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
+
+class VirtualItemDerived(m.VirtualItem):
+    def getInt(self):
+        return 42
+
+def test_cpp_space_test():
+    m.cpp_space_test(m.VirtualItemContainer(VirtualItemDerived()))
+
+#  <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<

tests/test_with_catch/test_interpreter.cpp

@@ -91,6 +91,48 @@ PYBIND11_EMBEDDED_MODULE(throw_error_already_set, ) {
     d["missing"].cast<py::object>();
 }
 
+// >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
+
+
+struct Item
+{
+    Item() = default;
+    virtual ~Item() = default;
+    virtual int getInt() const = 0;
+};
+
+struct ItemTrampoline : public Item, py::trampoline_self_life_support
+{
+    int getInt() const override { PYBIND11_OVERRIDE_PURE(int, Item, getInt, ); }
+};
+
+PYBIND11_EMBEDDED_MODULE(embedded_smart_holder, m) {
+
+    py::classh<Item, ItemTrampoline>(m, "Item")
+        .def(py::init<>())
+        .def("getInt", &Item::getInt)
+        ;
+}
+
+TEST_CASE("Embedded smart holder test") {
+
+    auto module = py::module_::import("embedded_smart_holder");
+
+    auto o = module.attr("Item")();  // create py
+
+    auto i = o.cast<std::shared_ptr<Item>>();  // cast cpp
+
+    if (i->getInt() != 42)  // test cpp
+        throw std::runtime_error("Not 42");
+
+    o = py::object(); // release py
+
+    if (i->getInt() != 42)  // test cpp
+        throw std::runtime_error("Not 42 after release");
+}
+
+// <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<
+
 TEST_CASE("PYTHONPATH is used to update sys.path") {
     // The setup for this TEST_CASE is in catch.cpp!
     auto sys_path = py::str(py::module_::import("sys").attr("path")).cast<std::string>();