Changes after code review

app/chip_tool/chip_tool.py

@@ -348,7 +348,7 @@ def __get_gateway_ip(self) -> str:
 
     async def start_container_no_server(self) -> None:
         """
-        Creates the chip-tool container without any service running
+        Creates the chip-tool container without any server running
         (ChipTool or ChipApp).
         """
         # Ensure there's no existing container running using the same name.

app/chip_tool/test_case.py

@@ -225,7 +225,6 @@ async def setup(self) -> None:
             )
 
         self.chip_tool = ChipTool()
-        await self.chip_tool.start_container_no_server()
 
         # Use test engine logger to log all events to test run.
         self.chip_tool.logger = test_engine_logger

test_collections/sdk_tests/__init__.py

@@ -13,8 +13,5 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
+from .support.python_testing import sdk_python_collection
 from .support.yaml_tests import custom_collection, sdk_collection
-
-# TODO The Python test support implementation is ongoing.
-# Uncomment line below when the implementation is done.
-# from .support.python_testing import sdk_python_collection

test_collections/sdk_tests/support/python_testing/models/python_testing_hooks_proxy.py

@@ -23,14 +23,14 @@ class SDKPythonTestRunnerHooks(TestRunnerHooks):
     is_finished = False
     results: Queue
 
-    def updates_test(self) -> Union[dict, None]:
+    def update_test(self) -> Union[dict, None]:
         try:
             result = self.results.get(block=False)
             return result
         except Empty:
             return None
 
-    def finished(self) -> bool:
+    def is_finished(self) -> bool:
         return SDKPythonTestRunnerHooks.is_finished
 
     def __init__(self) -> None: