diff options
author | Rakib Hasan <rmhasan@google.com> | 2023-03-22 17:13:04 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2023-03-22 17:13:04 +0000 |
commit | 1cf79c29a2838f6aaae7aa081436884464bfab53 (patch) | |
tree | 11a9174a8e07815e3db4bbf5c9f71ad49e85d324 | |
parent | 08ea0137c6dddb7a3ea51d0f382bcda12030d536 (diff) | |
parent | c069561a6ed27e1d279e56eef05b623f9aa9dae8 (diff) | |
download | csuite-1cf79c29a2838f6aaae7aa081436884464bfab53.tar.gz |
Merge "Webview: Use only one RunUtil instance in initialization"
-rw-r--r-- | test_scripts/src/main/java/com/android/webview/lib/WebviewInstallerToolPreparer.java | 51 |
1 files changed, 23 insertions, 28 deletions
diff --git a/test_scripts/src/main/java/com/android/webview/lib/WebviewInstallerToolPreparer.java b/test_scripts/src/main/java/com/android/webview/lib/WebviewInstallerToolPreparer.java index 349cd78..52e5c94 100644 --- a/test_scripts/src/main/java/com/android/webview/lib/WebviewInstallerToolPreparer.java +++ b/test_scripts/src/main/java/com/android/webview/lib/WebviewInstallerToolPreparer.java @@ -134,16 +134,15 @@ public class WebviewInstallerToolPreparer implements ITargetPreparer { Assert.assertNotEquals( "Argument --gcloud-cli-zip must be used.", mGcloudCliZipArchive, null); try { + RunUtil runUtil = mRunUtilProvider.get(); mGcloudCliDir = Files.createTempDirectory(null).toFile(); CommandResult unzipRes = - mRunUtilProvider - .get() - .runTimedCmd( - COMMAND_TIMEOUT_MILLIS, - "unzip", - mGcloudCliZipArchive.getAbsolutePath(), - "-d", - mGcloudCliDir.getAbsolutePath()); + runUtil.runTimedCmd( + COMMAND_TIMEOUT_MILLIS, + "unzip", + mGcloudCliZipArchive.getAbsolutePath(), + "-d", + mGcloudCliDir.getAbsolutePath()); Assert.assertEquals( "Unable to unzip the gcloud cli zip archive", @@ -156,7 +155,7 @@ public class WebviewInstallerToolPreparer implements ITargetPreparer { // gcloud and gsutil executables tracked by this class by setting the home // directory for processes that run those executables to a temporary directory // also tracked by this class. - mRunUtilProvider.get().setEnvVariable("HOME", mGcloudCliDir.getAbsolutePath()); + runUtil.setEnvVariable("HOME", mGcloudCliDir.getAbsolutePath()); File gcloudBin = mGcloudCliDir .toPath() @@ -167,31 +166,27 @@ public class WebviewInstallerToolPreparer implements ITargetPreparer { "printf \"1\\n1\" | %s init --console-only", gcloudBin.getAbsolutePath()); CommandResult gcloudInitRes = - mRunUtilProvider - .get() - .runTimedCmd( - COMMAND_TIMEOUT_MILLIS, - System.out, - System.out, - "sh", - "-c", - gcloudInitScript); + runUtil.runTimedCmd( + COMMAND_TIMEOUT_MILLIS, + System.out, + System.out, + "sh", + "-c", + gcloudInitScript); Assert.assertEquals( "gcloud cli initialization failed", gcloudInitRes.getStatus(), CommandStatus.SUCCESS); CommandResult chmodRes = - mRunUtilProvider - .get() - .runTimedCmd( - COMMAND_TIMEOUT_MILLIS, - System.out, - System.out, - "chmod", - "755", - "-v", - mWebviewInstallerTool.getAbsolutePath()); + runUtil.runTimedCmd( + COMMAND_TIMEOUT_MILLIS, + System.out, + System.out, + "chmod", + "755", + "-v", + mWebviewInstallerTool.getAbsolutePath()); Assert.assertEquals( "The 'chmod 755 -v <WebView installer tool>' command failed", |