Better hashing

Author: justmobilize

adafruit_connection_manager.py

@@ -111,11 +111,10 @@ def create_fake_ssl_context(
 
 
 def _get_radio_hash_key(radio):
-    class_name = radio.__class__.__name__
-    # trying to use wifi.radio as a key results in:
-    #   TypeError: unsupported type for __hash__: 'Radio'
-    # So just use the class name in this case
-    return class_name if class_name == "Radio" else radio
+    try:
+        return hash(radio)
+    except TypeError:
+        return radio.__class__.__name__
 
 
 def get_radio_socketpool(radio):

tests/get_radio_test.py

@@ -13,6 +13,18 @@
 import adafruit_connection_manager
 
 
+def test__get_radio_hash_key():
+    radio = mocket.MockRadio.Radio()
+    assert adafruit_connection_manager._get_radio_hash_key(radio) == hash(radio)
+
+
+def test__get_radio_hash_key_not_hashable():
+    radio = mocket.MockRadio.Radio()
+
+    with mock.patch("builtins.hash", side_effect=TypeError()):
+        assert adafruit_connection_manager._get_radio_hash_key(radio) == "Radio"
+
+
 def test_get_radio_socketpool_wifi(  # pylint: disable=unused-argument
     circuitpython_socketpool_module,
 ):