Merge "[Settings] Fix DataUsageSummaryPreferenceControllerTest" into rvc-dev

This commit is contained in:
Bonian Chen
2020-04-22 04:27:43 +00:00
committed by Android (Google) Code Review
2 changed files with 5 additions and 14 deletions

View File

@@ -114,7 +114,11 @@ public class DataUsageSummaryPreferenceController extends BasePreferenceControll
init(subscriptionId); init(subscriptionId);
} }
protected void init(int subscriptionId) { /**
* Initialize based on subscription ID provided
* @param subscriptionId is the target subscriptionId
*/
public void init(int subscriptionId) {
mSubscriptionId = subscriptionId; mSubscriptionId = subscriptionId;
mDefaultTemplate = DataUsageUtils.getDefaultTemplate(mContext, mSubscriptionId); mDefaultTemplate = DataUsageUtils.getDefaultTemplate(mContext, mSubscriptionId);

View File

@@ -55,7 +55,6 @@ import com.android.settingslib.net.DataUsageController;
import org.junit.After; import org.junit.After;
import org.junit.Before; import org.junit.Before;
import org.junit.Ignore;
import org.junit.Test; import org.junit.Test;
import org.junit.runner.RunWith; import org.junit.runner.RunWith;
import org.mockito.Answers; import org.mockito.Answers;
@@ -170,7 +169,6 @@ public class DataUsageSummaryPreferenceControllerTest {
} }
@Test @Test
@Ignore
public void testSummaryUpdate_onePlan_basic() { public void testSummaryUpdate_onePlan_basic() {
final long now = System.currentTimeMillis(); final long now = System.currentTimeMillis();
final DataUsageController.DataUsageInfo info = createTestDataUsageInfo(now); final DataUsageController.DataUsageInfo info = createTestDataUsageInfo(now);
@@ -198,7 +196,6 @@ public class DataUsageSummaryPreferenceControllerTest {
} }
@Test @Test
@Ignore
public void testSummaryUpdate_noPlan_basic() { public void testSummaryUpdate_noPlan_basic() {
final long now = System.currentTimeMillis(); final long now = System.currentTimeMillis();
final DataUsageController.DataUsageInfo info = createTestDataUsageInfo(now); final DataUsageController.DataUsageInfo info = createTestDataUsageInfo(now);
@@ -227,7 +224,6 @@ public class DataUsageSummaryPreferenceControllerTest {
} }
@Test @Test
@Ignore
public void testSummaryUpdate_noCarrier_basic() { public void testSummaryUpdate_noCarrier_basic() {
final long now = System.currentTimeMillis(); final long now = System.currentTimeMillis();
final DataUsageController.DataUsageInfo info = createTestDataUsageInfo(now); final DataUsageController.DataUsageInfo info = createTestDataUsageInfo(now);
@@ -255,7 +251,6 @@ public class DataUsageSummaryPreferenceControllerTest {
} }
@Test @Test
@Ignore
public void testSummaryUpdate_noPlanData_basic() { public void testSummaryUpdate_noPlanData_basic() {
final long now = System.currentTimeMillis(); final long now = System.currentTimeMillis();
@@ -283,7 +278,6 @@ public class DataUsageSummaryPreferenceControllerTest {
} }
@Test @Test
@Ignore
public void testSummaryUpdate_noLimitNoWarning() { public void testSummaryUpdate_noLimitNoWarning() {
final long now = System.currentTimeMillis(); final long now = System.currentTimeMillis();
final DataUsageController.DataUsageInfo info = createTestDataUsageInfo(now); final DataUsageController.DataUsageInfo info = createTestDataUsageInfo(now);
@@ -300,7 +294,6 @@ public class DataUsageSummaryPreferenceControllerTest {
} }
@Test @Test
@Ignore
public void testSummaryUpdate_warningOnly() { public void testSummaryUpdate_warningOnly() {
final long now = System.currentTimeMillis(); final long now = System.currentTimeMillis();
final DataUsageController.DataUsageInfo info = createTestDataUsageInfo(now); final DataUsageController.DataUsageInfo info = createTestDataUsageInfo(now);
@@ -321,7 +314,6 @@ public class DataUsageSummaryPreferenceControllerTest {
} }
@Test @Test
@Ignore
public void testSummaryUpdate_limitOnly() { public void testSummaryUpdate_limitOnly() {
final long now = System.currentTimeMillis(); final long now = System.currentTimeMillis();
final DataUsageController.DataUsageInfo info = createTestDataUsageInfo(now); final DataUsageController.DataUsageInfo info = createTestDataUsageInfo(now);
@@ -342,7 +334,6 @@ public class DataUsageSummaryPreferenceControllerTest {
} }
@Test @Test
@Ignore
public void testSummaryUpdate_limitAndWarning() { public void testSummaryUpdate_limitAndWarning() {
final long now = System.currentTimeMillis(); final long now = System.currentTimeMillis();
final DataUsageController.DataUsageInfo info = createTestDataUsageInfo(now); final DataUsageController.DataUsageInfo info = createTestDataUsageInfo(now);
@@ -365,7 +356,6 @@ public class DataUsageSummaryPreferenceControllerTest {
} }
@Test @Test
@Ignore
public void testSummaryUpdate_noSim_shouldSetWifiMode() { public void testSummaryUpdate_noSim_shouldSetWifiMode() {
mController.init(SubscriptionManager.INVALID_SUBSCRIPTION_ID); mController.init(SubscriptionManager.INVALID_SUBSCRIPTION_ID);
mController.mDataUsageController = mDataUsageController; mController.mDataUsageController = mDataUsageController;
@@ -393,13 +383,11 @@ public class DataUsageSummaryPreferenceControllerTest {
} }
@Test @Test
@Ignore
public void testMobileData_preferenceAvailable() { public void testMobileData_preferenceAvailable() {
assertThat(mController.getAvailabilityStatus()).isEqualTo(AVAILABLE); assertThat(mController.getAvailabilityStatus()).isEqualTo(AVAILABLE);
} }
@Test @Test
@Ignore
public void testMobileData_noSimNoWifi_preferenceDisabled() { public void testMobileData_noSimNoWifi_preferenceDisabled() {
final int subscriptionId = SubscriptionManager.INVALID_SUBSCRIPTION_ID; final int subscriptionId = SubscriptionManager.INVALID_SUBSCRIPTION_ID;
mController.init(subscriptionId); mController.init(subscriptionId);
@@ -412,7 +400,6 @@ public class DataUsageSummaryPreferenceControllerTest {
} }
@Test @Test
@Ignore
public void testMobileData_noSimWifi_preferenceDisabled() { public void testMobileData_noSimWifi_preferenceDisabled() {
final int subscriptionId = SubscriptionManager.INVALID_SUBSCRIPTION_ID; final int subscriptionId = SubscriptionManager.INVALID_SUBSCRIPTION_ID;
mController.init(subscriptionId); mController.init(subscriptionId);