Age | Commit message (Collapse) | Author |
|
1) Add the method overriding annotation.
2) Add more triaging tips for AP disconnected, wrong AP frequency and P2P/STA frequency mismatch cases.
3) Reduce 2G throughput to 1MB/s until it is fixed.
Bug: 338328902
Test: compile and manual test
Change-Id: Ia1e3d0254041e47dd918ed740006da63b5480382
|
|
1) Add the support of BLE scan throttling during 2G transfer (enabled by default)
2) Remove the unnecessary flag overriding and rely on the production config instead.
3) Add BT coex test to betocq.
4) Change the success rate target to 98%.
5) Add the check of AP connection. If AP is disconnected or connect to a wrong frequency on the target side, mark the test as failed.
6) consolidate AP connection and speed check codes to one function.
7) Add P2P frequency check for WFD/HS SCC test cases.
8) Reduce 2G speed check from 3 to 2 MB/s until it is improved in NC.
9) Remove AP frequency check for the test cases with empty wifi_ssid.
10) Fix typo in DFS test cases and reduce BT transfer size by 50%.
11) Skip p2p frequency check if wifi speed check is disabled or it is a DBS test.
12) Copy the latest snippet apk which moves transfer time count after
the file is created.
Bug:337326375
Test: manual test
Test: atest
Change-Id: I3708e7a19cd78efbfd63c4d46b4fdca88410934a
|
|
Bug: 338102234
Test: local
Change-Id: Ib0c358dd111ae40e71311633184dfb7c74898c53
|
|
NOTE: This also updates the CustomFeatureFlags and FakeFeatureFlagsImpl inside the test directory to (almost) exactly match the current code generation. The only difference is that the "isFlagReadOnlyOptimized" returns true for the read only flags even without optimization enabled.
Fixes: 337449122
Flag: test_only
Test: atest FlagJUnitHostTests
Change-Id: Ib35db6a1c51fd5702ad6536a65c1e40e7b45d393
|
|
|
|
This method has never been used, and unnecessarily exposes the implementation details of SetFlagsRule.
Flag: TEST_ONLY
NOTE FOR REVIEWERS - original patch and result patch are not identical.
PLEASE REVIEW CAREFULLY.
Diffs between the patches:
/**
> - * Returns a FeatureFlags used by SetFlagsRule of given FeatureFlags
> - *
> - * @param featureFlagsClass The class of FeatureFlags. The interface of FakeFeatureFlagsImpl
> - * @return A FakeFeatureFlagsImpl in type of FeatureFlags
> - */
> - public <T> T getFakeFeatureFlags(Class<T> featureFlagsClass) {
> - if (!featureFlagsClass.isInterface()
> - || !featureFlagsClass.getSimpleName().equals(FEATURE_FLAGS_CLASS_NAME)) {
> - throw new IllegalArgumentException(
> - String.format(
> - "%s is not a FeatureFlags. " + "Please pass in FeatureFlags interface",
> - featureFlagsClass));
> - }
> -
> - String packageName = featureFlagsClass.getPackageName();
> - String flagsClassName = String.format("%s.%s", packageName, FLAGS_CLASS_NAME);
> - Class<?> flagsClass = null;
> -
> - try {
> - flagsClass = Class.forName(flagsClassName);
> - } catch (ClassNotFoundException e) {
> - throw new UnsupportedOperationException(
> - String.format("Failed to load class %s.", flagsClassName));
> - }
> -
> - Object fakeFlagsImplInstance = getOrCreateFakeFlagsImp(flagsClass);
> -
> - return featureFlagsClass.cast(fakeFlagsImplInstance);
> - }
> -
> --- libraries/flag-helpers/junit/test/src/android/platform/test/flag/junit/FakeFeatureFlagsImpl.java
> +++ libraries/flag-helpers/junit/test/src/android/platform/test/flag/junit/FakeFeatureFlagsImpl.java
> - public HashSet<String> readOnlyFlagSet = new HashSet<>();
> + private HashSet<String> mReadOnlyFlagSet = new HashSet<>();
> + this.mReadOnlyFlagSet.add(Flags.FLAG_RO_ENABLED);
> + this.mReadOnlyFlagSet.add(Flags.FLAG_RO_DISABLED);
> + @Override
> + public boolean roEnabled() {
> + return this.mFlagMap.get(Flags.FLAG_RO_ENABLED);
> + }
> +
> + @Override
> + public boolean roDisabled() {
> + return this.mFlagMap.get(Flags.FLAG_RO_DISABLED);
> + }
> +
> - return readOnlyFlagSet.contains(flagName);
> + return mReadOnlyFlagSet.contains(flagName);
> --- libraries/flag-helpers/junit/test/src/android/platform/test/flag/junit/FeatureFlags.java
> +++ libraries/flag-helpers/junit/test/src/android/platform/test/flag/junit/FeatureFlags.java
> +
> + boolean roEnabled();
> +
> + boolean roDisabled();
> --- libraries/flag-helpers/junit/test/src/android/platform/test/flag/junit/Flags.java
> +++ libraries/flag-helpers/junit/test/src/android/platform/test/flag/junit/Flags.java
> -
> - /** Returns the flag value. */
> - public static boolean flagName1() {
> - return true;
> - }
> -
> - /** Returns a non-boolean flag value. */
> - public static int flagName2() {
> - return 1;
> - }
> + public static final String FLAG_RO_ENABLED = "android.platform.test.flag.junit.ro_enabled";
> + public static final String FLAG_RO_DISABLED = "android.platform.test.flag.junit.ro_disabled";
> + public static boolean roEnabled() {
> + return true;
> + }
> +
> + public static boolean roDisabled() {
> + return false;
> + }
> +
> +
> + @Override
> + public boolean roEnabled() {
> + return true;
> + }
> +
> + @Override
> + public boolean roDisabled() {
> + return false;
> + }
> --- libraries/flag-helpers/junit/test/src/android/platform/test/flag/junit/SetFlagsRuleTest.java
> +++ libraries/flag-helpers/junit/test/src/android/platform/test/flag/junit/SetFlagsRuleTest.java
> - public void getFakeFeatureFlags_afterSet() {
> - mSetFlagsRule.enableFlags(Flags.FLAG_FLAG_NAME3, Flags.FLAG_FLAG_NAME4);
> - FeatureFlags fakeFlagsImpl = mSetFlagsRule.getFakeFeatureFlags(FeatureFlags.class);
> - assertTrue(fakeFlagsImpl.flagName3());
> - assertTrue(fakeFlagsImpl.flagName4());
> -
> - mSetFlagsRule.disableFlags(Flags.FLAG_FLAG_NAME3, Flags.FLAG_FLAG_NAME4);
> - assertFalse(fakeFlagsImpl.flagName3());
> - assertFalse(fakeFlagsImpl.flagName4());
> - }
> -
> - @Test
> - public void getFakeFeatureFlags_thenSet() {
> - FeatureFlags fakeFlagsImpl = mSetFlagsRule.getFakeFeatureFlags(FeatureFlags.class);
> - if (this.mIsInitWithDefault) {
> - assertFalse(Flags.flagName3());
> - assertFalse(fakeFlagsImpl.flagName3());
> - assertTrue(Flags.flagName4());
> - assertTrue(fakeFlagsImpl.flagName4());
> -
> - mSetFlagsRule.enableFlags(Flags.FLAG_FLAG_NAME3);
> - mSetFlagsRule.disableFlags(Flags.FLAG_FLAG_NAME4);
> -
> - assertTrue(Flags.flagName3());
> - assertTrue(fakeFlagsImpl.flagName3());
> - assertFalse(Flags.flagName4());
> - assertFalse(fakeFlagsImpl.flagName4());
> - } else {
> - assertThrows(
> - NullPointerException.class,
> - () -> {
> - fakeFlagsImpl.flagName3();
> - });
> - assertFalse(Flags.flagName3());
> -
> - mSetFlagsRule.enableFlags(Flags.FLAG_FLAG_NAME3);
> -
> - assertTrue(Flags.flagName3());
> - assertTrue(fakeFlagsImpl.flagName3());
> - assertThrows(
> - NullPointerException.class,
> - () -> {
> - fakeFlagsImpl.flagName4();
> - });
> - assertThrows(
> - NullPointerException.class,
> - () -> {
> - Flags.flagName4();
> - });
> -
> - mSetFlagsRule.disableFlags(Flags.FLAG_FLAG_NAME4);
> -
> - assertFalse(Flags.flagName4());
> - assertFalse(fakeFlagsImpl.flagName4());
> - }
> - }
> -
> - @Test
> - public void getFakeFeatureFlags_castToWrongType() {
> - mSetFlagsRule.enableFlags(Flags.FLAG_FLAG_NAME3, Flags.FLAG_FLAG_NAME4);
> -
> - assertThrows(
> - IllegalArgumentException.class,
> - () -> {
> - FakeFeatureFlagsImpl fakeFlagsImpl =
> - mSetFlagsRule.getFakeFeatureFlags(FakeFeatureFlagsImpl.class);
> - });
> - assertThrows(
> - IllegalArgumentException.class,
> - () -> {
> - Flags fakeFlagsImpl = mSetFlagsRule.getFakeFeatureFlags(Flags.class);
> - });
> - assertThrows(
> - IllegalArgumentException.class,
> - () -> {
> - FeatureFlagsImpl fakeFlagsImpl =
> - mSetFlagsRule.getFakeFeatureFlags(FeatureFlagsImpl.class);
> - });
> - assertThrows(
> - IllegalArgumentException.class,
> - () -> {
> - FakeFeatureFlags fakeFlagsImpl =
> - mSetFlagsRule.getFakeFeatureFlags(FakeFeatureFlags.class);
> - });
> - }
> -
> - @Test
> - FakeFeatureFlagsImpl fakeFlagsImpl =
> - (FakeFeatureFlagsImpl) mSetFlagsRule.getFakeFeatureFlags(FeatureFlags.class);
> - fakeFlagsImpl.readOnlyFlagSet.add(Flags.FLAG_FLAG_NAME3);
> - mSetFlagsRule.enableFlags(Flags.FLAG_FLAG_NAME3);
> + mSetFlagsRule.enableFlags(Flags.FLAG_RO_ENABLED);
> - }
> -
> - private class FeatureFlagsImpl implements FeatureFlags {
> - @Override
> - public boolean flagName3() {
> - return false;
> - }
> -
> - @Override
> - public boolean flagName4() {
> - return true;
> - }
> - }
> -
> - private interface FakeFeatureFlags {
> - /** Returns the flag value. */
> - boolean flagName3();
> + assertThrows(
> + AssumptionViolatedException.class,
> + () -> {
> + mSetFlagsRule.disableFlags(Flags.FLAG_RO_ENABLED);
> + });
> + assertThrows(
> + AssumptionViolatedException.class,
> + () -> {
> + mSetFlagsRule.enableFlags(Flags.FLAG_RO_DISABLED);
> + });
> + assertThrows(
> + AssumptionViolatedException.class,
> + () -> {
> + mSetFlagsRule.disableFlags(Flags.FLAG_RO_DISABLED);
> + });
Original patch:
diff --git a/libraries/flag-helpers/junit/src_base/android/platform/test/flag/junit/SetFlagsRule.java b/libraries/flag-helpers/junit/src_base/android/platform/test/flag/junit/SetFlagsRule.java
old mode 100644
new mode 100644
--- a/libraries/flag-helpers/junit/src_base/android/platform/test/flag/junit/SetFlagsRule.java
+++ b/libraries/flag-helpers/junit/src_base/android/platform/test/flag/junit/SetFlagsRule.java
@@ -183,37 +183,6 @@
setFlagValue(fullFlagName, false);
}
}
-
- /**
- * Returns a FeatureFlags used by SetFlagsRule of given FeatureFlags
- *
- * @param featureFlagsClass The class of FeatureFlags. The interface of FakeFeatureFlagsImpl
- * @return A FakeFeatureFlagsImpl in type of FeatureFlags
- */
- public <T> T getFakeFeatureFlags(Class<T> featureFlagsClass) {
- if (!featureFlagsClass.isInterface()
- || !featureFlagsClass.getSimpleName().equals(FEATURE_FLAGS_CLASS_NAME)) {
- throw new IllegalArgumentException(
-
[[[Original patch trimmed due to size. Decoded string size: 11335. Decoded string SHA1: 5e69c5baf3fd2c06ad8faf0a7df442caf0f11bac.]]]
Result patch:
diff --git a/libraries/flag-helpers/junit/src_base/android/platform/test/flag/junit/SetFlagsRule.java b/libraries/flag-helpers/junit/src_base/android/platform/test/flag/junit/SetFlagsRule.java
index 0c86961..6c6e2fd 100644
--- a/libraries/flag-helpers/junit/src_base/android/platform/test/flag/junit/SetFlagsRule.java
+++ b/libraries/flag-helpers/junit/src_base/android/platform/test/flag/junit/SetFlagsRule.java
@@ -184,37 +184,6 @@
}
}
- /**
- * Returns a FeatureFlags used by SetFlagsRule of given FeatureFlags
- *
- * @param featureFlagsClass The class of FeatureFlags. The interface of FakeFeatureFlagsImpl
- * @return A FakeFeatureFlagsImpl in type of FeatureFlags
- */
- public <T> T getFakeFeatureFlags(Class<T> featureFlagsClass) {
- if (!featureFlagsClass.isInterface()
- || !featureFlagsClass.getSimpleName().equals(FEATURE_FLAGS_CLASS_NAME)) {
- throw new IllegalArgumentException(
- String.format(
-
[[[Result patch trimmed due to size. Decoded string size: 11451. Decoded string SHA1: 7433127fbb448fd40c3c3dc12433b48e0394301c.]]]
Change-Id: Ib9f814bbad6fb549685c6cdba85682fd0806c3c3
|
|
Bug: 322046897
Test: m betocq_test_suite and local_mobly_runner
Change-Id: Id2b110ec868f00521324545a4d62b93495055fb1
|
|
|
|
1) add iperf test.
2) add more parameters to enable/disable tests.
3) skip P/H flag read if it fails once.
Bug: 335236717
Test: compile and betocq test
Change-Id: I722b482c771294ccb6e1b8f727c7ca0f329e9fb5
|
|
A basic robolectric shared tests copied from MyRoboTests.
Test: atest MyRoboStrictTests && atest MyRoboTests
Bug: 334089788
Change-Id: Iab3b3fe130b15e694e07af9ba8a5c76b888cbaf7
|
|
This CL is created as a best effort to migrate test targets to the new Android ownership model.
It is based on historical data from repository history and insights from git blame.
Given the nature of this effort, there may be instances of incorrect attribution. If you find incorrect or unnecessary
attribution in this CL, please create a new CL to fix that.
For detailed guidelines and further information on the migration please refer to the link below,
go/new-android-ownership-model
Bug: 304529413
Test: N/A
Change-Id: Ia4b3c3f90bbe73cf6b64c041bc1ede86a95cbcc4
|
|
|
|
Also enable scc_2g_wfd_sta_test for all devices.
Bug: 310081463
Test: compile
Change-Id: I85aacb8fa7eaa7963c048b9818cac634d1a7c0ef
|
|
Test: compile
Bug: 333764163
Change-Id: I78c1bc94022427e7b3ad2752685a5ebcf2356fe0
|
|
|
|
1) Add test instructions.
2) Raise the baseline 5G80 SCC speed target to 40MB/s.
3) Enable BLE test in non-Quick-Start test.
4) Shorten the name of test config file and cuj test.
Bug: 333764163
Test: compile
Change-Id: Ic9b21d3876fbc5d75dc38268547597d7e86a4bdc
|
|
Bug: 333928068
Test: build
Change-Id: I3bf9276cbdd8b6e5fb8eaf29fe359f3cfc7b79d7
|
|
|
|
Also fix failure message formatting in FixedWifiMediumFunctionTestActor
Bug: 323080113
Test: local
Change-Id: I77d3cc24c9b1dace323b12621fe6e2f5f9a30f40
|
|
|
|
rename and cleanup the testbed name
rename the cuj_tests as compound_tests
Bug: 333764163
Test: Build
Change-Id: I5e8db1b6c2cbd0b70b67edcb9f8c903c61b9f09e
|
|
Test: CompilationFilterRuleTest
Bug: 331670804
Change-Id: I4f4dc67c88813aca0a4d2e1fe81a8db8b8f26adb
|
|
Bug: 333544187
Test: build
Change-Id: Ia28ba2731b541f146ffa8098fa8d661c246c2a42
|
|
Bug: 322046897
Test: atest -v batch_all_performance_test_suite -- --config local_dev_testbed.yml --testbed LocalPerformanceDefaultTestbed
Change-Id: I38b67e1083ed4bc89e8e578bfffb600e848683a1
|
|
Bug: 332919265
Test: build
Change-Id: I43edb767248d9b0e5363e5c2327c5d3e17d9d7bd
Merged-In: I43edb767248d9b0e5363e5c2327c5d3e17d9d7bd
|
|
TODO:
- hardware capabilities file
- atest targets per test class
- script to generate executable test zip
- openwrt support
Bug: 322046897
Test: atest; python3 with executable zip
Change-Id: I120c348b867f7fbf8adb286d2843500401368173
|
|
|
|
The exported flag doesn't have optimization method. Thus the test should
consider the flag is optimized.
Test: atest FlagJUnitTests
Bug: n/a
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:356c2c7530cb44c000d3df236e914403bd125c40)
Merged-In: I87850b03f4b14eece314dc9f1a9ecfafd9cd7299
Change-Id: I87850b03f4b14eece314dc9f1a9ecfafd9cd7299
|
|
Bug: 310232825
Test: Presubmit
Change-Id: I023021443d41206284136c6077b866b0de9d00f5
|
|
Files are moved from the internal projects.
Bug: b/331241555
Test: PowerRunnerTests, InstrumentationResultProtoParserTest
Change-Id: If9ae0370ee7f906790d66a56d878aaf2d76270f6
|
|
callback." into main
|
|
|
|
Bug: 330554564
Test: Local
Change-Id: Iaf1e930dbaa97682a824775a9c8a0ecaadc5e091
|
|
|
|
Test: TH
Change-Id: I5e258770fdf4364543da184548b7e65492cbabb2
|
|
Bug: 326389812
Test: Tested on forrest
Change-Id: I62f7a9c8ffdb4696e9c5ee2dc307bf841496eada
|
|
Bug: 312235820
Test: betocq test
Change-Id: Ic96bf334233a34aedd8b1f3cfe2259d0cd5bcf61
|
|
Bug: b/329091633
Test: Local
Change-Id: I3f2f05388575ae67c0d60825477059df3b3e5a49
|
|
Bug: 313924276
Change-Id: I79502e760002be572f3da77ae31e77e77e2d1e80
Test: TH
|
|
Test: N/A
Bug: 326389812
Change-Id: I333a9f5db25123dd44289c9c0cd6256f1984786e
|
|
Bug: 319669529
Merged-In: I5a93edd19911029b945e339ae0ba024f30539e88
Change-Id: I2b51eee67690675ab7f57715e33d502f05fac9e6
|
|
Original change: https://android-review.googlesource.com/c/platform/platform_testing/+/2969438
Change-Id: I5a93edd19911029b945e339ae0ba024f30539e88
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Original change: https://android-review.googlesource.com/c/platform/platform_testing/+/2969436
Change-Id: I0cb716ec6abeae741fff82ab540e96df35894ff6
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
This property is a no-op for this module type.
Test: m nothing
Bug: 215379393
Change-Id: I26197df373c1da9cad067ae0bdf5a505d24f45d1
|
|
This property is a no-op for this module type.
Test: m nothing
Bug: 215379393
Change-Id: I558836b52c181ba4491fadb2018e0457afaa1720
|
|
Original change: https://android-review.googlesource.com/c/platform/platform_testing/+/2963542
Change-Id: I792458622d0770b111c74fc82c6e0ec2f95af3ae
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
|
|
Original change: https://android-review.googlesource.com/c/platform/platform_testing/+/2959192
Change-Id: I03af10f7ff34245114adbcb1ea342c96f810b581
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Original change: https://android-review.googlesource.com/c/platform/platform_testing/+/2962425
Change-Id: I442ecc79e2722d2bb0236a0ee8861986d3de42d3
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
|