From 114a5b2fd3d53c452280fb77d3bc8c7976181515 Mon Sep 17 00:00:00 2001 From: Aaron Huang Date: Fri, 6 Aug 2021 17:53:14 +0800 Subject: [PATCH] Skip wifi stats test if link layer stats is not supported Wifi link layer is an optional feature so this test will be failed on wifi stats check if a device does not support it. Add a check to know if the device supports wifi link layer stats and skip it if it is not supported. Bug: 195518957 Test: CtsNetTestCases:BatteryStatsManagerTest Change-Id: I592dd5f1d6e13b020beadb11b9d913857a82e524 Merged-In: I592dd5f1d6e13b020beadb11b9d913857a82e524 --- .../net/src/android/net/cts/BatteryStatsManagerTest.java | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/tests/cts/net/src/android/net/cts/BatteryStatsManagerTest.java b/tests/cts/net/src/android/net/cts/BatteryStatsManagerTest.java index 4d60279f0f..cc13d34e6c 100644 --- a/tests/cts/net/src/android/net/cts/BatteryStatsManagerTest.java +++ b/tests/cts/net/src/android/net/cts/BatteryStatsManagerTest.java @@ -31,6 +31,7 @@ import android.content.Context; import android.net.ConnectivityManager; import android.net.Network; import android.net.cts.util.CtsNetUtils; +import android.net.wifi.WifiManager; import android.os.BatteryStatsManager; import android.os.Build; import android.os.connectivity.CellularBatteryStats; @@ -72,6 +73,7 @@ public class BatteryStatsManagerTest{ private Context mContext; private BatteryStatsManager mBsm; private ConnectivityManager mCm; + private WifiManager mWm; private CtsNetUtils mCtsNetUtils; @Before @@ -79,6 +81,7 @@ public class BatteryStatsManagerTest{ mContext = getContext(); mBsm = mContext.getSystemService(BatteryStatsManager.class); mCm = mContext.getSystemService(ConnectivityManager.class); + mWm = mContext.getSystemService(WifiManager.class); mCtsNetUtils = new CtsNetUtils(mContext); } @@ -130,6 +133,11 @@ public class BatteryStatsManagerTest{ cellularStatsAfter -> cellularBatteryStatsIncreased( cellularStatsBefore, cellularStatsAfter))); + if (!mWm.isEnhancedPowerReportingSupported()) { + Log.d(TAG, "Skip wifi stats test because wifi does not support link layer stats."); + return; + } + WifiBatteryStats wifiStatsBefore = runAsShell(UPDATE_DEVICE_STATS, mBsm::getWifiBatteryStats);