Rename Serializable.register to register_subclass

malcolm/core/__init__.py

@@ -1,4 +1,4 @@
-# Need to import these so that the register hooks are called
+# Need to import these so that the register_subclass hooks are called
 from malcolm.core.stringmeta import StringMeta  # noqa
 from malcolm.core.stringarraymeta import StringArrayMeta # noqa
 from malcolm.core.numbermeta import NumberMeta  # noqa

malcolm/core/attribute.py

@@ -4,7 +4,7 @@
 from malcolm.core.serializable import Serializable
 
 
-@Serializable.register("epics:nt/NTAttribute:1.0")
+@Serializable.register_subclass("epics:nt/NTAttribute:1.0")
 class Attribute(Notifier):
     """Represents a value with type information that may be backed elsewhere"""
 

malcolm/core/block.py

@@ -33,7 +33,7 @@ def __exit__(self, type, value, traceback):
         self.lock.acquire()
         return False
 
-@Serializable.register("malcolm:core/Block:1.0")
+@Serializable.register_subclass("malcolm:core/Block:1.0")
 class Block(Notifier):
     """Object consisting of a number of Attributes and Methods"""
 

malcolm/core/blockmeta.py

@@ -1,6 +1,6 @@
 from malcolm.core.meta import Meta
 from malcolm.core.serializable import Serializable
 
-@Serializable.register("malcolm:core/BlockMeta:1.0")
+@Serializable.register_subclass("malcolm:core/BlockMeta:1.0")
 class BlockMeta(Meta):
     pass

malcolm/core/booleanarraymeta.py

@@ -2,7 +2,7 @@
 from malcolm.core.serializable import Serializable
 
 
-@Serializable.register("malcolm:core/BooleanArrayMeta:1.0")
+@Serializable.register_subclass("malcolm:core/BooleanArrayMeta:1.0")
 class BooleanArrayMeta(ScalarMeta):
     """Meta object containing information for a boolean array"""
 

malcolm/core/booleanmeta.py

@@ -2,7 +2,7 @@
 from malcolm.core.serializable import Serializable
 
 
-@Serializable.register("malcolm:core/BooleanMeta:1.0")
+@Serializable.register_subclass("malcolm:core/BooleanMeta:1.0")
 class BooleanMeta(ScalarMeta):
     """Meta object containing information for a boolean"""
 

malcolm/core/choicearraymeta.py

@@ -4,7 +4,7 @@
 from malcolm.core.serializable import Serializable
 
 
-@Serializable.register("malcolm:core/ChoiceArrayMeta:1.0")
+@Serializable.register_subclass("malcolm:core/ChoiceArrayMeta:1.0")
 class ChoiceArrayMeta(ScalarMeta):
     """Meta object containing information for a choice array"""
 

malcolm/core/choicemeta.py

@@ -4,7 +4,7 @@
 from malcolm.core.serializable import Serializable
 
 
-@Serializable.register("malcolm:core/ChoiceMeta:1.0")
+@Serializable.register_subclass("malcolm:core/ChoiceMeta:1.0")
 class ChoiceMeta(ScalarMeta):
     """Meta object containing information for a enum"""
 

malcolm/core/mapmeta.py

@@ -7,7 +7,7 @@
 REQUIRED = object()
 
 
-@Serializable.register("malcolm:core/MapMeta:1.0")
+@Serializable.register_subclass("malcolm:core/MapMeta:1.0")
 class MapMeta(Notifier):
     """An object containing a set of ScalarMeta objects"""
 

malcolm/core/method.py

@@ -8,7 +8,7 @@
 from malcolm.core.map import Map
 
 
-@Serializable.register("malcolm:core/Method:1.0")
+@Serializable.register_subclass("malcolm:core/Method:1.0")
 class Method(Notifier):
     """Exposes a function with metadata for arguments and return values"""
 

malcolm/core/numberarraymeta.py

@@ -7,7 +7,7 @@
 from malcolm.compat import base_string
 
 
-@Serializable.register("malcolm:core/NumberArrayMeta:1.0")
+@Serializable.register_subclass("malcolm:core/NumberArrayMeta:1.0")
 class NumberArrayMeta(ScalarMeta):
     """Meta object containing information for an array of numerical values"""
 

malcolm/core/numbermeta.py

@@ -7,7 +7,7 @@
 from malcolm.compat import base_string
 
 
-@Serializable.register("malcolm:core/NumberMeta:1.0")
+@Serializable.register_subclass("malcolm:core/NumberMeta:1.0")
 class NumberMeta(ScalarMeta):
     """Meta object containing information for a numerical value"""
 

malcolm/core/pointgeneratormeta.py

@@ -3,7 +3,7 @@
 from scanpointgenerator import CompoundGenerator
 
 
-@Serializable.register("malcolm:core/PointGeneratorMeta:1.0")
+@Serializable.register_subclass("malcolm:core/PointGeneratorMeta:1.0")
 class PointGeneratorMeta(ScalarMeta):
 
     def __init__(self, name, description):

malcolm/core/serializable.py

@@ -4,7 +4,7 @@
 class Serializable(object):
     """Mixin class for serializable objects"""
 
-    # This will be set by subclasses calling cls.register()
+    # This will be set by subclasses calling cls.register_subclass()
     typeid = None
     endpoints = None
     # dict mapping typeid name -> cls
@@ -63,7 +63,7 @@ def deserialize(cls, name, d):
         return subcls.from_dict(name, d)
 
     @classmethod
-    def register(cls, typeid):
+    def register_subclass(cls, typeid):
         """Register a subclass so from_dict() works
 
         Args:

malcolm/core/stringarraymeta.py

@@ -2,7 +2,7 @@
 from malcolm.core.serializable import Serializable
 
 
-@Serializable.register("malcolm:core/StringArrayMeta:1.0")
+@Serializable.register_subclass("malcolm:core/StringArrayMeta:1.0")
 class StringArrayMeta(ScalarMeta):
     """Meta object containing information for a string array"""
 

malcolm/core/stringmeta.py

@@ -2,7 +2,7 @@
 from malcolm.core.serializable import Serializable
 
 
-@Serializable.register("malcolm:core/StringMeta:1.0")
+@Serializable.register_subclass("malcolm:core/StringMeta:1.0")
 class StringMeta(ScalarMeta):
     """Meta object containing information for a string"""
 

tests/test_core/test_meta.py

@@ -10,7 +10,7 @@
 from malcolm.core.meta import Meta
 from malcolm.core.serializable import Serializable
 
-Serializable.register("meta:test")(Meta)
+Serializable.register_subclass("meta:test")(Meta)
 
 class TestMeta(unittest.TestCase):
     def test_init(self):

tests/test_core/test_notifier.py

@@ -10,7 +10,7 @@
 from malcolm.core.serializable import Serializable
 
 
-Serializable.register("notifier:test")(Notifier)
+Serializable.register_subclass("notifier:test")(Notifier)
 
 class TestInit(unittest.TestCase):
 

tests/test_core/test_scalarmeta.py

@@ -12,7 +12,7 @@
 
 # Register ScalarMeta as a sublcass of itself so we
 # can instantiate it for testing purposes.
-ScalarMeta.register("scalarmeta:test")(ScalarMeta)
+ScalarMeta.register_subclass("scalarmeta:test")(ScalarMeta)
 
 class TestInit(unittest.TestCase):
 

tests/test_core/test_serializable.py

@@ -10,14 +10,14 @@
 
 # Register Serializable as a sublcass of itself so we
 # can instantiate it for testing purposes.
-Serializable.register("serializable:test")(Serializable)
+Serializable.register_subclass("serializable:test")(Serializable)
 
 
 class TestSerialization(unittest.TestCase):
 
     def test_to_dict(self):
 
-        @Serializable.register("foo:1.0")
+        @Serializable.register_subclass("foo:1.0")
         class DummySerializable(Serializable):
             from_dict = Mock()
 
@@ -26,7 +26,7 @@ class DummySerializable(Serializable):
 
     def test_base_deserialize_calls_from_dict(self):
 
-        @Serializable.register("foo:1.0")
+        @Serializable.register_subclass("foo:1.0")
         class DummySerializable(Serializable):
             from_dict = Mock()
 
@@ -39,7 +39,7 @@ class DummySerializable(Serializable):
 
     def test_from_dict_calls_update(self):
 
-        @Serializable.register("foo:1.0")
+        @Serializable.register_subclass("foo:1.0")
         class DummySerializable(Serializable):
             update = Mock()
 
@@ -52,7 +52,7 @@ class DummySerializable(Serializable):
 
     def test_update_calls_set(self):
 
-        @Serializable.register("foo:1.0")
+        @Serializable.register_subclass("foo:1.0")
         class DummySerializable(Serializable):
             set_value = Mock()