Apply formatter and fix linter issues

Signed-off-by: Sathvik K Gatti <[email protected]>
Signed-off-by: Suyambulingam Rathinasamy Muthupandi <[email protected]>
Signed-off-by: Charles Kim <[email protected]>

Author: s-gatti

test_collections/matter/sdk_tests/support/chip/chip_server.py

@@ -122,7 +122,6 @@ async def start(
         self.__use_paa_certs = use_paa_certs
         self.__server_type = server_type
 
-
         if server_type == ChipServerType.CHIP_CAMERA_CONTROLLER:
             prefix = CHIP_CAMERA_CONTROLLER_EXE
             command = ["interactive", "server"]

test_collections/matter/sdk_tests/support/yaml_tests/models/chip_suite.py

@@ -89,7 +89,10 @@ async def setup(self) -> None:
             self.runner.reset_pics_state()
 
         self.__dut_commissioned_successfully = False
-        if self.server_type == ChipServerType.CHIP_TOOL or self.server_type == ChipServerType.CHIP_CAMERA_CONTROLLER:
+        if (
+            self.server_type == ChipServerType.CHIP_TOOL
+            or self.server_type == ChipServerType.CHIP_CAMERA_CONTROLLER
+        ):
             logger.info("Commission DUT")
             user_response = await prompt_for_commissioning_mode(
                 self, logger, None, self.cancel
@@ -227,7 +230,10 @@ async def cleanup(self) -> None:
         # Only unpair if commissioning was successfull during setup
         if self.__dut_commissioned_successfully:
             # Unpair is not applicable for simulated apps case
-            if self.server_type == ChipServerType.CHIP_TOOL or self.server_type == ChipServerType.CHIP_CAMERA_CONTROLLER:
+            if (
+                self.server_type == ChipServerType.CHIP_TOOL
+                or self.server_type == ChipServerType.CHIP_CAMERA_CONTROLLER
+            ):
                 logger.info("Unpairing DUT from server")
                 await self.runner.unpair()
             elif self.server_type == ChipServerType.CHIP_APP:

test_collections/matter/sdk_tests/support/yaml_tests/models/test_case.py

@@ -28,8 +28,8 @@
 from ...chip.chip_server import ChipServerType
 from ...models.matter_test_models import MatterTestStep, MatterTestType
 from ...yaml_tests.models.chip_test import ChipManualPromptTest, ChipTest
-from .yaml_test_models import YamlTest
 from .test_suite import SuiteType
+from .yaml_test_models import YamlTest
 
 # Custom type variable used to annotate the factory method in YamlTestCase.
 T = TypeVar("T", bound="YamlTestCase")
@@ -173,7 +173,8 @@ def _append_automated_test_step(self, yaml_step: MatterTestStep) -> None:
         Disabled steps are ignored.
         (Such tests will be marked as 'Steps Disabled' elsewhere)
 
-        UserPrompt, PromptWithResponse or VerifyVideoStream are special cases that will prompt test operator for input.
+        UserPrompt, PromptWithResponse or VerifyVideoStream are special cases that will
+        prompt test operator for input.
         """
         if yaml_step.disabled:
             test_engine_logger.info(
@@ -182,7 +183,11 @@ def _append_automated_test_step(self, yaml_step: MatterTestStep) -> None:
             return
 
         step = TestStep(yaml_step.label)
-        if yaml_step.command in ["UserPrompt", "PromptWithResponse", "VerifyVideoStream"]:
+        if yaml_step.command in [
+            "UserPrompt",
+            "PromptWithResponse",
+            "VerifyVideoStream",
+        ]:
             step = ManualVerificationTestStep(
                 name=yaml_step.label,
                 verification=yaml_step.verification,

test_collections/matter/sdk_tests/support/yaml_tests/models/test_declarations.py

@@ -13,7 +13,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-from typing import Type
+from typing import Optional, Type
 
 from app.test_engine.models.test_declarations import (
     TestCaseDeclaration,
@@ -26,7 +26,6 @@
 from .test_case import YamlTestCase
 from .test_suite import SuiteType, YamlTestSuite
 from .yaml_test_models import YamlTest
-from typing import Optional
 
 
 class YamlCollectionDeclaration(TestCollectionDeclaration):

test_collections/matter/sdk_tests/support/yaml_tests/models/yaml_test_models.py

@@ -13,14 +13,12 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-from typing import Any
+from typing import Any, Optional
 
 from pydantic_yaml import YamlModelMixin
 
 from ...models.matter_test_models import MatterTest
-
 from .test_suite import SuiteType
-from typing import Optional
 
 ###
 # This file declares YAML models that are used to parse the YAML Test Cases.
@@ -29,5 +27,6 @@
 
 class YamlTest(YamlModelMixin, MatterTest):
     suite_type: Optional[SuiteType] = None
+
     def __init__(self, **kwargs: Any) -> None:
         super().__init__(steps=kwargs["tests"], **kwargs)

test_collections/matter/sdk_tests/support/yaml_tests/models/yaml_test_parser.py

@@ -13,15 +13,15 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
+import re
 from pathlib import Path
 
 from loguru import logger
 from pydantic import ValidationError
 
 from ...models.matter_test_models import MatterTestType
-from .yaml_test_models import YamlTest
 from .test_suite import SuiteType
-import re
+from .yaml_test_models import YamlTest
 
 
 class YamlParserException(Exception):
@@ -60,8 +60,12 @@ def _get_types(test: YamlTest) -> tuple[MatterTestType, SuiteType]:
     if all(s.disabled is True for s in steps):
         return MatterTestType.MANUAL, SuiteType.MANUAL
 
-    # if any step has a UserPrompt, PromptWithResponse or VerifyVideoStream command, categorize as semi-automated
-    if any(s.command in ["UserPrompt", "PromptWithResponse", "VerifyVideoStream"] for s in steps):
+    # if any step has a UserPrompt, PromptWithResponse or VerifyVideoStream command,
+    # categorize as semi-automated
+    if any(
+        s.command in ["UserPrompt", "PromptWithResponse", "VerifyVideoStream"]
+        for s in steps
+    ):
         if re.search(camera_test_pattern, test.name):
             return MatterTestType.SEMI_AUTOMATED, SuiteType.CAMERA_AUTOMATED
         else:
@@ -72,7 +76,8 @@ def _get_types(test: YamlTest) -> tuple[MatterTestType, SuiteType]:
     if re.search(camera_test_pattern, test.name):
         return MatterTestType.AUTOMATED, SuiteType.CAMERA_AUTOMATED
     else:
-        return MatterTestType.AUTOMATED, SuiteType.AUTOMATED    
+        return MatterTestType.AUTOMATED, SuiteType.AUTOMATED
+
 
 def parse_yaml_test(path: Path) -> YamlTest:
     """Parse a single YAML file into YamlTest model.

test_collections/matter/sdk_tests/support/yaml_tests/sdk_yaml_tests.py

@@ -76,7 +76,7 @@ def _init_test_suites(yaml_version: str) -> dict[SuiteType, YamlSuiteDeclaration
             name="CameraAppTestSuite",
             suite_type=SuiteType.CAMERA_AUTOMATED,
             version=yaml_version,
-        )
+        ),
     }
 
 
@@ -110,9 +110,9 @@ def _parse_all_yaml(
                 suites[SuiteType.SIMULATED].add_test_case(test_case)
             else:
                 if test_case.suite_type == SuiteType.CAMERA_AUTOMATED:
-                  suites[SuiteType.CAMERA_AUTOMATED].add_test_case(test_case)
+                    suites[SuiteType.CAMERA_AUTOMATED].add_test_case(test_case)
                 else:
-                  suites[SuiteType.AUTOMATED].add_test_case(test_case)
+                    suites[SuiteType.AUTOMATED].add_test_case(test_case)
 
         except YamlParserException:
             # If an exception was raised during parse process, the yaml file will be