Commit Graph

20 Commits

Author SHA1 Message Date
Steven Moreland
a9de5ee4ce Increase 16 KB test timeout, to see how it affects flakes.
Bug: 405230142
Test: N/A
Change-Id: I6370880160eb6024afa505ed1e42542184940d91
2025-03-21 17:54:38 -07:00
Pawan Wagh
cc282f8470 Use WaitForDeviceAvailable
WaitForDeviceOnline doesn't gurantee that device
can execute commands

Test: atest -c Enable16KbTest
Bug: 401597066
Change-Id: Icf55035ebe4b4a028892dc22665dafc8a5b492b4
2025-03-14 06:46:47 +00:00
Pawan Wagh
63a8931a41 Merge "16KB -> 16 KB" into main 2025-02-20 19:28:35 -08:00
Steven Moreland
911a99932c 16KB -> 16 KB
To be consistent with our docs and communication elsewhere.

Bugs: me
Change-Id: I0233412e452a27a14eb14336b0229114d022183c
Test: N/A
2025-02-19 15:11:18 -08:00
Pawan Wagh
24a65ba02b Connect and enable root before running each testcase
- Enable16KbTest is flaky for aosp targets. Make sure that device
is connected and with root each time before and after running the
test.
- Add OWNERS file Enable16KbTest

Test: atest -c Enable16KbTest
Bug: 396984821
Change-Id: Ifffb73496b0947dda13a4022733677bd645b7e16
2025-02-19 20:59:06 +00:00
Pawan Wagh
4a55472cf3 Check if device is online before starting 16kb test
Test: atest -c Enable16KbTest
Bug: 390535191
Change-Id: I32cc404f8e84fba0e90ecb5b801d98908242292d
2025-02-11 22:39:38 +00:00
Vilas Bhat
d30433c92e 16KB: Fix failing tests for AOSP targets
UiAutomator is case sensitive. On AOSP builds, the "Erase all data"
string is displayed in all caps in the dialog box on trying to erase all
data and switch to ext4 mode. This is a workaround fix for the issue.

Bug: 390535191
Flag: EXEMPT bugfix
Test: Enable16kbTest on AOSP Pixel 8 targets
Change-Id: I119fe076eb3eb037a23f07af77b9ad4935cb4064
2025-01-22 00:12:56 +00:00
Xin Li
cfa19e11a6 Merge 24Q4 (ab/12406339) into aosp-main-future
Bug: 370570306
Merged-In: Ie90e7495dd4a134538bae6e3e08eea0d02134b14
Change-Id: I20517e9ee410e95f2cbeb1247c0c0288ed9f006f
2024-11-11 21:38:40 -08:00
Cole Faust
66c5be9fdf Remove dependencies on the 1-variant fallback
When adding a dependencies, if the variants don't match, but the
dependency only has 1 variant anyways, soong will always use that
variant. This makes it hard to add new variants to soong, because the
1-variant fallback stops being used and you start getting missing
variant errors. Make changes to bp files such that all dependencies
correctly specify the variant to use.

Bug: 372091092
Flag: EXEMPT refactor
Test: m nothing
Change-Id: I8487eff4de7349c922901160f55717e7d3296650
2024-10-22 16:30:59 -07:00
Jihoon Kang
4ae6ead8d6 Make java_sdk_library dependencies explicit
modules should specify the submodule of java_sdk_library that the module
actually depends on

Test: CI
Bug: 358613520
Merged-In: I78dec317d4e74275287e3a5b7f5ffe2b10db05ab
Change-Id: I78dec317d4e74275287e3a5b7f5ffe2b10db05ab
2024-09-10 22:43:05 +00:00
Jihoon Kang
9b946d241d Make java_sdk_library dependencies explicit
modules should specify the submodule of java_sdk_library that the module
actually depends on

Test: CI
Flag: EXEMPT build dependency cleanup
Bug: 358613520
Change-Id: I78dec317d4e74275287e3a5b7f5ffe2b10db05ab
2024-09-10 22:33:56 +00:00
Pawan Wagh
e1a69960f9 Merge "Fix flaky test for persistent notifications" into main am: b6fc7a6f58 am: fdd68f473b
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Settings/+/3156445

Change-Id: I0635a73774b0db08c401ef2523b2da13b0b359f9
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-07-03 15:47:26 +00:00
Pawan Wagh
999b8a9f33 Fix flaky test for persistent notifications
Persistent notifications test sometimes fails due
to setup activity doesn't exit due to failed intent.
Adding setup exit command when on device tests are starting and
adding timeout for transition.

Test: m Settings && adb install -r $ANDROID_PRODUCT_OUT/system_ext/priv-app/Settings/Settings.apk
Flag: EXEMPT bugfix
Bug: 349820489
Change-Id: Iaf1c1f481612367bbce919b27e89f88a24abb582
2024-07-02 23:57:57 +00:00
Julien Desprez
e4087bf29f Enable 16kb tests on general-tests suite
This doesn't need to be built on device side.

Flag: TEST_ONLY
Test: https://android-build.corp.google.com/builds/abtd/run/L50300030004790194
Bug: 342232954

Change-Id: I86493e8d197ca6acc24743e44b63a764bf4d36df
2024-06-25 02:56:04 +00:00
Pawan Wagh
df6affdc3d Merge "Move Enable16KbTest to device-tests" into main 2024-06-12 01:10:40 +00:00
Pawan Wagh
3aa1405acb Move Enable16KbTest to device-tests
general-tests are not configured for the devices
which should be tested. Moving test to device-test suite.

Test: m Enable16KbTest && atest -c Enable16KbTest
Bug: 345763025
Change-Id: I01abf4ec91c4f7b588de12ac9cb71b805849e4ac
2024-06-10 21:19:27 +00:00
Treehugger Robot
c3c33df4fc Merge "Add Enable16KbTest to TEST_MAPPING" into main 2024-06-06 16:28:25 +00:00
Pawan Wagh
9021526247 Fix test config
test app was failing to install when running in integration due to
shared uid. Removing the sharedUid field from test.

Test: m Enable16KbTest && atest -c Enable16KbTest
Bug: 341995374

Change-Id: I37f481c552f8f0779bcd8c69cf4e40598590f5db
2024-06-05 23:02:50 +00:00
Pawan Wagh
0dbc009db1 Add Enable16KbTest to TEST_MAPPING
Test: m Enable16KbTest && atest -c Enable16KbTest
Bug: 341995374
Change-Id: I92bf4a92dcb9fed4d4ae3cb35f6032b095a640d3
2024-06-04 23:12:39 +00:00
Pawan Wagh
10ff699b99 Integration tests for 16kb dev option
Adding tests to verify end to end functionality for dev options.
This test verifies
- /data reformatting to ext4
- switching to 16kb kernel
- switching back to 4kb kernel
- Persistent notifications in page-agnostic mode
- Disabling developer option in page-agnostic mode

Test: m Enable16KbTest && atest -c Enable16KbTest
Bug: 341995374
Change-Id: Ieaa149831b5205a15b259982f0349dc94d4fae92
2024-06-04 23:11:37 +00:00