Skip to content

Commit 919f32d

Browse files
alexrudd2alexrudd2
committed
pre-commit run --all-files
1 parent f25e30d commit 919f32d

File tree

5 files changed

+37
-35
lines changed

5 files changed

+37
-35
lines changed

‎pymodbus/device.py‎

Lines changed: 22 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -221,21 +221,26 @@ def __str__(self):
221221
# Properties
222222
# -------------------------------------------------------------------------#
223223
VendorName = dict_property(
224-
lambda s: s.__data, 0 # pylint: disable=protected-access
224+
lambda s: s.__data,
225+
0, # pylint: disable=protected-access
225226
)
226227
ProductCode = dict_property(
227-
lambda s: s.__data, 1 # pylint: disable=protected-access
228+
lambda s: s.__data,
229+
1, # pylint: disable=protected-access
228230
)
229231
MajorMinorRevision = dict_property(
230-
lambda s: s.__data, 2 # pylint: disable=protected-access
232+
lambda s: s.__data,
233+
2, # pylint: disable=protected-access
231234
)
232235
VendorUrl = dict_property(lambda s: s.__data, 3) # pylint: disable=protected-access
233236
ProductName = dict_property(
234-
lambda s: s.__data, 4 # pylint: disable=protected-access
237+
lambda s: s.__data,
238+
4, # pylint: disable=protected-access
235239
)
236240
ModelName = dict_property(lambda s: s.__data, 5) # pylint: disable=protected-access
237241
UserApplicationName = dict_property(
238-
lambda s: s.__data, 6 # pylint: disable=protected-access
242+
lambda s: s.__data,
243+
6, # pylint: disable=protected-access
239244
)
240245

241246

@@ -429,24 +434,30 @@ def summary(self):
429434
# Properties
430435
# -------------------------------------------------------------------------#
431436
BusMessage = dict_property(
432-
lambda s: s.__data, 0 # pylint: disable=protected-access
437+
lambda s: s.__data,
438+
0, # pylint: disable=protected-access
433439
)
434440
BusCommunicationError = dict_property(
435-
lambda s: s.__data, 1 # pylint: disable=protected-access
441+
lambda s: s.__data,
442+
1, # pylint: disable=protected-access
436443
)
437444
BusExceptionError = dict_property(
438-
lambda s: s.__data, 2 # pylint: disable=protected-access
445+
lambda s: s.__data,
446+
2, # pylint: disable=protected-access
439447
)
440448
SlaveMessage = dict_property(
441-
lambda s: s.__data, 3 # pylint: disable=protected-access
449+
lambda s: s.__data,
450+
3, # pylint: disable=protected-access
442451
)
443452
SlaveNoResponse = dict_property(
444-
lambda s: s.__data, 4 # pylint: disable=protected-access
453+
lambda s: s.__data,
454+
4, # pylint: disable=protected-access
445455
)
446456
SlaveNAK = dict_property(lambda s: s.__data, 5) # pylint: disable=protected-access
447457
SlaveBusy = dict_property(lambda s: s.__data, 6) # pylint: disable=protected-access
448458
BusCharacterOverrun = dict_property(
449-
lambda s: s.__data, 7 # pylint: disable=protected-access
459+
lambda s: s.__data,
460+
7, # pylint: disable=protected-access
450461
)
451462
Event = dict_property(lambda s: s.__data, 8) # pylint: disable=protected-access
452463

‎pymodbus/utilities.py‎

Lines changed: 8 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -77,26 +77,20 @@ def dict_property(store, index):
7777
getter = lambda self: store( # pylint: disable=unnecessary-lambda-assignment
7878
self
7979
)[index]
80-
setter = (
81-
lambda self, value: store( # pylint: disable=unnecessary-lambda-assignment
82-
self
83-
).__setitem__(index, value)
84-
)
80+
setter = lambda self, value: store( # pylint: disable=unnecessary-lambda-assignment
81+
self
82+
).__setitem__(index, value)
8583
elif isinstance(store, str):
8684
getter = lambda self: self.__getattribute__( # pylint: disable=unnecessary-dunder-call,unnecessary-lambda-assignment
8785
store
88-
)[
89-
index
90-
]
86+
)[index]
9187
setter = lambda self, value: self.__getattribute__( # pylint: disable=unnecessary-dunder-call,unnecessary-lambda-assignment
9288
store
93-
).__setitem__(
94-
index, value
95-
)
89+
).__setitem__(index, value)
9690
else:
97-
getter = lambda self: store[ # pylint: disable=unnecessary-lambda-assignment
98-
index
99-
]
91+
getter = (
92+
lambda self: store[index] # pylint: disable=unnecessary-lambda-assignment
93+
)
10094
setter = lambda self, value: store.__setitem__( # pylint: disable=unnecessary-lambda-assignment
10195
index, value
10296
)

‎test/sub_messages/test_register_read_messages.py‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,7 @@ def setup_method(self):
5858
ReadWriteMultipleRegistersRequest(
5959
write_registers=0xAB,
6060
**arguments,
61-
): b"\x00\x01\x00\x05\x00\x01\x00"
62-
b"\x01\x02\x00\xAB",
61+
): b"\x00\x01\x00\x05\x00\x01\x00" b"\x01\x02\x00\xAB",
6362
}
6463
self.response_read = {
6564
ReadRegistersResponseBase(self.values): TEST_MESSAGE,

‎test/sub_server/test_server_asyncio.py‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -323,13 +323,13 @@ async def test_async_udp_server_serve_forever_twice(self):
323323

324324
async def test_async_udp_server_roundtrip(self):
325325
"""Test sending and receiving data on udp socket"""
326-
expected_response = b"\x01\x00\x00\x00\x00\x05\x01\x03\x02\x00\x11" # value of 17 as per context
326+
expected_response = (
327+
b"\x01\x00\x00\x00\x00\x05\x01\x03\x02\x00\x11"
328+
) # value of 17 as per context
327329
BasicClient.dataTo = TEST_DATA # slave 1, read register
328330
BasicClient.done = asyncio.Future()
329331
await self.start_server(do_udp=True)
330-
random_port = self.server.transport._sock.getsockname()[ # pylint: disable=protected-access
331-
1
332-
]
332+
random_port = self.server.transport._sock.getsockname()[1] # pylint: disable=protected-access
333333
transport, _ = await self.loop.create_datagram_endpoint(
334334
BasicClient, remote_addr=("127.0.0.1", random_port)
335335
)

‎test/test_transaction.py‎

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -65,10 +65,8 @@ def test_calculate_expected_response_length(self):
6565
self._tm.client = mock.MagicMock()
6666
self._tm.client.framer = mock.MagicMock()
6767
self._tm._set_adu_size() # pylint: disable=protected-access
68-
assert (
69-
not self._tm._calculate_response_length( # pylint: disable=protected-access
70-
0
71-
)
68+
assert not self._tm._calculate_response_length( # pylint: disable=protected-access
69+
0
7270
)
7371
self._tm.base_adu_size = 10
7472
assert (

0 commit comments

Comments
 (0)