Skip to content

Commit e1c2de7

Browse files
committed
Fix some flake8 issues
1 parent b48ec14 commit e1c2de7

2 files changed

Lines changed: 25 additions & 13 deletions

File tree

tools/rdm/TestDefinitions.py

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -8654,22 +8654,26 @@ class SetBackgroundQueuedStatusPolicyWithNoData(TestMixins.SetWithNoDataMixin,
86548654
PID = 'BACKGROUND_QUEUED_STATUS_POLICY'
86558655

86568656

8657-
class SetBackgroundQueuedStatusPolicyWithExtraData(TestMixins.SetWithDataMixin,
8658-
OptionalParameterTestFixture):
8657+
class SetBackgroundQueuedStatusPolicyWithExtraData(
8658+
TestMixins.SetWithDataMixin,
8659+
OptionalParameterTestFixture):
86598660
"""Send a SET BACKGROUND_QUEUED_STATUS_POLICY command with extra data."""
86608661
PID = 'BACKGROUND_QUEUED_STATUS_POLICY'
86618662

86628663

86638664
class AllSubDevicesGetBackgroundQueuedStatusPolicyDescription(
86648665
TestMixins.AllSubDevicesGetMixin,
86658666
OptionalParameterTestFixture):
8666-
"""Send a get BACKGROUND_QUEUED_STATUS_POLICY_DESCRIPTION to ALL_SUB_DEVICES."""
8667+
"""Send a get BACKGROUND_QUEUED_STATUS_POLICY_DESCRIPTION to
8668+
ALL_SUB_DEVICES.
8669+
"""
86678670
PID = 'BACKGROUND_QUEUED_STATUS_POLICY_DESCRIPTION'
86688671
DATA = [0x00]
86698672

86708673

8671-
# class GetBackgroundQueuedStatusPolicyDescription(TestMixins.,
8672-
# OptionalParameterTestFixture):
8674+
# class GetBackgroundQueuedStatusPolicyDescription(
8675+
# TestMixins.,
8676+
# OptionalParameterTestFixture):
86738677
# CATEGORY = TestCategory.
86748678
# PID = 'BACKGROUND_QUEUED_STATUS_POLICY_DESCRIPTION'
86758679
# TODO(peter): Test get
@@ -8685,7 +8689,9 @@ class GetBackgroundQueuedStatusPolicyDescriptionWithNoData(
86858689
class GetBackgroundQueuedStatusPolicyDescriptionWithExtraData(
86868690
TestMixins.GetWithDataMixin,
86878691
OptionalParameterTestFixture):
8688-
"""GET BACKGROUND_QUEUED_STATUS_POLICY_DESCRIPTION with more than 1 byte of data."""
8692+
"""GET BACKGROUND_QUEUED_STATUS_POLICY_DESCRIPTION with more than 1 byte of
8693+
data.
8694+
"""
86898695
PID = 'BACKGROUND_QUEUED_STATUS_POLICY_DESCRIPTION'
86908696

86918697

tools/rdm/list_rdm_tests.py

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -154,14 +154,17 @@ def GetWithExtraData(names, pid, pid_test_base_name, get_size):
154154
dummy_data = GenerateDummyData(get_size)
155155
if dummy_data is None:
156156
print((" #DATA = b'foo' # TODO(%s): Specify extra data if this isn't "
157-
"enough. Ensure the first %d bytes are sane/valid.") % (getpass.getuser(), get_size))
157+
"enough. Ensure the first %d bytes are sane/valid.") %
158+
(getpass.getuser(), get_size))
158159
elif dummy_data != 'foo':
159160
# Doesn't match default, explicitly set value
160161
print((" DATA = b'%s' # TODO(%s): Specify extra data if this isn't "
161-
"enough. Ensure the first %d bytes are sane/valid.") % (dummy_data, getpass.getuser(), get_size))
162+
"enough. Ensure the first %d bytes are sane/valid.") %
163+
(dummy_data, getpass.getuser(), get_size))
162164
else:
163165
print((" #DATA = b'%s' # TODO(%s): Specify extra data if this isn't "
164-
"enough. Ensure the first %d bytes are sane/valid.") % (dummy_data, getpass.getuser(), get_size))
166+
"enough. Ensure the first %d bytes are sane/valid.") %
167+
(dummy_data, getpass.getuser(), get_size))
165168
print('')
166169
print('')
167170

@@ -277,14 +280,17 @@ def SetWithExtraData(names, pid, pid_test_base_name, set_size):
277280
dummy_data = GenerateDummyData(set_size)
278281
if dummy_data is None:
279282
print((" #DATA = b'foo' # TODO(%s): Specify extra data if this isn't "
280-
"enough. Ensure the first %d bytes are sane/valid.") % (getpass.getuser(), set_size))
283+
"enough. Ensure the first %d bytes are sane/valid.") %
284+
(getpass.getuser(), set_size))
281285
elif dummy_data != 'foo':
282286
# Doesn't match default, explicitly set value
283287
print((" DATA = b'%s' # TODO(%s): Specify extra data if this isn't "
284-
"enough. Ensure the first %d bytes are sane/valid.") % (dummy_data, getpass.getuser(), set_size))
288+
"enough. Ensure the first %d bytes are sane/valid.") %
289+
(dummy_data, getpass.getuser(), set_size))
285290
else:
286291
print((" #DATA = b'%s' # TODO(%s): Specify extra data if this isn't "
287-
"enough. Ensure the first %d bytes are sane/valid.") % (dummy_data, getpass.getuser(), set_size))
292+
"enough. Ensure the first %d bytes are sane/valid.") %
293+
(dummy_data, getpass.getuser(), set_size))
288294
print('')
289295
print('')
290296

@@ -353,7 +359,7 @@ def main():
353359
(pid.GetRequest(PidStore.RDM_GET).HasAtoms())):
354360
for atom in pid.GetRequest(PidStore.RDM_GET).GetAtoms():
355361
get_size += atom.size
356-
#TODO(Peter): Should we just print this total all the time?
362+
# TODO(Peter): Should we just print this total all the time?
357363
if get_size != pid.GetRequest(PidStore.RDM_GET).GetAtoms()[0].size:
358364
print('# Get requires %d bytes' % (get_size))
359365

0 commit comments

Comments
 (0)