[Python] Lint for unintentional tuples (#41131)

* Bump autopep8 version in restyled configuration

* Check for unintentional tuple

* Restyled by autopep8

* Improve codestyle

---------

Co-authored-by: Restyled.io <[email protected]>

Author: arkq

.restyled.yaml

@@ -224,7 +224,7 @@ restylers:
           - "**/*.sh"
           - "**/*.bash"
     - name: autopep8
-      image: restyled/restyler-autopep8:v2.0.0
+      image: restyled/restyler-autopep8:v2.3.0
       command:
           - autopep8
           - "--in-place"

credentials/generate_revocation_set.py

@@ -864,7 +864,7 @@ def cli():
 @optgroup.option('--use-test-net-dcld', type=str, default='', metavar='PATH', help="Location of `dcld` binary, to use `dcld` for mirroring TestNet.")
 @optgroup.option('--use-main-net-http', is_flag=True, type=str, help="Use RESTful API with HTTPS against public MainNet observer.")
 @optgroup.option('--use-test-net-http', is_flag=True, type=str, help="Use RESTful API with HTTPS against public TestNet observer.")
-@optgroup.option('--use-local-data', is_flag=True, type=bool, help="Fake response directory: see \" DATA_DIR/",)
+@optgroup.option('--use-local-data', is_flag=True, type=bool, help="Fake response directory: see \" DATA_DIR/")
 @optgroup.group('Required arguments if use-local-data is used', cls=AllOptionGroup)
 @optgroup.option('--certificates', type=click.File('rb'), multiple=True, help='Paths to PEM formated certificates (i.e. PAA) in DCL but missing from the revocation-points-response file.')
 @optgroup.option('--crls', type=click.File('rb'), multiple=True, help='Paths to the crl der files')

pyproject.toml

@@ -22,7 +22,8 @@ exclude = [
 ]
 
 [tool.ruff.lint]
-select = ["E4", "E7", "E9", "F"]
+select = ["COM", "E1", "E4", "E7", "E9", "F", "W6"]
 ignore = [
+    "COM812", # Do not enforce trailing commas in multi-line collections
     "E721", # We use it for good reasons
 ]

scripts/examples/tests/test.py

@@ -40,7 +40,7 @@ def build_actual_output(root: str, out: str) -> List[str]:
     subprocess.run([
         binary,
         project,
-    ], stdout=subprocess.PIPE, check=True, encoding='UTF-8', )
+    ], stdout=subprocess.PIPE, check=True, encoding='UTF-8')
 
     with open(cmake, 'rt') as f:
         for line in f.readlines():

scripts/py_matter_idl/matter/idl/test_matter_idl_parser.py

@@ -748,7 +748,7 @@ def test_endpoints(self):
                                                ServerClusterInstantiation(
                                                    name="Bar"),
                                            ],
-                                           client_bindings=["Bar", "Test"],)
+                                           client_bindings=["Bar", "Test"])
                                   ])
         self.assertIdlEqual(actual, expected)
 
@@ -783,7 +783,7 @@ def test_cluster_instantiation(self):
                                                                           storage=AttributeStorage.CALLBACK, default=True),
                                                ]),
                                            ],
-                                           client_bindings=[],)
+                                           client_bindings=[])
                                   ])
         self.assertIdlEqual(actual, expected)
 
@@ -908,7 +908,7 @@ def test_cluster_reference_globals(self):
                     ],
                     structs=[
                         Struct(name="MyStruct", fields=[
-                            Field(name="subStruct", code=0, data_type=DataType(name="TestStruct"), qualities=FieldQuality.NULLABLE), ],
+                            Field(name="subStruct", code=0, data_type=DataType(name="TestStruct"), qualities=FieldQuality.NULLABLE)],
                         ),
                         global_struct,
                     ],

scripts/py_matter_idl/matter/idl/test_zapxml.py

@@ -347,7 +347,7 @@ def testFeatures(self):
                          Idl(clusters=[
                              Cluster(name='TestFeatures',
                                      code=20, bitmaps=[bitmap])
-                         ])),
+                         ]))
 
     def testGlobalStruct(self):
         idl = XmlToIdl('''<?xml version="1.0"?>
@@ -447,7 +447,7 @@ def testStruct(self):
                                              ),
                                              qualities=AttributeQuality.READABLE,
                                              readacl=AccessPrivilege.VIEW,
-                                             writeacl=AccessPrivilege.OPERATE)]), ]))
+                                             writeacl=AccessPrivilege.OPERATE)])]))
 
     def testSkipsNotProcessedFields(self):
         # Zap has extra fields that are generally not processed
@@ -497,7 +497,7 @@ def testSkipsNotProcessedFields(self):
                                              ),
                                              qualities=AttributeQuality.READABLE,
                                              readacl=AccessPrivilege.VIEW,
-                                             writeacl=AccessPrivilege.OPERATE)]), ]))
+                                             writeacl=AccessPrivilege.OPERATE)])]))
 
 
 if __name__ == '__main__':

scripts/py_matter_yamltests/matter/yamltests/parser.py

@@ -87,8 +87,8 @@ def __init__(self, content, target_name, candidate_names=[]):
 
 class PostProcessCheckStatus(Enum):
     '''Indicates the post processing check step status.'''
-    SUCCESS = 'success',
-    WARNING = 'warning',
+    SUCCESS = 'success'
+    WARNING = 'warning'
     ERROR = 'error'
 
 

scripts/tests/run_fuzztest_coverage.py

@@ -137,7 +137,7 @@ def run_fuzz_test(context):
 
     try:
         if context.run_mode == FuzzTestMode.UNIT_TEST_MODE:
-            subprocess.run([context.fuzz_test_binary_path, ], env=env, check=True)
+            subprocess.run([context.fuzz_test_binary_path], env=env, check=True)
             logging.info("Fuzz Test Suite executed in Unit Test Mode.\n")
         elif context.run_mode == FuzzTestMode.CONTINUOUS_FUZZ_MODE:
             cmd_args = [context.fuzz_test_binary_path, f"--fuzz={context.selected_fuzz_test_case}"]

scripts/tools/bouffalolab/generate_factory_data.py

@@ -227,7 +227,7 @@ def convert_pem_to_der(chip_cert, action, pem):
 
         der = Path(pem).with_suffix(".der")
         if not os.path.isfile(der):
-            cmd = [chip_cert, action, pem, der, "--x509-der", ]
+            cmd = [chip_cert, action, pem, der, "--x509-der"]
             subprocess.run(cmd)
 
         return der

scripts/tools/memory/memdf/report.py

@@ -69,7 +69,7 @@ def postprocess_report_by(config: Config, key: str, info: Mapping) -> None:
     """For --report-by=region, select all sections."""
     assert key == 'report.by'
     if config.get(key) == 'region':
-        config.put('section.select-all', True),
+        config.put('section.select-all', True)
 
 
 REPORT_BY_CONFIG: ConfigDescription = {