commit 80ff454da817905beb67af6759af3e447d535de9
author Mike Frysinger <vapier@chromium.org> Fri May 06 23:52:04 2022 -0400
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> Sun May 08 07:02:43 2022 +0000
tree fced8d029571aeccd89929688bb5337a9fa89616
parent 404435963d45dea01da5aec5a12ee8a874adf122

lint: use generators with all/any helpers

Rather than generate lists before passing to all/any functions, create
a generator and pass that directly.  This avoids creating a temporary
list that we then immediately throw away.

BUG=b:231761335
TEST=`./run_tests` passes & CQ passes

Change-Id: Ia22c0118e9a7fadbfa8e59535125d2157761e59b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/3633287
Commit-Queue: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Chris McDonald <cjmcdonald@chromium.org>

api/controller/artifacts_unittest.py

diff --git a/api/controller/artifacts_unittest.py b/api/controller/artifacts_unittest.py
index d1ec749..4641ba7 100644
--- a/api/controller/artifacts_unittest.py
+++ b/api/controller/artifacts_unittest.py
@@ -1032,5 +1032,5 @@
     artifacts.FetchMetadata(request, response, self.api_config)
     actual_filepaths = [fp.path.path for fp in response.filepaths]
     self.assertEqual(sorted(actual_filepaths), sorted(self.expected_filepaths))
-    self.assertTrue(all([fp.path.location == common_pb2.Path.OUTSIDE
-                         for fp in response.filepaths]))
+    self.assertTrue(all(fp.path.location == common_pb2.Path.OUTSIDE
+                        for fp in response.filepaths))