Fix test case with __new__ (#3285)

Fix test case with new (#3285)
6e6975e2 · Aaron Gokaslan · GitHub · d0f3c51f · 6e6975e2
Unverified Commit 6e6975e2 authored Sep 20, 2021 by Aaron Gokaslan Committed by GitHub Sep 20, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

tests/test_class.cpp tests/test_class.cpp +3 -2

No files found.
--- a/tests/test_class.cpp
+++ b/tests/test_class.cpp
@@ -63,8 +63,9 @@ TEST_SUBMODULE(class_, m) {
        .def_static("new_instance", &NoConstructor::new_instance, "Return an instance");

    py::class_<NoConstructorNew>(m, "NoConstructorNew")
-        .def(py::init([](NoConstructorNew *self) { return self; })) // Need a NOOP __init__
-        .def_static("__new__", [](const py::object *) { return NoConstructorNew::new_instance(); } );
+        .def(py::init([](const NoConstructorNew &self) { return self; })) // Need a NOOP __init__
+        .def_static("__new__",
+                    [](const py::object &) { return NoConstructorNew::new_instance(); });

    // test_inheritance
    class Pet {