diff options
author | Android Build Coastguard Worker <android-build-coastguard-worker@google.com> | 2022-04-22 01:11:10 +0000 |
---|---|---|
committer | Android Build Coastguard Worker <android-build-coastguard-worker@google.com> | 2022-04-22 01:11:10 +0000 |
commit | dc2a7f3d014ef2199f9de6f33b05afa7fde595d7 (patch) | |
tree | 34a81af095a69c5e94e06bc4bdedd0b0a9238b82 | |
parent | 9b509def12c039f97927a25e192fa6e6197e8388 (diff) | |
parent | f5008b11d543b2dbce00e0fa3a13f59775df05a2 (diff) | |
download | csuite-dc2a7f3d014ef2199f9de6f33b05afa7fde595d7.tar.gz |
Snap for 8485504 from f5008b11d543b2dbce00e0fa3a13f59775df05a2 to tm-releaseandroid-vts-13.0_r8android-vts-13.0_r7android-vts-13.0_r6android-vts-13.0_r5android-vts-13.0_r4android-vts-13.0_r3android-vts-13.0_r2android-vts-13.0_r1android-security-13.0.0_r9android-security-13.0.0_r8android-security-13.0.0_r7android-security-13.0.0_r6android-security-13.0.0_r5android-security-13.0.0_r4android-security-13.0.0_r3android-security-13.0.0_r2android-security-13.0.0_r19android-security-13.0.0_r18android-security-13.0.0_r17android-security-13.0.0_r16android-security-13.0.0_r15android-security-13.0.0_r14android-security-13.0.0_r13android-security-13.0.0_r12android-security-13.0.0_r11android-security-13.0.0_r10android-security-13.0.0_r1android-platform-13.0.0_r2android-platform-13.0.0_r1android-cts-13.0_r8android-cts-13.0_r7android-cts-13.0_r6android-cts-13.0_r5android-cts-13.0_r4android-cts-13.0_r3android-cts-13.0_r2android-cts-13.0_r1android-13.0.0_r8android-13.0.0_r7android-13.0.0_r6android-13.0.0_r5android-13.0.0_r4android-13.0.0_r31android-13.0.0_r3android-13.0.0_r2android-13.0.0_r12android-13.0.0_r1android13-tests-releaseandroid13-security-releaseandroid13-s3-releaseandroid13-s2-releaseandroid13-s1-releaseandroid13-release
Change-Id: I5020f5d19269c4641be180591b48b7278019eed2
-rw-r--r-- | test_scripts/src/main/java/com/android/webview/tests/WebviewAppLaunchTest.java | 32 |
1 files changed, 19 insertions, 13 deletions
diff --git a/test_scripts/src/main/java/com/android/webview/tests/WebviewAppLaunchTest.java b/test_scripts/src/main/java/com/android/webview/tests/WebviewAppLaunchTest.java index 7cd1718..d08d2e3 100644 --- a/test_scripts/src/main/java/com/android/webview/tests/WebviewAppLaunchTest.java +++ b/test_scripts/src/main/java/com/android/webview/tests/WebviewAppLaunchTest.java @@ -110,13 +110,14 @@ public class WebviewAppLaunchTest extends BaseHostJUnit4Test { public void testAppLaunch() throws DeviceNotAvailableException, ApkInstallerException, IOException { AssertionError lastError = null; - // Try the latest webview version - installWebview(mOrderedWebviewApks.get(0)); + WebviewPackage lastWebviewInstalled = installWebview(mOrderedWebviewApks.get(0)); try { assertAppLaunchNoCrash(); } catch (AssertionError e) { lastError = e; + } finally { + uninstallWebview(); } // If the app doesn't crash, complete the test. @@ -126,7 +127,6 @@ public class WebviewAppLaunchTest extends BaseHostJUnit4Test { // If the app crashes, try the app with the original webview version that comes with the // device. - uninstallWebview(); try { assertAppLaunchNoCrash(); } catch (AssertionError newError) { @@ -138,13 +138,14 @@ public class WebviewAppLaunchTest extends BaseHostJUnit4Test { } for (int idx = 1; idx < mOrderedWebviewApks.size(); idx++) { - uninstallWebview(); - installWebview(mOrderedWebviewApks.get(idx)); + lastWebviewInstalled = installWebview(mOrderedWebviewApks.get(idx)); try { assertAppLaunchNoCrash(); } catch (AssertionError e) { lastError = e; continue; + } finally { + uninstallWebview(); } break; } @@ -152,7 +153,7 @@ public class WebviewAppLaunchTest extends BaseHostJUnit4Test { throw new AssertionError( String.format( "Package %s crashed since webview version %s", - mPackageName, getCurrentWebviewPackage().getVersion()), + mPackageName, lastWebviewInstalled.getVersion()), lastError); } @@ -166,8 +167,6 @@ public class WebviewAppLaunchTest extends BaseHostJUnit4Test { deviceUtils.unfreezeRotation(); mApkInstaller.uninstallAllInstalledPackages(); - - uninstallWebview(); printWebviewVersion(); } @@ -187,13 +186,18 @@ public class WebviewAppLaunchTest extends BaseHostJUnit4Test { }); } + private void printWebviewVersion(WebviewPackage currentWebview) + throws DeviceNotAvailableException { + CLog.i("Current webview implementation: %s", currentWebview.getPackageName()); + CLog.i("Current webview version: %s", currentWebview.getVersion()); + } + private void printWebviewVersion() throws DeviceNotAvailableException { - WebviewPackage impl = getCurrentWebviewPackage(); - CLog.i("Current webview implementation: %s", impl.getPackageName()); - CLog.i("Current webview version: %s", impl.getVersion()); + WebviewPackage currentWebview = getCurrentWebviewPackage(); + printWebviewVersion(currentWebview); } - private void installWebview(File apk) + private WebviewPackage installWebview(File apk) throws ApkInstallerException, IOException, DeviceNotAvailableException { ApkInstaller.getInstance(getDevice()).install(apk.toPath()); CommandResult res = @@ -202,7 +206,9 @@ public class WebviewAppLaunchTest extends BaseHostJUnit4Test { "cmd webviewupdate set-webview-implementation com.android.webview"); Assert.assertEquals( "Failed to set webview update: " + res, res.getStatus(), CommandStatus.SUCCESS); - printWebviewVersion(); + WebviewPackage currentWebview = getCurrentWebviewPackage(); + printWebviewVersion(currentWebview); + return currentWebview; } private void uninstallWebview() throws DeviceNotAvailableException { |