Improve naming of method returning all dependencies of type

Ref. #672

rflx/model/message.py

@@ -121,8 +121,8 @@ def __init__(
 
         assert len(self.identifier.parts) > 1, "type identifier must contain package"
 
-        self._enum_literals = mty.qualified_enum_literals(self.all_types, self.package)
-        self._type_literals = mty.qualified_type_literals(self.all_types)
+        self._enum_literals = mty.qualified_enum_literals(self.dependencies, self.package)
+        self._type_literals = mty.qualified_type_literals(self.dependencies)
 
         if not state and (structure or types):
             try:
@@ -197,8 +197,8 @@ def refined_type(self, refinements: Sequence["Refinement"]) -> rty.Message:
         )
 
     @property
-    def all_types(self) -> List[mty.Type]:
-        return [self, *unique(a for t in self.types.values() for a in t.all_types)]
+    def dependencies(self) -> List[mty.Type]:
+        return [self, *unique(a for t in self.types.values() for a in t.dependencies)]
 
     @abstractmethod
     def copy(

rflx/model/type_.py

@@ -49,7 +49,7 @@ def type_(self) -> rty.Type:
         return rty.Undefined()
 
     @property
-    def all_types(self) -> ty.List["Type"]:
+    def dependencies(self) -> ty.List["Type"]:
         """
         Return a list consisting of the type and all types on which the type depends. The
         dependencies are determined recursively.
@@ -521,8 +521,8 @@ def element_size(self) -> expr.Expr:
         return expr.Size(self.element_type.name)
 
     @property
-    def all_types(self) -> ty.List["Type"]:
-        return [self, *self.element_type.all_types]
+    def dependencies(self) -> ty.List["Type"]:
+        return [self, *self.element_type.dependencies]
 
 
 class Opaque(Composite):

tests/unit/model/message_test.py

@@ -3090,14 +3090,14 @@ def test_paths() -> None:
     }
 
 
-def test_message_all_types() -> None:
-    assert TLV_MESSAGE.all_types == [
+def test_message_dependencies() -> None:
+    assert TLV_MESSAGE.dependencies == [
         TLV_MESSAGE,
         TLV_TAG,
         TLV_LENGTH,
         OPAQUE,
     ]
-    assert SEQUENCE_MESSAGES_MESSAGE.all_types == [
+    assert SEQUENCE_MESSAGES_MESSAGE.dependencies == [
         SEQUENCE_MESSAGES_MESSAGE,
         SEQUENCE_LENGTH,
         SEQUENCE_INNER_MESSAGES,

tests/unit/model/type_test.py

@@ -39,11 +39,11 @@ class NewType(Type):
     assert NewType("P::T").type_ == rty.Undefined()
 
 
-def test_type_all_types() -> None:
+def test_type_dependencies() -> None:
     class NewType(Type):
         pass
 
-    assert NewType("P::T").all_types == [NewType("P::T")]
+    assert NewType("P::T").dependencies == [NewType("P::T")]
 
 
 def test_modular_size() -> None:
@@ -295,12 +295,12 @@ def test_enumeration_invalid_multiple_duplicate_elements() -> None:
     )
 
 
-def test_sequence_all_types() -> None:
-    assert models.SEQUENCE_MODULAR_VECTOR.all_types == [
+def test_sequence_dependencies() -> None:
+    assert models.SEQUENCE_MODULAR_VECTOR.dependencies == [
         models.SEQUENCE_MODULAR_VECTOR,
         models.SEQUENCE_MODULAR_INTEGER,
     ]
-    assert models.SEQUENCE_INNER_MESSAGES.all_types == [
+    assert models.SEQUENCE_INNER_MESSAGES.dependencies == [
         models.SEQUENCE_INNER_MESSAGES,
         models.SEQUENCE_INNER_MESSAGE,
         models.SEQUENCE_LENGTH,

tools/generate_spark_test_code.py

@@ -24,7 +24,7 @@
     tests.data.models.NULL_MODEL,
     tests.data.models.TLV_MODEL,
     tests.data.models.NULL_MESSAGE_IN_TLV_MESSAGE_MODEL,
-    Model(tests.data.models.FIXED_SIZE_SIMPLE_MESSAGE.all_types),
+    Model(tests.data.models.FIXED_SIZE_SIMPLE_MESSAGE.dependencies),
 ]
 
 SPECIFICATION_FILES = [