Skip to content

Commit

Permalink
Fix some pylint issues (#437)
Browse files Browse the repository at this point in the history
  • Loading branch information
jboddey authored and sofyakurilova committed May 14, 2024
1 parent 3a6271d commit 6013944
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 5 deletions.
2 changes: 1 addition & 1 deletion framework/python/src/common/session.py
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ def _load_version(self, default_version):
version_cmd = util.run_command(
'dpkg-query --showformat=\'${Version}\' --show testrun')
# index 1 of response is the stderr byte stream so if
# it has any data in it, there was an error and we
# it has any data in it, there was an error and we
# did not resolve the version and we'll use the fallback
if len(version_cmd[1]) == 0:
version = version_cmd[0]
Expand Down
3 changes: 2 additions & 1 deletion modules/test/tls/python/src/tls_module.py
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,8 @@ def __init__(self,
# # cert_table = (f'| Property | Value |\n'
# # f'|---|---|\n'
# # f"| {'Version':<17} | {version_value:^25} |\n"
# # f"| {'Signature Alg.':<17} | {signature_alg_value:^25} |\n"
# # f"| {'Signature Alg.':<17} |
# {signature_alg_value:^25} |\n"
# # f"| {'Validity from':<17} | {not_before:^25} |\n"
# # f"| {'Valid to':<17} | {not_after:^25} |")

Expand Down
2 changes: 1 addition & 1 deletion testing/api/test_api.py
Original file line number Diff line number Diff line change
Expand Up @@ -541,7 +541,7 @@ def test_multiple_runs(testing_devices, testrun):
@pytest.mark.skip()
def test_create_invalid_chars(empty_devices_dir, testrun):
# local_delete_devices(ALL_DEVICES)
# We must start test run with no devices in local/devices for this test
# We must start test run with no devices in local/devices for this test
# to function as expected!
assert len(local_get_devices()) == 0

Expand Down
3 changes: 2 additions & 1 deletion testing/tests/test_tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,8 @@ def test_list_tests(capsys, results, test_matrix):
for test in collect_actual_results(results[tester]):
if test.name in test_matrix[tester]['expected_results']:
print(
f' {test.name}: {test.result} (exp: {test_matrix[tester]["expected_results"][test.name]})'
f''' {test.name}: {test.result} (exp: {test_matrix[
tester]["expected_results"][test.name]})'''
)
else:
print(f' {test.name}: {test.result}')
Expand Down
3 changes: 2 additions & 1 deletion testing/unit/tls/tls_module_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -476,7 +476,8 @@ def tls_module_ca_cert_spaces_test(self):
cert_out_dir=OUTPUT_DIR,
root_certs_dir=tmp_dir)

cert_valid = tls_util.validate_local_ca_signature(device_cert_path=cert_path)
cert_valid = tls_util.validate_local_ca_signature(
device_cert_path=cert_path)
self.assertEqual(cert_valid[0], True)


Expand Down

0 comments on commit 6013944

Please sign in to comment.