Skip to content

Commit 347c5ab

Browse files
committed
lint code
1 parent a9d6d85 commit 347c5ab

File tree

2 files changed

+14
-15
lines changed

2 files changed

+14
-15
lines changed

src/python_testing/matter_testing_infrastructure/chip/testing/matter_testing.py

+2-5
Original file line numberDiff line numberDiff line change
@@ -43,8 +43,8 @@
4343

4444
import chip.testing.conversions as conversions
4545
import chip.testing.matchers as matchers
46-
import chip.testing.timeoperations as timeoperations
4746
import chip.testing.runner as runner
47+
import chip.testing.timeoperations as timeoperations
4848
from chip.tlv import uint
4949

5050
# isort: off
@@ -70,12 +70,10 @@
7070
from chip.storage import PersistentStorage
7171
from chip.testing.commissioning import CommissioningInfo, CustomCommissioningParameters, SetupPayloadInfo, commission_devices
7272
from chip.testing.global_attribute_ids import GlobalAttributeIds
73-
from chip.testing.runner import get_test_info, run_tests_no_exit, run_tests, TestStep, TestInfo, TestRunnerHooks
7473
from chip.testing.pics import read_pics_from_file
75-
from chip.tracing import TracingContext
74+
from chip.testing.runner import TestRunnerHooks, TestStep
7675
from mobly import asserts, base_test, signals, utils
7776

78-
7977
# TODO: Add utility to commission a device if needed
8078
# TODO: Add utilities to keep track of controllers/fabrics
8179

@@ -895,7 +893,6 @@ def write_to_app_pipe(self, command_dict: dict, app_pipe_name: Optional[str] = N
895893
import json
896894
command = json.dumps(command_dict)
897895

898-
import os
899896
dut_ip = os.getenv('LINUX_DUT_IP')
900897

901898
if dut_ip is None:

src/python_testing/matter_testing_infrastructure/chip/testing/runner.py

+12-10
Original file line numberDiff line numberDiff line change
@@ -17,20 +17,21 @@
1717

1818
import asyncio
1919
import importlib
20-
import sys
2120
import logging
22-
import typing
2321
import os
24-
from typing import Optional
22+
import sys
23+
import typing
24+
from dataclasses import dataclass
25+
from datetime import datetime, timedelta, timezone
2526
from pathlib import Path
27+
from typing import Optional
2628
from unittest.mock import MagicMock
27-
from dataclasses import dataclass
28-
from datetime import datetime, timezone, timedelta
2929

3030
from chip.clusters import Attribute
31+
from mobly import signals
3132
from mobly.config_parser import ENV_MOBLY_LOGPATH, TestRunConfig
3233
from mobly.test_runner import TestRunner
33-
from mobly import signals
34+
3435
try:
3536
from matter_yamltests.hooks import TestRunnerHooks
3637
except ImportError:
@@ -50,6 +51,7 @@ def StartFromString(self, destination):
5051
pass
5152

5253
from typing import TYPE_CHECKING
54+
5355
if TYPE_CHECKING:
5456
from chip.testing.matter_testing import MatterTestConfig
5557

@@ -152,7 +154,7 @@ def default_matter_test_main():
152154
default_matter_test_main()
153155
"""
154156

155-
from chip.testing.matter_testing import parse_matter_test_args, _find_test_class
157+
from chip.testing.matter_testing import _find_test_class, parse_matter_test_args
156158

157159
matter_test_config = parse_matter_test_args()
158160

@@ -184,10 +186,10 @@ def run_tests_no_exit(test_class, matter_test_config,
184186
event_loop: asyncio.AbstractEventLoop, hooks: TestRunnerHooks,
185187
default_controller=None, external_stack=None) -> bool:
186188

187-
from chip.testing.matter_testing import MatterStackState, stash_globally
188-
189189
# Lazy import to avoid circular dependency
190190
from typing import TYPE_CHECKING
191+
192+
from chip.testing.matter_testing import MatterStackState, stash_globally
191193
if TYPE_CHECKING:
192194
from chip.testing.matter_testing import CommissionDeviceTest
193195
else:
@@ -315,7 +317,7 @@ class MockTestRunner():
315317
def __init__(self, abs_filename: str, classname: str, test: str, endpoint: int = None,
316318
pics: dict[str, bool] = None, paa_trust_store_path=None):
317319

318-
from chip.testing.matter_testing import MatterTestConfig, MatterStackState
320+
from chip.testing.matter_testing import MatterStackState, MatterTestConfig
319321

320322
self.kvs_storage = 'kvs_admin.json'
321323
self.config = MatterTestConfig(endpoint=endpoint, paa_trust_store_path=paa_trust_store_path,

0 commit comments

Comments
 (0)