Merge pull request #136 from labuzm/fix_wrapper

fix freeing of python wrapper

src/kaa/nodes.pxi

@@ -67,17 +67,11 @@ cdef cppclass CPyNodeWrapper(CForeignNodeWrapper):
 
         py_wrapper._reset()
         result.unwrap_result()
-        this._release_wrapper()
 
     __dealloc__() nogil:
         if this.added_to_parent:
             with gil:
-                this._release_wrapper()
-
-    void _release_wrapper() with gil:
-        Py_XDECREF(this.py_wrapper)
-        this.py_wrapper = NULL
-        this.added_to_parent = False
+                Py_XDECREF(this.py_wrapper)
 
 
 @cython.freelist(NODE_FREELIST_SIZE)

tests/test_tree.py

@@ -0,0 +1,23 @@
+import pytest
+
+from kaa.nodes import Node
+
+from tests.utils import TestScene
+
+
+class ChildNode(Node):
+    def on_attach(self):
+        assert self.parent
+
+    def on_detach(self):
+        # parent is already deleted
+        assert not self.parent
+
+
+@pytest.mark.usefixtures('test_engine')
+def test_on_callbacks():
+    parent = Node()
+    parent.add_child(ChildNode())
+    scene = TestScene(lambda scene, dt: parent.delete())
+    scene.root.add_child(parent)
+    scene.run_on_engine(1)

tests/utils.py

@@ -0,0 +1,19 @@
+from kaa.engine import Scene
+
+
+class TestScene(Scene):
+    def __init__(self, update_function):
+        self._frames = None
+        self._test_update_function = update_function
+
+    def update(self, dt):
+        if self._frames == 0:
+            self.engine.quit()
+            return
+
+        self._test_update_function(self, dt)
+        self._frames -= 1
+
+    def run_on_engine(self, frames):
+        self._frames = frames
+        self.engine.run(self)