address review

Author: blink1073

test/asynchronous/unified_format.py

@@ -803,9 +803,7 @@ async def _collectionOperation_assertIndexNotExists(self, target, **kwargs):
     async def _collectionOperation_assertCollectionExists(self, target, **kwargs):
         database_name = kwargs["database_name"]
         collection_name = kwargs["collection_name"]
-        collection_name_list = list(
-            await self.client.get_database(database_name).list_collection_names()
-        )
+        collection_name_list = await self.client.get_database(database_name).list_collection_names()
         self.assertIn(collection_name, collection_name_list)
 
     async def _databaseOperation_assertIndexExists(self, target, **kwargs):
@@ -821,9 +819,7 @@ async def _databaseOperation_assertIndexNotExists(self, target, **kwargs):
     async def _databaseOperation_assertCollectionExists(self, target, **kwargs):
         database_name = kwargs["database_name"]
         collection_name = kwargs["collection_name"]
-        collection_name_list = list(
-            await self.client.get_database(database_name).list_collection_names()
-        )
+        collection_name_list = await self.client.get_database(database_name).list_collection_names()
         self.assertIn(collection_name, collection_name_list)
 
     async def kill_all_sessions(self):

test/unified_format.py

@@ -800,7 +800,7 @@ def _collectionOperation_assertIndexNotExists(self, target, **kwargs):
     def _collectionOperation_assertCollectionExists(self, target, **kwargs):
         database_name = kwargs["database_name"]
         collection_name = kwargs["collection_name"]
-        collection_name_list = list(self.client.get_database(database_name).list_collection_names())
+        collection_name_list = self.client.get_database(database_name).list_collection_names()
         self.assertIn(collection_name, collection_name_list)
 
     def _databaseOperation_assertIndexExists(self, target, **kwargs):
@@ -816,7 +816,7 @@ def _databaseOperation_assertIndexNotExists(self, target, **kwargs):
     def _databaseOperation_assertCollectionExists(self, target, **kwargs):
         database_name = kwargs["database_name"]
         collection_name = kwargs["collection_name"]
-        collection_name_list = list(self.client.get_database(database_name).list_collection_names())
+        collection_name_list = self.client.get_database(database_name).list_collection_names()
         self.assertIn(collection_name, collection_name_list)
 
     def kill_all_sessions(self):