Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f08ebe00 authored by David Su's avatar David Su
Browse files

Fix AccessPointPreferenceTest#refresh_openNetwork_updateContentDescription 

AccessPoint now uses
WifiManager#calculateSignalLevel(rssi) instead of
WifiManager#calculateSignalLevel(rssi, numLevels).

WifiManager#calculateSignalLevel(rssi, numLevels)
previously redirected to
ShadowWifiManager#calculateSignalLevel(rssi, numLevels),
which always returned numLevels - 1 = 5 - 1 = 4.

WifiManager#calculateSignalLevel(rssi) is not
redirected by ShadowWifiManager, so it returns 0.
Fixed by updating the expected value of the assert.

Bug: 147703223
Test: make RunSettingsLibRoboTests -j40
Change-Id: Ief86fde8ced462e9e9cbb75af0b95c2f7ab26cdf
parent 2f269667

packages/SettingsLib/tests/robotests/src/com/android/settingslib/wifi/AccessPointPreferenceTest.java

+1 −3
Original line number Diff line number Diff line
@@ -26,7 +26,6 @@ import android.content.Context; 
import android.graphics.drawable.ColorDrawable;



import org.junit.Before;

import org.junit.Ignore;

import org.junit.Test;

import org.junit.runner.RunWith;

import org.mockito.Mock;
@@ -59,7 +58,6 @@ public class AccessPointPreferenceTest { 
    }



    @Test

    @Ignore

    public void refresh_openNetwork_updateContentDescription() {

        final String ssid = "ssid";

        final String summary = "connected";
@@ -74,7 +72,7 @@ public class AccessPointPreferenceTest { 


        assertThat(AccessPointPreference.buildContentDescription(

                RuntimeEnvironment.application, pref, ap))

                .isEqualTo("ssid,connected,Wifi signal full.,Secure network");

                .isEqualTo("ssid,connected,Wifi disconnected.,Secure network");

    }



    @Test