Avoid raising StopIteration for PEP-0479 compliance

Author: nocarryr

pydispatch/dispatch.py

@@ -84,12 +84,11 @@ class Foo(Dispatcher):
     __skip_initialized = True
     def __new__(cls, *args, **kwargs):
         def iter_bases(_cls):
-            if _cls is object:
-                raise StopIteration
-            yield _cls
-            for b in _cls.__bases__:
-                for _cls_ in iter_bases(b):
-                    yield _cls_
+            if _cls is not object:
+                yield _cls
+                for b in _cls.__bases__:
+                    for _cls_ in iter_bases(b):
+                        yield _cls_
         skip_initialized = Dispatcher._Dispatcher__skip_initialized
         if not skip_initialized or cls not in Dispatcher._Dispatcher__initialized_subclasses:
             props = {}

tests/test_subclass_init.py

@@ -1,5 +1,47 @@
 import math
 import time
+import warnings
+
+def test_dispatcher_construction():
+    from pydispatch import Dispatcher, Property
+
+    class A(Dispatcher):
+        foo = Property()
+        bar = Property()
+        baz = Property()
+        _events_ = [
+            'on_stuff', 'on_more_stuff',
+        ]
+    class B(A):
+        a = Property()
+        b = Property()
+        c = Property()
+        _events_ = [
+            'on_even_more_stuff', 'on_one_more_thing'
+        ]
+
+    with warnings.catch_warnings(record=True) as w_list:
+        warnings.simplefilter("always")
+        a = A()
+        b = B()
+        for w in w_list:
+            # Check for PEP-0479 (StopIteration) issues
+            assert not issubclass(w.category, DeprecationWarning)
+            assert not issubclass(w.category, PendingDeprecationWarning)
+        assert len(w_list) == 0
+
+    a_prop_names = {'foo', 'bar', 'baz'}
+    a_event_names = {'on_stuff', 'on_more_stuff'}
+    assert a_prop_names == set(a._Dispatcher__property_events.keys())
+    assert a_event_names == set(a._Dispatcher__events.keys())
+
+    b_prop_names = {'a', 'b', 'c'}
+    b_prop_names |= a_prop_names
+    b_event_names = {'on_even_more_stuff', 'on_one_more_thing'}
+    b_event_names |= a_event_names
+
+    assert b_prop_names == set(b._Dispatcher__property_events.keys())
+    assert b_event_names == set(b._Dispatcher__events.keys())
 
 def test_subclass_new_timing():
     from pydispatch import Dispatcher, Property