Fix prospector warnings

tests/software_tests/database/test_abstract_data_record.py

@@ -1,5 +1,6 @@
 import pytest
 from mock import Mock, patch
+
 from uds.database.abstract_data_record import AbstractDataRecord
 
 SCRIPT_LOCATION = "uds.database.abstract_data_record"

uds/can/flow_control.py

@@ -57,9 +57,9 @@ class CanFlowStatus(ValidatedEnum, NibbleEnum):
 
     ContinueToSend: "CanFlowStatus" = 0x0  # type: ignore
     """Asks to resume Consecutive Frames transmission."""
-    Wait: "CanFlowStatus" = 0x1  # type: ignore  # noqa: F841
+    Wait: "CanFlowStatus" = 0x1  # type: ignore
     """Asks to pause Consecutive Frames transmission."""
-    Overflow: "CanFlowStatus" = 0x2  # type: ignore  # noqa: F841
+    Overflow: "CanFlowStatus" = 0x2  # type: ignore
     """Asks to abort transmission of a diagnostic message."""
 
 

uds/database/abstract_data_record.py

@@ -21,7 +21,7 @@ class DecodedDataRecord(TypedDict):
 
     name: str
     raw_value: int
-    physical_value: DataRecordPhysicalValueAlias
+    physical_value: DataRecordPhysicalValueAlias  # noqa: F841
 
 
 class DataRecordType(ValidatedEnum):
@@ -40,7 +40,7 @@ class AbstractDataRecord(ABC):
 
     def __init__(self, name: str) -> None:
         """
-        Common part of initialization for all Data Records.
+        Initialize common part for all Data Records.
 
         :param name: Name to assign to this Data Record.
 
@@ -55,17 +55,17 @@ def name(self) -> str:
         """Name of this Data Record."""
         return self.__name
 
-    @property
+    @property  # noqa: F841
     @abstractmethod
     def data_record_type(self) -> DataRecordType:
         """Type of this Data Record."""
 
-    @property
+    @property  # noqa: F841
     @abstractmethod
     def length(self) -> int:
-        """Number of bits that this Data Record is carried over."""
+        """Get number of bits that this Data Record is stored over."""
 
-    @property
+    @property  # noqa: F841
     @abstractmethod
     def is_reoccurring(self) -> bool:
         """
@@ -76,7 +76,7 @@ def is_reoccurring(self) -> bool:
         - True - number of occurrences might vary
         """
 
-    @property
+    @property  # noqa: F841
     @abstractmethod
     def min_occurrences(self) -> int:
         """
@@ -86,7 +86,7 @@ def min_occurrences(self) -> int:
             equals True.
         """
 
-    @property
+    @property  # noqa: F841
     @abstractmethod
     def max_occurrences(self) -> int:
         """
@@ -97,13 +97,13 @@ def max_occurrences(self) -> int:
         .. warning:: No maximal number (infinite number of occurrences) is represented by `0` value.
         """
 
-    @property
+    @property  # noqa: F841
     @abstractmethod
     def contains(self) -> Tuple["AbstractDataRecord", ...]:
-        """Data Records contained by this Data Record."""
+        """Get Data Records contained by this Data Record."""
 
     @abstractmethod
-    def decode(self, raw_value: int) -> DecodedDataRecord:
+    def decode(self, raw_value: int) -> DecodedDataRecord:  # noqa: F841
         """
         Decode physical value for provided raw value.
 
@@ -113,7 +113,7 @@ def decode(self, raw_value: int) -> DecodedDataRecord:
         """
 
     @abstractmethod
-    def encode(self, physical_value: DataRecordPhysicalValueAlias) -> int:
+    def encode(self, physical_value: DataRecordPhysicalValueAlias) -> int:  # noqa: F841
         """
         Encode raw value for provided physical value.
 

uds/transmission_attributes/transmission_direction.py

@@ -11,7 +11,7 @@
 class TransmissionDirection(ValidatedEnum, StrEnum):
     """Direction of a communication."""
 
-    RECEIVED: "TransmissionDirection" = "Rx"  # type: ignore  # noqa: F841
+    RECEIVED: "TransmissionDirection" = "Rx"  # type: ignore
     """Incoming transmission from the perspective of the code."""
-    TRANSMITTED: "TransmissionDirection" = "Tx"  # type: ignore  # noqa: F841
+    TRANSMITTED: "TransmissionDirection" = "Tx"  # type: ignore
     """Outgoing transmission from the perspective of the code."""