commit 5c0fc9fbe43e1a2a7cbf9170a21714f64cb0370c
author Rob Mohr <mohrr@google.com> Tue Aug 13 05:14:25 2024 +0000
committer CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Aug 13 05:14:25 2024 +0000
tree 874fd7dc094a850eb50eb779cb777922b3badccf
parent 6ff9e50d1327c4dcc67cd783da2bc04dc843dc01

txt_roller: Reformat tests

Change-Id: Id654ec9c7b20d7292988542caa701e77575b3a18
Reviewed-on: https://pigweed-review.googlesource.com/c/infra/recipes/+/228966
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
Pigweed-Auto-Submit: Rob Mohr <mohrr@google.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
Lint: Lint 🤖 <android-build-ayeaye@system.gserviceaccount.com>
Reviewed-by: Danielle Kay <danikay@google.com>

recipes/txt_roller.py

diff --git a/recipes/txt_roller.py b/recipes/txt_roller.py
index 60e3bf3..d920116 100644
--- a/recipes/txt_roller.py
+++ b/recipes/txt_roller.py
@@ -181,32 +181,33 @@
             **kwargs,
         )
 
-    yield (
-        api.test('success')
-        + properties(txt_path='foo.txt', project_remote=_url("foo"))
-        + api.roll_util.properties(commit_divider='--divider--')
-        + trigger('foo')
-        + api.roll_util.forward_roll()
-        + commit_data('foo.txt', prefix='')
-        + api.auto_roller.success()
+    yield api.test(
+        'success',
+        properties(txt_path='foo.txt', project_remote=_url("foo")),
+        api.roll_util.properties(commit_divider='--divider--'),
+        trigger('foo'),
+        api.roll_util.forward_roll(),
+        commit_data('foo.txt', prefix=''),
+        api.auto_roller.success(),
     )
 
-    yield (
-        api.test('bad-trigger', status='FAILURE')
-        + properties(txt_path='foo.txt', project_remote=_url("foo"))
-        + trigger('bar')
+    yield api.test(
+        'bad-trigger',
+        properties(txt_path='foo.txt', project_remote=_url("foo")),
+        trigger('bar'),
+        status='FAILURE',
     )
 
-    yield (
-        api.test('no-trigger')
-        + properties(txt_path='foo.txt', project_remote=_url("foo"))
-        + api.roll_util.forward_roll()
-        + commit_data('foo.txt', prefix='')
-        + api.auto_roller.success()
+    yield api.test(
+        'no-trigger',
+        properties(txt_path='foo.txt', project_remote=_url("foo")),
+        api.roll_util.forward_roll(),
+        commit_data('foo.txt', prefix=''),
+        api.auto_roller.success(),
     )
 
-    yield (
-        api.test('backwards')
-        + properties(txt_path='foo.txt', project_remote=_url("foo"))
-        + api.roll_util.backward_roll()
+    yield api.test(
+        'backwards',
+        properties(txt_path='foo.txt', project_remote=_url("foo")),
+        api.roll_util.backward_roll(),
     )