use common hex_id() func

Author: rmorshea

src/idom/core/component.py

@@ -5,6 +5,7 @@
 from functools import wraps
 from typing import Any, Callable, Dict, Tuple, Union
 
+from .utils import hex_id
 from .vdom import VdomDict
 
 
@@ -74,6 +75,6 @@ def __repr__(self) -> str:
         else:
             items = ", ".join(f"{k}={v!r}" for k, v in args.items())
             if items:
-                return f"{self._function.__name__}({hex(id(self))}, {items})"
+                return f"{self._function.__name__}({hex_id(self)}, {items})"
             else:
-                return f"{self._function.__name__}({hex(id(self))})"
+                return f"{self._function.__name__}({hex_id(self)})"

src/idom/core/layout.py

@@ -25,7 +25,7 @@
 from .component import AbstractComponent
 from .events import EventHandler
 from .hooks import LifeCycleHook
-from .utils import CannotAccessResource, HasAsyncResources, async_resource
+from .utils import CannotAccessResource, HasAsyncResources, async_resource, hex_id
 from .vdom import validate_vdom
 
 
@@ -251,7 +251,7 @@ def _render_model_event_handlers_without_old_state(
 
         model_event_handlers = new_state.model["eventHandlers"] = {}
         for event, handler in handlers_by_event.items():
-            target = hex(id(handler))[2:]
+            target = hex_id(handler)
             new_state.targets_by_event[event] = target
             self._event_handlers[target] = handler
             model_event_handlers[event] = {
@@ -285,10 +285,10 @@ def _render_model_children(
         for index, child in enumerate(raw_children):
             if isinstance(child, dict):
                 child_type = DICT_TYPE
-                key = child.get("key") or hex(id(child))[2:]
+                key = child.get("key") or hex_id(child)
             elif isinstance(child, AbstractComponent):
                 child_type = COMPONENT_TYPE
-                key = getattr(child, "key", None) or hex(id(child))[2:]
+                key = getattr(child, "key", None) or hex_id(child)
             else:
                 child = str(child)
                 child_type = STRING_TYPE
@@ -336,13 +336,13 @@ def _render_model_children_without_old_state(
         new_children = new_state.model["children"] = []
         for index, child in enumerate(raw_children):
             if isinstance(child, dict):
-                key = child.get("key") or hex(id(child))
+                key = child.get("key") or hex_id(child)
                 child_state = _ModelState(new_state, index, key, None)
                 self._render_model(None, child_state, child)
                 new_children.append(child_state.model)
                 new_state.children_by_key[key] = child_state
             elif isinstance(child, AbstractComponent):
-                key = getattr(child, "key", "") or hex(id(child))
+                key = getattr(child, "key", "") or hex_id(child)
                 life_cycle_hook = LifeCycleHook(child, self)
                 child_state = _ModelState(new_state, index, key, life_cycle_hook)
                 self._render_component(None, child_state, child)

src/idom/core/utils.py

@@ -22,6 +22,10 @@
     from async_generator import asynccontextmanager
 
 
+def hex_id(obj: Any) -> str:
+    return format(id(obj), "x")
+
+
 _Rsrc = TypeVar("_Rsrc")
 _Self = TypeVar("_Self", bound="HasAsyncResources")
 

tests/test_core/test_component.py

@@ -1,4 +1,5 @@
 import idom
+from idom.core.utils import hex_id
 
 
 def test_component_repr():
@@ -8,7 +9,7 @@ def MyComponent(a, *b, **c):
 
     mc1 = MyComponent(1, 2, 3, x=4, y=5)
 
-    expected = f"MyComponent({hex(id(mc1))}, a=1, b=(2, 3), c={{'x': 4, 'y': 5}})"
+    expected = f"MyComponent({hex_id(mc1)}, a=1, b=(2, 3), c={{'x': 4, 'y': 5}})"
     assert repr(mc1) == expected
 
     # not enough args supplied to function

tests/test_core/test_layout.py

@@ -6,6 +6,7 @@
 
 import idom
 from idom.core.layout import LayoutEvent, LayoutUpdate
+from idom.core.utils import hex_id
 from tests.general_utils import HookCatcher, assert_same_items
 
 
@@ -21,7 +22,7 @@ def MyComponent():
 
     my_component = MyComponent()
     layout = idom.Layout(my_component)
-    assert str(layout) == f"Layout(MyComponent({hex(id(my_component))}))"
+    assert str(layout) == f"Layout(MyComponent({hex_id(my_component)}))"
 
 
 def test_layout_expects_abstract_component():