PawletOS/app_Settings
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Improve visual - Collapse the conditionals in all cases

Browse Source 
Changing the threshold value to make collapsing mechanism of conditionals
work in all cases, where the threshold value is changed to 0 from 2.

Bug: 122310542
Test: robotests
Change-Id: I227114acdc6770baa0c133397d08e3ad77f6c572
This commit is contained in:
Mill Chen
2019-01-04 21:28:20 +08:00
parent 06c514db6f
commit 8a1bccbc30
2 changed files with 29 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/com/android/settings/homepage/contextualcards/conditional/ConditionContextualCardController.java
View File

@@ -40,7 +40,7 @@ import java.util.stream.Collectors;
*/
public class ConditionContextualCardController implements ContextualCardController,
ConditionListener, LifecycleObserver, OnStart, OnStop {
public static final int EXPANDING_THRESHOLD = 2;
public static final int EXPANDING_THRESHOLD = 0;
private static final double UNSUPPORTED_RANKING = -99999.0;
private static final String TAG = "ConditionCtxCardCtrl";

30
tests/robotests/src/com/android/settings/homepage/contextualcards/conditional/ConditionContextualCardControllerTest.java
View File

@@ -112,7 +112,8 @@ public class ConditionContextualCardControllerTest {
}
@Test
public void getConditionalCards_hasOneConditionCard_shouldGetOneFullWidthCard() {
public void getConditionalCards_hasOneConditionCardAndExpanded_shouldGetOneFullWidthCard() {
mController.setIsExpanded(true);
final Map<Integer, List<ContextualCard>> conditionalCards =
mController.buildConditionalCardsWithFooterOrHeader(generateConditionCards(1));
@@ -120,11 +121,24 @@ public class ConditionContextualCardControllerTest {
assertThat(conditionalCards.get(CardType.CONDITIONAL)).hasSize(1);
assertThat(conditionalCards.get(CardType.CONDITIONAL).get(0).isHalfWidth()).isFalse();
assertThat(conditionalCards.get(CardType.CONDITIONAL_HEADER)).isEmpty();
assertThat(conditionalCards.get(CardType.CONDITIONAL_FOOTER)).isNotEmpty();
}
@Test
public void getConditionalCards_hasOneConditionCardAndCollapsed_shouldGetConditionalHeader() {
mController.setIsExpanded(false);
final Map<Integer, List<ContextualCard>> conditionalCards =
mController.buildConditionalCardsWithFooterOrHeader(generateConditionCards(1));
assertThat(conditionalCards).hasSize(3);
assertThat(conditionalCards.get(CardType.CONDITIONAL)).isEmpty();
assertThat(conditionalCards.get(CardType.CONDITIONAL_HEADER)).isNotEmpty();
assertThat(conditionalCards.get(CardType.CONDITIONAL_FOOTER)).isEmpty();
}
@Test
public void getConditionalCards_hasTwoConditionCards_shouldGetTwoHalfWidthCards() {
public void getConditionalCards_hasTwoConditionCardsAndExpanded_shouldGetTwoHalfWidthCards() {
mController.setIsExpanded(true);
final Map<Integer, List<ContextualCard>> conditionalCards =
mController.buildConditionalCardsWithFooterOrHeader(generateConditionCards(2));
@@ -134,6 +148,18 @@ public class ConditionContextualCardControllerTest {
assertThat(card.isHalfWidth()).isTrue();
}
assertThat(conditionalCards.get(CardType.CONDITIONAL_HEADER)).isEmpty();
assertThat(conditionalCards.get(CardType.CONDITIONAL_FOOTER)).isNotEmpty();
}
@Test
public void getConditionalCards_hasTwoConditionCardsAndCollapsed_shouldGetConditionalHeader() {
mController.setIsExpanded(false);
final Map<Integer, List<ContextualCard>> conditionalCards =
mController.buildConditionalCardsWithFooterOrHeader(generateConditionCards(2));
assertThat(conditionalCards).hasSize(3);
assertThat(conditionalCards.get(CardType.CONDITIONAL)).isEmpty();
assertThat(conditionalCards.get(CardType.CONDITIONAL_HEADER)).isNotEmpty();
assertThat(conditionalCards.get(CardType.CONDITIONAL_FOOTER)).isEmpty();
}