add a safety check to __getattr__ to avoid unwanted recursion

Author: joerendleman

src/robotlibcore/core/hybrid.py

@@ -106,6 +106,8 @@ def __get_members_from_instance(self, instance):
             yield name, getattr(owner, name)
 
     def __getattr__(self, name):
+        if name == "attributes":
+            return super().__getattribute__(name)
         if name in self.attributes:
             return self.attributes[name]
         msg = "{!r} object has no attribute {!r}".format(type(self).__name__, name)

utest/test_robotlibcore.py

@@ -104,3 +104,13 @@ def test_library_cannot_be_class():
     with pytest.raises(TypeError) as exc_info:
         HybridCore([HybridLibrary])
     assert str(exc_info.value) == "Libraries must be modules or instances, got class 'HybridLibrary' instead."
+
+def test_get_library_attr():
+    class TestClass(HybridCore):
+        def __init__(self):
+            self.a  = self.b *2
+            super().__init__()
+
+    with pytest.raises(AttributeError):
+        TestClass()
+