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

Commit 6a1f77ef authored by Fan Zhang's avatar Fan Zhang Committed by Android (Google) Code Review
Browse files

Merge "Use proto in contextualCard object"

parents 2f324c94 8cd659af
Loading
Loading
Loading
Loading
−2.36 KiB (2.79 KiB)

File changed.

No diff preview for this file type.

+11 −12
Original line number Diff line number Diff line
@@ -21,14 +21,12 @@ import static android.provider.SettingsSlicesContract.KEY_WIFI;
import android.annotation.Nullable;

import com.android.settings.homepage.deviceinfo.DataUsageSlice;
import com.android.settings.intelligence.ContextualCardProto.ContextualCard;
import com.android.settings.intelligence.ContextualCardProto.ContextualCardList;
import com.android.settings.wifi.WifiSlice;

import com.google.android.settings.intelligence.libs.contextualcards.ContextualCard;
import com.google.android.settings.intelligence.libs.contextualcards.ContextualCardProvider;

import java.util.ArrayList;
import java.util.List;

/** Provides dynamic card for SettingsIntelligence. */
public class SettingsContextualCardProvider extends ContextualCardProvider {

@@ -36,21 +34,22 @@ public class SettingsContextualCardProvider extends ContextualCardProvider {

    @Override
    @Nullable
    public List<ContextualCard> getContextualCards() {
        final List<ContextualCard> cards = new ArrayList<>();
    public ContextualCardList getContextualCards() {
        final ContextualCard wifiCard =
                new ContextualCard.Builder()
                ContextualCard.newBuilder()
                        .setSliceUri(WifiSlice.WIFI_URI.toString())
                        .setName(KEY_WIFI)
                        .setCardName(KEY_WIFI)
                        .build();
        final ContextualCard dataUsageCard =
                new ContextualCard.Builder()
                ContextualCard.newBuilder()
                        .setSliceUri(DataUsageSlice.DATA_USAGE_CARD_URI.toString())
                        .setName(DataUsageSlice.PATH_DATA_USAGE_CARD)
                        .setCardName(DataUsageSlice.PATH_DATA_USAGE_CARD)
                        .build();
        final ContextualCardList cards = ContextualCardList.newBuilder()
                .addCard(wifiCard)
                .addCard(dataUsageCard)
                .build();

        cards.add(wifiCard);
        cards.add(dataUsageCard);
        return cards;
    }
}
+6 −7
Original line number Diff line number Diff line
@@ -24,9 +24,9 @@ import android.content.Context;
import android.net.Uri;
import android.os.Bundle;

import com.android.settings.intelligence.ContextualCardProto.ContextualCardList;
import com.android.settings.testutils.SettingsRobolectricTestRunner;

import com.google.android.settings.intelligence.libs.contextualcards.ContextualCard;
import com.google.android.settings.intelligence.libs.contextualcards.ContextualCardProvider;

import org.junit.Before;
@@ -34,8 +34,6 @@ import org.junit.Test;
import org.junit.runner.RunWith;
import org.robolectric.RuntimeEnvironment;

import java.util.ArrayList;

@RunWith(SettingsRobolectricTestRunner.class)
public class SettingsContextualCardProviderTest {

@@ -54,11 +52,12 @@ public class SettingsContextualCardProviderTest {
    }

    @Test
    public void contentProviderCall_returnCorrectSize() {
    public void contentProviderCall_returnCorrectSize() throws Exception {
        final Bundle returnValue =
                mResolver.call(mUri, ContextualCardProvider.METHOD_GET_CARD_LIST, "", null);
        final ArrayList<ContextualCard> cards =
                returnValue.getParcelableArrayList(ContextualCardProvider.BUNDLE_CARD_LIST);
        assertThat(cards.size()).isEqualTo(2);
        final ContextualCardList cards =
              ContextualCardList.parseFrom(
                  returnValue.getByteArray(ContextualCardProvider.BUNDLE_CARD_LIST));
        assertThat(cards.getCardCount()).isEqualTo(2);
    }
}
 No newline at end of file