commit 7a722263825ddcb193075f0893c1f497b260ba37
author Clemenz Portmann <portmannc@google.com> Wed Apr 06 09:58:07 2022 -0700
committer Clemenz Portmann <portmannc@google.com> Wed Apr 06 09:58:07 2022 -0700
tree 1a2c1fd6d6ac840616c23bda745672a4912e3e46
parent 7a467f1c1e7d82cb42366ee0487d227481f8a280

ITS: clean up pylint errors in tools/run_all_tests.py

bug: 227289987
Change-Id: I41a8a31adf22846de79fe0f0ec497339df811952

apps/CameraITS/tools/run_all_tests.py

diff --git a/apps/CameraITS/tools/run_all_tests.py b/apps/CameraITS/tools/run_all_tests.py
index d30c010..3a983b4 100644
--- a/apps/CameraITS/tools/run_all_tests.py
+++ b/apps/CameraITS/tools/run_all_tests.py
@@ -529,10 +529,9 @@
           # socket FAILs.
           with open(MOBLY_TEST_SUMMARY_TXT_FILE, 'r') as file:
             test_code = output.returncode
-            test_failed = False
             test_skipped = False
             test_not_yet_mandated = False
-            test_mpc_req = ""
+            test_mpc_req = ''
             content = file.read()
 
             # Find media performance class logging
@@ -542,7 +541,8 @@
               # MPC12_CAMERA_LAUNCH_PATTERN or MPC12_JPEG_CAPTURE_PATTERN in
               # ItsTestActivity.java.
               mpc_string_match = re.search(
-                  '^(1080p_jpeg_capture_time_ms:|camera_launch_time_ms:)', one_line)
+                  '^(1080p_jpeg_capture_time_ms:|camera_launch_time_ms:)',
+                  one_line)
               if mpc_string_match:
                 test_mpc_req = one_line
                 break
@@ -570,7 +570,6 @@
                 logging.info('Retry %s/%s', s, test)
               else:
                 num_fail += 1
-                test_failed = True
                 break
             os.remove(MOBLY_TEST_SUMMARY_TXT_FILE)
         logging.info('%s %s/%s', return_string, s, test)