commit 524fc3b5d46ca8fb7025cf29457592862e6c6d0c
author Greg Edelston <gredelston@google.com> Mon Aug 31 11:01:14 2020 -0600
committer Greg Edelston <gredelston@google.com> Wed Sep 02 17:30:14 2020 +0000
tree 59333df2557725480b2562db5ce43587955a56ab
parent 2bf4943e8b2e375e018d1ad6046b527aa63a4a32

au_test_harness: Rename whitelist_chrome_crashes

Per go/COIL, we are renaming the test_that command-line arg
'whitelist-chrome-crashes' to 'allow-chrome-crashes'.
This CL changes the au_test_harness scripts to accommodate that change.

Uploaded with --no-verify because the lint errors were nontrivial to
fix.

BUG=b:159917945
TEST=None

Cq-Depend: chromium:2385861
Change-Id: Ia810d25b5f4605d38d2839b90ef7030159c1a3cb
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crostestutils/+/2385862
Tested-by: Greg Edelston <gredelston@google.com>
Reviewed-by: Sean Abraham <seanabraham@chromium.org>

au_test_harness/cros_au_test_harness.py

diff --git a/au_test_harness/cros_au_test_harness.py b/au_test_harness/cros_au_test_harness.py
index 2e4ec71..8a1ddc5 100755
--- a/au_test_harness/cros_au_test_harness.py
+++ b/au_test_harness/cros_au_test_harness.py
@@ -182,8 +182,8 @@
   parser.add_argument('--verbose', default=True, action='store_true',
                       help='Print out rather than capture output as much as '
                       'possible.')
-  parser.add_argument('--whitelist_chrome_crashes', default=False,
-                      dest='whitelist_chrome_crashes', action='store_true',
+  parser.add_argument('--allow_chrome_crashes', default=False,
+                      dest='allow_chrome_crashes', action='store_true',
                       help='Treat Chrome crashes as non-fatal.')
   parser.add_argument('--verify_suite_name', default=None,
                       help='Specify the verify suite to run.')

au_test_harness/vm_au_worker.py

diff --git a/au_test_harness/vm_au_worker.py b/au_test_harness/vm_au_worker.py
index c24e189..47193a5 100644
--- a/au_test_harness/vm_au_worker.py
+++ b/au_test_harness/vm_au_worker.py
@@ -28,7 +28,7 @@
     self.graphics_flag = '--no-display' if options.no_graphics else ''
     if not self.board:
       cros_build_lib.Die('Need board to convert base image to vm.')
-    self.whitelist_chrome_crashes = options.whitelist_chrome_crashes
+    self.allow_chrome_crashes = options.allow_chrome_crashes
 
   def _StopVM(self):
     """Stops an existing VM."""
@@ -102,8 +102,8 @@
               ]
     if self.graphics_flag:
       command.append(self.graphics_flag)
-    if self.whitelist_chrome_crashes:
-      command.append('--test_that-args=--whitelist-chrome-crashes')
+    if self.allow_chrome_crashes:
+      command.append('--test_that-args=--allow-chrome-crashes')
     if self.ssh_private_key:
       command.append('--private-key=%s' % self.ssh_private_key)