Resolving linting errors

Author: j-ororke

src/python_testing/support_modules/idm_support.py

@@ -282,9 +282,9 @@ def verify_attribute_path(self, read_response: dict, path: AttributePath):
                         # Extra assertion to ensure cluster_id exists before accessing (defense in depth)
                         asserts.assert_in(cluster_id, read_response.tlvAttributes[ep],
                                           f"Cluster {cluster_id} not found in endpoint {ep}")
-                        returned_attrs = sorted([x for x in read_response.tlvAttributes[ep][cluster_id].keys()])
-                        attr_list = sorted([x for x in read_response.tlvAttributes[ep][cluster_id][
-                            ClusterObjects.ALL_CLUSTERS[cluster_id].Attributes.AttributeList.attribute_id]])
+                        returned_attrs = sorted(list(read_response.tlvAttributes[ep][cluster_id].keys()))
+                        attr_list = sorted(list(read_response.tlvAttributes[ep][cluster_id][
+                            ClusterObjects.ALL_CLUSTERS[cluster_id].Attributes.AttributeList.attribute_id]))
                         asserts.assert_equal(
                             returned_attrs,
                             attr_list,
@@ -326,7 +326,7 @@ def verify_all_endpoints_clusters(self, read_response: dict):
                 "PartsList attribute should always be present")
 
             # Server list matches returned clusters
-            returned_clusters = sorted(list(read_response.tlvAttributes[endpoint].keys()))
+            returned_clusters = sorted(read_response.tlvAttributes[endpoint].keys())
             server_list = sorted(read_response.tlvAttributes[endpoint][Clusters.Descriptor.id][
                 Clusters.Descriptor.Attributes.ServerList.attribute_id])
             asserts.assert_equal(returned_clusters, server_list,
@@ -339,9 +339,9 @@ def verify_all_endpoints_clusters(self, read_response: dict):
                 if global_attribute_ids.cluster_id_type(cluster) != global_attribute_ids.ClusterIdType.kStandard:
                     continue
 
-                returned_attrs = sorted([x for x in read_response.tlvAttributes[endpoint][cluster].keys()])
-                attr_list = sorted([x for x in read_response.tlvAttributes[endpoint][cluster][
-                    ClusterObjects.ALL_CLUSTERS[cluster].Attributes.AttributeList.attribute_id]])
+                returned_attrs = sorted(list(read_response.tlvAttributes[endpoint][cluster].keys()))
+                attr_list = sorted(list(read_response.tlvAttributes[endpoint][cluster][
+                    ClusterObjects.ALL_CLUSTERS[cluster].Attributes.AttributeList.attribute_id]))
                 asserts.assert_equal(returned_attrs, attr_list,
                                      f"Mismatch for {cluster} at endpoint {endpoint}")
 
@@ -407,7 +407,7 @@ async def read_cluster_all_endpoints(self, cluster):
             if global_attribute_ids.cluster_id_type(cluster.id) == global_attribute_ids.ClusterIdType.kStandard:
                 returned_attrs = sorted([x.attribute_id for x in read_request[endpoint][cluster].keys()
                                          if x != Clusters.Attribute.DataVersion])
-                attr_list = sorted([x for x in read_request[endpoint][cluster][cluster.Attributes.AttributeList]])
+                attr_list = sorted(list(read_request[endpoint][cluster][cluster.Attributes.AttributeList]))
                 asserts.assert_equal(
                     returned_attrs,
                     attr_list,
@@ -799,7 +799,7 @@ def verify_empty_wildcard(self, attr_path, read_request):
 
         # Server list validation
         for endpoint in read_request.tlvAttributes:
-            returned_clusters = sorted(list(read_request.tlvAttributes[endpoint].keys()))
+            returned_clusters = sorted(read_request.tlvAttributes[endpoint].keys())
             server_list = sorted(read_request.tlvAttributes[endpoint][Clusters.Descriptor.id]
                                  [Clusters.Descriptor.Attributes.ServerList.attribute_id])
             asserts.assert_equal(returned_clusters, server_list)