Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Set a simple Ime for Ime stress tests

Browse Source 
To avoid test failures caused by the default Imes on some test devices overriding some system behavoirs, enable a simple Ime(com.android.apps.inputmethod.simpleime) before test and disable it after test by adding a test rule.

Also merge DisableLockRule, PressHomeBeforeTestRule, ScreenOrientationRule into one ImeStressTestRule.

Lower the height of SimpleIme to prevent keyboard from covering the input box.

Bump up the sdk version of InputMethodStressTests to "current" and fix the falilres in NotificationTest caused by the upgrade.

Test: atest com.android.inputmethod.stresstest
Bug: 271370105
Bug: 271059090
Bug: 271059311
Change-Id: I7bc097b458761424a2d7e9bbef48aa1f7406fcd3
This commit is contained in:
czq
2023-03-06 13:07:52 +08:00
parent d66f6e75e5
commit 19c5ddb5a7
11 changed files with 176 additions and 182 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
services/tests/InputMethodSystemServerTests/test-apps/SimpleTestIme/res/values/dimens.xml
View File

@@ -16,9 +16,9 @@
-->
<resources>
<dimen name="text_size_normal">24dp</dimen>
<dimen name="text_size_normal">20dp</dimen>
<dimen name="text_size_symbol">14dp</dimen>
<dimen name="keyboard_header_height">40dp</dimen>
<dimen name="keyboard_row_height">50dp</dimen>
<dimen name="keyboard_header_height">30dp</dimen>
<dimen name="keyboard_row_height">40dp</dimen>
</resources>

5
tests/InputMethodStressTest/Android.bp
View File

@@ -32,5 +32,8 @@ android_test {
"general-tests",
"vts",
],
sdk_version: "31",
data: [
":SimpleTestIme",
],
sdk_version: "current",
}

2
tests/InputMethodStressTest/AndroidManifest.xml
View File

@@ -17,7 +17,7 @@
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
package="com.android.inputmethod.stresstest">
<uses-permission android:name="android.permission.POST_NOTIFICATIONS"/>
<application>
<activity android:name=".ImeStressTestUtil$TestActivity"
android:configChanges="orientation|screenSize"/>

1
tests/InputMethodStressTest/AndroidTest.xml
View File

@@ -25,6 +25,7 @@
<target_preparer class="com.android.tradefed.targetprep.suite.SuiteApkInstaller">
<option name="cleanup-apks" value="true" />
<option name="test-file-name" value="SimpleTestIme.apk" />
<option name="test-file-name" value="InputMethodStressTest.apk" />
</target_preparer>

12
tests/InputMethodStressTest/src/com/android/inputmethod/stresstest/AutoShowTest.java
View File

@@ -61,14 +61,10 @@ import java.util.List;
@RunWith(Parameterized.class)
public final class AutoShowTest {
@Rule(order = 0) public DisableLockScreenRule mDisableLockScreenRule =
new DisableLockScreenRule();
@Rule(order = 1) public UnlockScreenRule mUnlockScreenRule = new UnlockScreenRule();
@Rule(order = 2) public ScreenOrientationRule mScreenOrientationRule =
new ScreenOrientationRule(true /* isPortrait */);
@Rule(order = 3) public PressHomeBeforeTestRule mPressHomeBeforeTestRule =
new PressHomeBeforeTestRule();
@Rule(order = 4) public ScreenCaptureRule mScreenCaptureRule =
@Rule(order = 0) public UnlockScreenRule mUnlockScreenRule = new UnlockScreenRule();
@Rule(order = 1) public ImeStressTestRule mImeStressTestRule =
new ImeStressTestRule(true /* useSimpleTestIme */);
@Rule(order = 2) public ScreenCaptureRule mScreenCaptureRule =
new ScreenCaptureRule("/sdcard/InputMethodStressTest");
@Parameterized.Parameters(
name = "windowFocusFlags={0}, softInputVisibility={1}, softInputAdjustment={2}")

53
tests/InputMethodStressTest/src/com/android/inputmethod/stresstest/DisableLockScreenRule.java
View File

@@ -1,53 +0,0 @@
/*
* Copyright (C) 2022 The Android Open Source Project
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package com.android.inputmethod.stresstest;
import android.support.test.uiautomator.UiDevice;
import androidx.test.platform.app.InstrumentationRegistry;
import org.junit.rules.TestWatcher;
import org.junit.runner.Description;
import java.io.IOException;
/** Disable lock screen during the test. */
public class DisableLockScreenRule extends TestWatcher {
private static final String LOCK_SCREEN_OFF_COMMAND = "locksettings set-disabled true";
private static final String LOCK_SCREEN_ON_COMMAND = "locksettings set-disabled false";
private final UiDevice mUiDevice =
UiDevice.getInstance(InstrumentationRegistry.getInstrumentation());
@Override
protected void starting(Description description) {
try {
mUiDevice.executeShellCommand(LOCK_SCREEN_OFF_COMMAND);
} catch (IOException e) {
throw new RuntimeException("Could not disable lock screen.", e);
}
}
@Override
protected void finished(Description description) {
try {
mUiDevice.executeShellCommand(LOCK_SCREEN_ON_COMMAND);
} catch (IOException e) {
throw new RuntimeException("Could not enable lock screen.", e);
}
}
}

14
tests/InputMethodStressTest/src/com/android/inputmethod/stresstest/ImeOpenCloseStressTest.java
View File

@@ -68,14 +68,10 @@ public final class ImeOpenCloseStressTest {
private static final String TAG = "ImeOpenCloseStressTest";
private static final int NUM_TEST_ITERATIONS = 10;
@Rule(order = 0) public DisableLockScreenRule mDisableLockScreenRule =
new DisableLockScreenRule();
@Rule(order = 1) public UnlockScreenRule mUnlockScreenRule = new UnlockScreenRule();
@Rule(order = 2) public ScreenOrientationRule mScreenOrientationRule =
new ScreenOrientationRule(true /* isPortrait */);
@Rule(order = 3) public PressHomeBeforeTestRule mPressHomeBeforeTestRule =
new PressHomeBeforeTestRule();
@Rule(order = 4) public ScreenCaptureRule mScreenCaptureRule =
@Rule(order = 0) public UnlockScreenRule mUnlockScreenRule = new UnlockScreenRule();
@Rule(order = 1) public ImeStressTestRule mImeStressTestRule =
new ImeStressTestRule(true /* useSimpleTestIme */);
@Rule(order = 2) public ScreenCaptureRule mScreenCaptureRule =
new ScreenCaptureRule("/sdcard/InputMethodStressTest");
private final Instrumentation mInstrumentation;
@@ -499,8 +495,6 @@ public final class ImeOpenCloseStressTest {
@Test
public void testRotateScreenWithKeyboardOn() throws Exception {
// TODO(b/256739702): Keyboard disappears after rotating screen to landscape mode if
// android:configChanges="orientation|screenSize" is not set
Intent intent =
createIntent(
mWindowFocusFlags,

153
tests/InputMethodStressTest/src/com/android/inputmethod/stresstest/ImeStressTestRule.java Normal file
View File

@@ -0,0 +1,153 @@
/*
* Copyright (C) 2023 The Android Open Source Project
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package com.android.inputmethod.stresstest;
import android.app.Instrumentation;
import android.os.RemoteException;
import android.support.test.uiautomator.UiDevice;
import androidx.test.platform.app.InstrumentationRegistry;
import org.checkerframework.checker.nullness.qual.NonNull;
import org.junit.rules.TestWatcher;
import org.junit.runner.Description;
import java.io.IOException;
/**
* Do setup and cleanup for Ime stress tests, including disabling lock and auto-rotate screen,
* pressing home and enabling a simple test Ime during the tests.
*/
public class ImeStressTestRule extends TestWatcher {
private static final String LOCK_SCREEN_OFF_COMMAND = "locksettings set-disabled true";
private static final String LOCK_SCREEN_ON_COMMAND = "locksettings set-disabled false";
private static final String SET_PORTRAIT_MODE_COMMAND = "settings put system user_rotation 0";
private static final String SET_LANDSCAPE_MODE_COMMAND = "settings put system user_rotation 1";
private static final String SIMPLE_IME_ID =
"com.android.apps.inputmethod.simpleime/.SimpleInputMethodService";
private static final String ENABLE_IME_COMMAND = "ime enable " + SIMPLE_IME_ID;
private static final String SET_IME_COMMAND = "ime set " + SIMPLE_IME_ID;
private static final String DISABLE_IME_COMMAND = "ime disable " + SIMPLE_IME_ID;
private static final String RESET_IME_COMMAND = "ime reset";
@NonNull private final Instrumentation mInstrumentation;
@NonNull private final UiDevice mUiDevice;
// Whether the screen orientation is set to portrait.
private boolean mIsPortrait;
// Whether to use a simple test Ime or system default Ime for test.
private final boolean mUseSimpleTestIme;
public ImeStressTestRule(boolean useSimpleTestIme) {
mInstrumentation = InstrumentationRegistry.getInstrumentation();
mUiDevice = UiDevice.getInstance(mInstrumentation);
// Default is portrait mode
mIsPortrait = true;
mUseSimpleTestIme = useSimpleTestIme;
}
public void setIsPortrait(boolean isPortrait) {
mIsPortrait = isPortrait;
}
@Override
protected void starting(Description description) {
disableLockScreen();
setOrientation();
mUiDevice.pressHome();
if (mUseSimpleTestIme) {
enableSimpleIme();
} else {
resetImeToDefault();
}
mInstrumentation.waitForIdleSync();
}
@Override
protected void finished(Description description) {
if (mUseSimpleTestIme) {
disableSimpleIme();
}
unfreezeRotation();
restoreLockScreen();
}
private void disableLockScreen() {
try {
executeShellCommand(LOCK_SCREEN_OFF_COMMAND);
} catch (IOException e) {
throw new RuntimeException("Could not disable lock screen.", e);
}
}
private void restoreLockScreen() {
try {
executeShellCommand(LOCK_SCREEN_ON_COMMAND);
} catch (IOException e) {
throw new RuntimeException("Could not enable lock screen.", e);
}
}
private void setOrientation() {
try {
mUiDevice.freezeRotation();
executeShellCommand(
mIsPortrait ? SET_PORTRAIT_MODE_COMMAND : SET_LANDSCAPE_MODE_COMMAND);
} catch (IOException e) {
throw new RuntimeException("Could not set screen orientation.", e);
} catch (RemoteException e) {
throw new RuntimeException("Could not freeze rotation.", e);
}
}
private void unfreezeRotation() {
try {
mUiDevice.unfreezeRotation();
} catch (RemoteException e) {
throw new RuntimeException("Could not unfreeze screen rotation.", e);
}
}
private void enableSimpleIme() {
try {
executeShellCommand(ENABLE_IME_COMMAND);
executeShellCommand(SET_IME_COMMAND);
} catch (IOException e) {
throw new RuntimeException("Could not enable SimpleTestIme.", e);
}
}
private void disableSimpleIme() {
try {
executeShellCommand(DISABLE_IME_COMMAND);
} catch (IOException e) {
throw new RuntimeException("Could not disable SimpleTestIme.", e);
}
}
private void resetImeToDefault() {
try {
executeShellCommand(RESET_IME_COMMAND);
} catch (IOException e) {
throw new RuntimeException("Could not reset Ime to default.", e);
}
}
private @NonNull String executeShellCommand(@NonNull String cmd) throws IOException {
return mUiDevice.executeShellCommand(cmd);
}
}

12
tests/InputMethodStressTest/src/com/android/inputmethod/stresstest/NotificationTest.java
View File

@@ -77,11 +77,10 @@ public final class NotificationTest {
private static final BySelector REPLY_SEND_BUTTON_SELECTOR =
By.res("com.android.systemui", "remote_input_send").enabled(true);
@Rule
public UnlockScreenRule mUnlockScreenRule = new UnlockScreenRule();
@Rule
public ScreenCaptureRule mScreenCaptureRule =
@Rule(order = 0) public UnlockScreenRule mUnlockScreenRule = new UnlockScreenRule();
@Rule(order = 1) public ImeStressTestRule mImeStressTestRule =
new ImeStressTestRule(true /* useSimpleTestIme */);
@Rule(order = 2) public ScreenCaptureRule mScreenCaptureRule =
new ScreenCaptureRule("/sdcard/InputMethodStressTest");
private Context mContext;
@@ -141,7 +140,8 @@ public final class NotificationTest {
// Post inline reply notification.
PendingIntent pendingIntent = PendingIntent.getBroadcast(
mContext, REPLY_REQUEST_CODE, new Intent().setAction(ACTION_REPLY),
mContext, REPLY_REQUEST_CODE,
new Intent().setAction(ACTION_REPLY).setClass(mContext, NotificationTest.class),
PendingIntent.FLAG_UPDATE_CURRENT | PendingIntent.FLAG_MUTABLE);
RemoteInput remoteInput = new RemoteInput.Builder(REPLY_INPUT_KEY)
.setLabel(REPLY_INPUT_LABEL)

34
tests/InputMethodStressTest/src/com/android/inputmethod/stresstest/PressHomeBeforeTestRule.java
View File

@@ -1,34 +0,0 @@
/*
* Copyright (C) 2023 The Android Open Source Project
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package com.android.inputmethod.stresstest;
import android.support.test.uiautomator.UiDevice;
import androidx.test.platform.app.InstrumentationRegistry;
import org.junit.rules.TestWatcher;
import org.junit.runner.Description;
/** This rule will press home before a test case. */
public class PressHomeBeforeTestRule extends TestWatcher {
private final UiDevice mUiDevice =
UiDevice.getInstance(InstrumentationRegistry.getInstrumentation());
@Override
protected void starting(Description description) {
mUiDevice.pressHome();
}
}

66
tests/InputMethodStressTest/src/com/android/inputmethod/stresstest/ScreenOrientationRule.java
View File

@@ -1,66 +0,0 @@
/*
* Copyright (C) 2022 The Android Open Source Project
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package com.android.inputmethod.stresstest;
import android.os.RemoteException;
import android.support.test.uiautomator.UiDevice;
import androidx.test.platform.app.InstrumentationRegistry;
import org.junit.rules.TestWatcher;
import org.junit.runner.Description;
import java.io.IOException;
/**
* Disable auto-rotate during the test and set the screen orientation to portrait or landscape
* before the test starts.
*/
public class ScreenOrientationRule extends TestWatcher {
private static final String SET_PORTRAIT_MODE_CMD = "settings put system user_rotation 0";
private static final String SET_LANDSCAPE_MODE_CMD = "settings put system user_rotation 1";
private final boolean mIsPortrait;
private final UiDevice mUiDevice =
UiDevice.getInstance(InstrumentationRegistry.getInstrumentation());
ScreenOrientationRule(boolean isPortrait) {
mIsPortrait = isPortrait;
}
@Override
protected void starting(Description description) {
try {
mUiDevice.freezeRotation();
mUiDevice.executeShellCommand(mIsPortrait ? SET_PORTRAIT_MODE_CMD :
SET_LANDSCAPE_MODE_CMD);
} catch (IOException e) {
throw new RuntimeException("Could not set screen orientation.", e);
} catch (RemoteException e) {
throw new RuntimeException("Could not freeze rotation.", e);
}
}
@Override
protected void finished(Description description) {
try {
mUiDevice.unfreezeRotation();
} catch (RemoteException e) {
throw new RuntimeException("Could not unfreeze screen rotation.", e);
}
}
}