Commit Graph

7 Commits

Author SHA1 Message Date
LaMont Jones
74f7786401 Fix container for aconfig flags.
This app is in system_ext, not system.

Bug: b/325307746
Test: manual
Flag: NA
Change-Id: Ibb1c7a4b338b6bf900a626e7ec66a28e0f90169d
2024-02-26 14:54:31 -08:00
Oriol Prieto Gasco
59234c574e Set the container field of aconfig flags
Please see go/aconfig-flag-containers for more context

Test: m
Bug: 312769710
Change-Id: I95efda5d266ad45202501d82d256d9e9f8e7826a
2024-02-07 20:00:34 +00:00
Sunny Goyal
d609f6ea39 Revert "Set aconfig container to "system""
This reverts commit 2916945dbf.

Reason for revert: breaks android-studio development flow

Change-Id: I219eb87c170b2b0dd1bb98894afa680bffb360b5
2024-01-19 21:35:29 +00:00
Oriol Prieto Gasco
2916945dbf Set aconfig container to "system"
Bug: 312769710
Flag: NA
Test: TH
Change-Id: I7fbb8e31629b7aa90fbdf2b029932c111cce8d42
2024-01-10 19:52:57 +00:00
Anushree Ganjam
2a14b978c6 Remove the test aconfig lib.
zhidou@ from ACE team is working on AConfigFakelib which will be useful
to set fake flag values in both unit tests and UI automator tests.
Meanwhile, we will mock the flag values for unit tests and UI automator
tests will have all flags values defaulted to false.

testAConfigLib with "test:true" doesn't generate fake flags values and
forces us to set the flag values using "SetFlagsRule". "SetFlagsRule"
doesn't work with Out Of Proc tests and hence doesn't serve the purpose
for us. AConfigFakelib and FakeFlagTargetPreparer should use for Out Of
Proc tests.

testAConfigLib with "test:true" is suitable for java host tests where
there is no device involved. Launcher tests are instrumentation tests and tests run on actual
devices. So we can use the production version of the AConfig lib and device
config flag values will be fetched for UI Automator tests.

Also migrating "enable_twoline_allapps" to trunk stable. Unit tests is
using mockito to mock the flag value as false. The owner of
"enable_twoline_allapps" can mock the value to true in unit tests as
next step.

Bug: 294913042
Test: Local studio build and soong build
Flag: enable_twoline_allapps

Change-Id: Iea42bf18197832a9ecbddc1c3aaa8c9e36169131
2023-08-30 15:20:51 -07:00
Anushree Ganjam
813e96e1ab Include all *.aconfig files under aconfig directory
Forward fix for  https://googleplex-android-review.git.corp.google.com/c/platform/vendor/unbundled_google/packages/NexusLauncher/+/24579326/4/build.gradle

Bug: 294913042
Test: Local studio build
Flag: NA
Change-Id: Ib16469096c92eff4b50167e58fe5132e5467fdde
2023-08-29 15:47:10 -07:00
Anushree Ganjam
8a6b74e51b Move aconfig files to aconfig folder
- Set packageName as com.android.launcher3
- Follow folder structure as per go/sysui-aconfig-structure

Bug: 294913042

Test: adb shell device_config put launcher com.android.launcher3.enable_all_apps_rv_preinflation true

Flag: enable_all_apps_rv_preinflation

Change-Id: Ib45a24d9100b657e7776c6120bdf9625533e9ed7
2023-08-28 18:27:11 -07:00