lints changes

project-chip · Jan 9, 2024 · 54445a1 · 54445a1
1 parent f260377
commit 54445a1
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 10 deletions.
diff --git a/src/controller/python/test/test_scripts/base.py b/src/controller/python/test/test_scripts/base.py
@@ -1437,7 +1437,6 @@ def TestSubscriptionResumptionCapacityStep1(self, nodeid: int, endpoint: int, su
 
     def TestSubscriptionResumptionCapacityStep2(self, nodeid: int, endpoint: int, remote_ip: str, ssh_port: int,
                                                 remote_server_app: str, subscription_capacity: int):
-        remoteDeviceRestarted = False
         updateLock = threading.Lock()
         updateCv = threading.Condition(updateLock)
 
@@ -1474,7 +1473,6 @@ def run(self):
                              os.path.join(CHIP_REPO, "out/debug/standalone", self.remote_server_app),
                              self.subscription_capacity))
                     with updateLock:
-                        remoteDeviceRestarted = True
                         updateCv.notifyAll()
 
                 finally:
@@ -1485,11 +1483,10 @@ def run(self):
             restartRemoteThread = _restartRemoteDevice(remote_ip, ssh_port, remote_server_app, subscription_capacity)
             restartRemoteThread.start()
             with updateCv:
-                while remoteDeviceRestarted is False:
-                    if not updateCv.wait(8.0):
-                        self.logger.error(
-                            "Failed to restart the remote device")
-                        break
+                if not updateCv.wait(8.0):
+                    self.logger.error(
+                        "Failed to restart the remote device")
+
             # Wait for some time so that the device will be resolving the address of the first controller
             time.sleep(3)
 

diff --git a/src/controller/python/test/test_scripts/subscription_resumption_capacity_test_ctrl1.py b/src/controller/python/test/test_scripts/subscription_resumption_capacity_test_ctrl1.py
@@ -21,7 +21,6 @@
 
 import os
 import sys
-import time
 from optparse import OptionParser
 
 from base import BaseTestHelper, FailIfNot, TestFail, TestTimeout, logger

diff --git a/src/controller/python/test/test_scripts/subscription_resumption_capacity_test_ctrl2.py b/src/controller/python/test/test_scripts/subscription_resumption_capacity_test_ctrl2.py
@@ -21,7 +21,6 @@
 
 import os
 import sys
-import time
 from optparse import OptionParser
 
 from base import BaseTestHelper, FailIfNot, TestFail, TestTimeout, logger

diff --git a/src/controller/python/test/test_scripts/subscription_resumption_test.py b/src/controller/python/test/test_scripts/subscription_resumption_test.py
@@ -21,7 +21,6 @@
 
 import os
 import sys
-import time
 from optparse import OptionParser
 
 from base import BaseTestHelper, FailIfNot, TestFail, TestTimeout, logger