Skip to content

Commit 931f825

Browse files
committed
style: apply ruff formatting
1 parent 610d613 commit 931f825

3 files changed

Lines changed: 17 additions & 16 deletions

File tree

src/publisher/mqtt_publisher.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -261,7 +261,12 @@ async def __handle_imported_energy(self, topic: str, payload: str) -> None:
261261
)
262262

263263
def __publish(
264-
self, topic: str, payload: WirePayload | None, *, retain: bool = True, qos: int = 0
264+
self,
265+
topic: str,
266+
payload: WirePayload | None,
267+
*,
268+
retain: bool = True,
269+
qos: int = 0,
265270
) -> None:
266271
LOG.debug("Publishing to MQTT topic %s with payload %s", topic, payload)
267272
loop = asyncio.get_running_loop()

src/status_publisher/charge/chrg_mgmt_data.py

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -104,18 +104,18 @@ def publish(self, charge_mgmt_data: ChrgMgmtData) -> ChrgMgmtDataProcessingResul
104104
self._transform_and_publish(
105105
topic=mqtt_topics.BMS_CHARGE_STATUS,
106106
value=charge_mgmt_data.bms_charging_status,
107-
transform=lambda x: f"UNKNOWN {charge_mgmt_data.bmsChrgSts}"
108-
if x is None
109-
else x.name,
107+
transform=lambda x: (
108+
f"UNKNOWN {charge_mgmt_data.bmsChrgSts}" if x is None else x.name
109+
),
110110
)
111111

112112
self._transform_and_publish(
113113
topic=mqtt_topics.DRIVETRAIN_CHARGING_STOP_REASON,
114114
value=charge_mgmt_data.charging_stop_reason,
115115
validator=lambda x: x != ChargingStopReason.NO_REASON,
116-
transform=lambda x: f"UNKNOWN {charge_mgmt_data.bmsChrgSpRsn}"
117-
if x is None
118-
else x.name,
116+
transform=lambda x: (
117+
f"UNKNOWN {charge_mgmt_data.bmsChrgSpRsn}" if x is None else x.name
118+
),
119119
)
120120

121121
self._publish(
@@ -153,9 +153,9 @@ def publish(self, charge_mgmt_data: ChrgMgmtData) -> ChrgMgmtDataProcessingResul
153153
topic=mqtt_topics.DRIVETRAIN_BATTERY_HEATING_STOP_REASON,
154154
value=charge_mgmt_data.heating_stop_reason,
155155
validator=lambda x: x != HeatingStopReason.NO_REASON,
156-
transform=lambda x: f"UNKNOWN ({charge_mgmt_data.bmsPTCHeatResp})"
157-
if x is None
158-
else x.name,
156+
transform=lambda x: (
157+
f"UNKNOWN ({charge_mgmt_data.bmsPTCHeatResp})" if x is None else x.name
158+
),
159159
)
160160

161161
self._transform_and_publish(

tests/test_mqtt_publisher.py

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -104,9 +104,7 @@ async def on_charger_connection_state_changed(
104104

105105
async def _call_and_flush(self, fn: Any, *args: Any, **kwargs: Any) -> AsyncMock:
106106
mock = AsyncMock()
107-
with patch.object(
108-
self.mqtt_client, "_MqttPublisher__async_publish", mock
109-
):
107+
with patch.object(self.mqtt_client, "_MqttPublisher__async_publish", mock):
110108
self.mqtt_client._MqttPublisher__connected.set() # type: ignore[attr-defined]
111109
fn(*args, **kwargs)
112110
await asyncio.sleep(0)
@@ -153,9 +151,7 @@ async def test_publish_float_forwards_retain_false(self) -> None:
153151
m.assert_called_once_with("saic/foo", 1.5, retain=False, qos=0)
154152

155153
async def test_publish_json_default_is_retained(self) -> None:
156-
m = await self._call_and_flush(
157-
self.mqtt_client.publish_json, "foo", {"a": 1}
158-
)
154+
m = await self._call_and_flush(self.mqtt_client.publish_json, "foo", {"a": 1})
159155
m.assert_called_once()
160156
args, kwargs = m.call_args
161157
assert args[0] == "saic/foo"

0 commit comments

Comments
 (0)