Skip to content

Commit 5302c81

Browse files
committed
reduced cyclomatic complexity
1 parent 9d87788 commit 5302c81

File tree

2 files changed

+5
-8
lines changed

2 files changed

+5
-8
lines changed

02_mqtt-cluster/test/test_uns_spb_enums.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -319,9 +319,9 @@ def test_dataset_via_enum(value: sparkplug_b_pb2.Payload.DataSet, spb_obj):
319319
("String", sparkplug_b_pb2.Payload.Metric()), # negative test case
320320
],
321321
)
322-
def test_template_via_enum(value: sparkplug_b_pb2.Template, spb_obj):
322+
def test_template_via_enum(value, spb_obj):
323323
"""
324-
Test case for value setting Template via the ENUMs
324+
Test case for value setting Template via the ENUMs sparkplug_b_pb2.Template
325325
"""
326326
if isinstance(value, sparkplug_b_pb2.Payload.Template):
327327
SPBMetricDataTypes(sparkplug_b_pb2.Template).set_value_in_sparkplug(value=value, spb_object=spb_obj)

02_mqtt-cluster/test/test_uns_spb_helper_metrics.py

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -318,13 +318,12 @@ def test_add_metric_and_ddata_msg(timestamp: float, metrics: list[dict]):
318318
sparkplug_message = SpBMessageGenerator()
319319
payload = sparkplug_message.get_device_data_payload(timestamp=timestamp)
320320
alias = 0
321+
# creating the payload from the dict object provided as parameter
321322
for metric in metrics:
322323
name: str = metric["name"]
323324
datatype: int = metric["datatype"]
324325
value = metric.get("value", None)
325-
metric_timestamp = metric.get("timestamp", None)
326-
if metric_timestamp is None:
327-
metric_timestamp = timestamp
326+
metric_timestamp = metric.get("timestamp", timestamp)
328327
sparkplug_message.add_metric(
329328
payload_or_template=payload, name=name, alias=alias, datatype=datatype, value=value, timestamp=metric_timestamp
330329
)
@@ -594,9 +593,7 @@ def test_add_historical_metric_and_ddata_msg(metrics: list[dict]):
594593
name: str = metric["name"]
595594
datatype: int = metric["datatype"]
596595
value = metric.get("value", None)
597-
metric_timestamp = metric.get("timestamp", None)
598-
if metric_timestamp is None:
599-
metric_timestamp = timestamp
596+
metric_timestamp = metric.get("timestamp", timestamp)
600597
sparkplug_message.add_historical_metric(
601598
payload=payload, name=name, datatype=datatype, value=value, timestamp=metric_timestamp
602599
)

0 commit comments

Comments
 (0)