summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
17 hoursIntegrate the latest changesHEADmastermainKai Shi
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
38 hoursIntegrate the latest changesKai Shi
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
3 days[BetoCQ] Replace all "sponge_properties" with "properties"Xianyuan Jia
Bug: 338102234 Test: local Change-Id: Ib0c358dd111ae40e71311633184dfb7c74898c53
5 daysSimplify how SetFlagsRule provides defaults to FakeFeatureFlagsImplJeff DeCew
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
7 daysMerge "Remove SetFlagsRule.getFakeFeatureFlags()" into mainJeff DeCew
8 daysRemove SetFlagsRule.getFakeFeatureFlags()Jeff DeCew
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
8 daysAdd embedded_launcher: false to betocq_test_suiteXianyuan Jia
Bug: 322046897 Test: m betocq_test_suite and local_mobly_runner Change-Id: Id2b110ec868f00521324545a4d62b93495055fb1
8 daysMerge "Integrate the latest changes" into mainKai Shi
8 daysIntegrate the latest changesKai Shi
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
10 daysAdded a strict robolectric test moduleKevin Liu
A basic robolectric shared tests copied from MyRoboTests. Test: atest MyRoboStrictTests && atest MyRoboTests Bug: 334089788 Change-Id: Iab3b3fe130b15e694e07af9ba8a5c76b888cbaf7
10 daysMigrate Test Targets to New Android Ownership ModelAditya Choudhary
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
2024-04-17Merge "Add Aware test case and enable it for QuickShare CUJ." into mainTreehugger Robot
2024-04-17Add Aware test case and enable it for QuickShare CUJ.Kai Shi
Also enable scc_2g_wfd_sta_test for all devices. Bug: 310081463 Test: compile Change-Id: I85aacb8fa7eaa7963c048b9818cac634d1a7c0ef
2024-04-17Update test instructions and fix a typo in BLE testKai Shi
Test: compile Bug: 333764163 Change-Id: I78c1bc94022427e7b3ad2752685a5ebcf2356fe0
2024-04-12Merge "Add the test instructions" into mainTreehugger Robot
2024-04-11Add the test instructionsKai Shi
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
2024-04-11fix typoes for scc_5g_all_wifi_dbs_2g_sta_testSteven Liu
Bug: 333928068 Test: build Change-Id: I3bf9276cbdd8b6e5fb8eaf29fe359f3cfc7b79d7
2024-04-11Merge "Add run_identifier property to BetoCQ v2 tests" into mainTreehugger Robot
2024-04-11Add run_identifier property to BetoCQ v2 testsXianyuan Jia
Also fix failure message formatting in FixedWifiMediumFunctionTestActor Bug: 323080113 Test: local Change-Id: I77d3cc24c9b1dace323b12621fe6e2f5f9a30f40
2024-04-11Merge "Add an option for package name in CompilationFilterRule" into mainTreehugger Robot
2024-04-10rename test suite nameSteven Liu
rename and cleanup the testbed name rename the cuj_tests as compound_tests Bug: 333764163 Test: Build Change-Id: I5e8db1b6c2cbd0b70b67edcb9f8c903c61b9f09e
2024-04-10Add an option for package name in CompilationFilterRuleBecky Wen
Test: CompilationFilterRuleTest Bug: 331670804 Change-Id: I4f4dc67c88813aca0a4d2e1fe81a8db8b8f26adb
2024-04-09correct the result message for the failure of 'prior BT' connection.Steven Liu
Bug: 333544187 Test: build Change-Id: Ia28ba2731b541f146ffa8098fa8d661c246c2a42
2024-04-08Integrate the latest changesKai Shi
Bug: 322046897 Test: atest -v batch_all_performance_test_suite -- --config local_dev_testbed.yml --testbed LocalPerformanceDefaultTestbed Change-Id: I38b67e1083ed4bc89e8e578bfffb600e848683a1
2024-04-08fix typo for the low throughput tipSteven Liu
Bug: 332919265 Test: build Change-Id: I43edb767248d9b0e5363e5c2327c5d3e17d9d7bd Merged-In: I43edb767248d9b0e5363e5c2327c5d3e17d9d7bd
2024-04-03Initial commit for BetoCQ v2Xianyuan Jia
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
2024-03-29Merge "skip verify optimization if flag is exported" into mainZhi Dou
2024-03-28skip verify optimization if flag is exportedZhi Dou
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
2024-03-28shell-as: Remove hardcoded PAGE_SIZE usageVilas Bhat
Bug: 310232825 Test: Presubmit Change-Id: I023021443d41206284136c6077b866b0de9d00f5
2024-03-27Add power test host runner.Gopinath Elanchezhian
Files are moved from the internal projects. Bug: b/331241555 Test: PowerRunnerTests, InstrumentationResultProtoParserTest Change-Id: If9ae0370ee7f906790d66a56d878aaf2d76270f6
2024-03-26Merge "Added a unit test for DynamicRuleChain that tests the failed() ↵Treehugger Robot
callback." into main
2024-03-22Merge "Build power trace config" into mainVinod Benure
2024-03-22Build power trace configVinod Benure
Bug: 330554564 Test: Local Change-Id: Iaf1e930dbaa97682a824775a9c8a0ecaadc5e091
2024-03-21Merge "Allow devs to ignore the 'Wrong java version' error" into mainSorin Basca
2024-03-21Allow devs to ignore the 'Wrong java version' errorSorin Basca
Test: TH Change-Id: I5e258770fdf4364543da184548b7e65492cbabb2
2024-03-19Change dittobench output to logcat and don't wait for completionParth Sane
Bug: 326389812 Test: Tested on forrest Change-Id: I62f7a9c8ffdb4696e9c5ee2dc307bf841496eada
2024-03-13Allow connection medium selectionKai Shi
Bug: 312235820 Test: betocq test Change-Id: Ic96bf334233a34aedd8b1f3cfe2259d0cd5bcf61
2024-03-11Build boot trace configs.Gopinath Elanchezhian
Bug: b/329091633 Test: Local Change-Id: I3f2f05388575ae67c0d60825477059df3b3e5a49
2024-03-11Add Java 21 as a known versionSorin Basca
Bug: 313924276 Change-Id: I79502e760002be572f3da77ae31e77e77e2d1e80 Test: TH
2024-03-08Add a crystalball test rule for dittobenchParth Sane
Test: N/A Bug: 326389812 Change-Id: I333a9f5db25123dd44289c9c0cd6256f1984786e
2024-03-06Merge Android 14 QPR2 to AOSP mainXin Li
Bug: 319669529 Merged-In: I5a93edd19911029b945e339ae0ba024f30539e88 Change-Id: I2b51eee67690675ab7f57715e33d502f05fac9e6
2024-02-21Drop platform_apis from java_library am: 105c0930eftemp_319669529Spandan Das
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>
2024-02-21Drop platform_apis from java_library am: 41805ae8e6Spandan Das
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>
2024-02-21Drop platform_apis from java_librarySpandan Das
This property is a no-op for this module type. Test: m nothing Bug: 215379393 Change-Id: I26197df373c1da9cad067ae0bdf5a505d24f45d1
2024-02-21Drop platform_apis from java_librarySpandan Das
This property is a no-op for this module type. Test: m nothing Bug: 215379393 Change-Id: I558836b52c181ba4491fadb2018e0457afaa1720
2024-02-14Merge "Remove timestamps from statsd-config-protos.jar" into main am: fead5397a3Treehugger Robot
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>
2024-02-14Merge "Remove timestamps from statsd-config-protos.jar" into mainTreehugger Robot
2024-02-14Merge "Add telephony country code overriding" into main am: 64055a915eTreehugger Robot
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>
2024-02-14Replace "sponge_properties" with generic "properties" am: b4a0639207Xianyuan Jia
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>
2024-02-14Merge "Add telephony country code overriding" into mainTreehugger Robot