Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(56)

Unified Diff: chrome/android/junit/src/org/chromium/chrome/browser/ntp/cards/SuggestionsSectionTest.java

Issue 2398463005: Revert of 📰 Use the separate button style for the NoArticles status (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/junit/src/org/chromium/chrome/browser/ntp/cards/SuggestionsSectionTest.java
diff --git a/chrome/android/junit/src/org/chromium/chrome/browser/ntp/cards/SuggestionsSectionTest.java b/chrome/android/junit/src/org/chromium/chrome/browser/ntp/cards/SuggestionsSectionTest.java
index b79c074da4c22c6cf53316eb30650d3b43516bc2..4539ac8e0871d3cc7b13d42bc1cac612c4052a4e 100644
--- a/chrome/android/junit/src/org/chromium/chrome/browser/ntp/cards/SuggestionsSectionTest.java
+++ b/chrome/android/junit/src/org/chromium/chrome/browser/ntp/cards/SuggestionsSectionTest.java
@@ -4,28 +4,27 @@
package org.chromium.chrome.browser.ntp.cards;
+import static org.chromium.chrome.browser.ntp.cards.ContentSuggestionsTestUtils.createDummySuggestions;
+import static org.chromium.chrome.browser.ntp.cards.ContentSuggestionsTestUtils.createInfo;
+import static org.chromium.chrome.browser.ntp.cards.ContentSuggestionsTestUtils.createSection;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNotNull;
-import static org.mockito.ArgumentMatchers.any;
-import static org.mockito.ArgumentMatchers.anyInt;
-import static org.mockito.ArgumentMatchers.eq;
+import static org.junit.Assert.assertNull;
+import static org.mockito.Mockito.any;
+import static org.mockito.Mockito.anyInt;
+import static org.mockito.Mockito.eq;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.never;
import static org.mockito.Mockito.times;
import static org.mockito.Mockito.verify;
-import org.junit.Test;
-import org.junit.runner.RunWith;
-import org.robolectric.annotation.Config;
-
-import static org.chromium.chrome.browser.ntp.cards.ContentSuggestionsTestUtils.createDummySuggestions;
-import static org.chromium.chrome.browser.ntp.cards.ContentSuggestionsTestUtils.createInfo;
-import static org.chromium.chrome.browser.ntp.cards.ContentSuggestionsTestUtils.createSection;
-
import org.chromium.base.test.util.Feature;
import org.chromium.chrome.browser.ntp.snippets.CategoryStatus;
import org.chromium.chrome.browser.ntp.snippets.SnippetArticle;
import org.chromium.testing.local.LocalRobolectricTestRunner;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.robolectric.annotation.Config;
import java.util.List;
@@ -35,10 +34,8 @@
@RunWith(LocalRobolectricTestRunner.class)
@Config(manifest = Config.NONE)
public class SuggestionsSectionTest {
- /**
- * Number of items in a section when there are no suggestions: header, status, action, progress.
- */
- private static final int EMPTY_SECTION_COUNT = 4;
+ /** Number of items in a section when there are no suggestions: header, status, progress. */
+ private static final int EMPTY_SECTION_COUNT = 3;
@Test
@Feature({"Ntp"})
@@ -47,17 +44,31 @@
List<SnippetArticle> snippets = createDummySuggestions(3);
SuggestionsSection section;
- section = new SuggestionsSection(createInfo(42, true, true), observerMock);
+ // Part 1: ShowMoreButton = true.
+ section = new SuggestionsSection(42, createInfo(true, true), observerMock, null);
section.setStatus(CategoryStatus.AVAILABLE);
assertNotNull(section.getActionItem());
- // Without snippets.
+ // 1.1: Without snippets.
assertEquals(-1, section.getDismissSiblingPosDelta(section.getActionItem()));
assertEquals(1, section.getDismissSiblingPosDelta(section.getStatusItem()));
- // With snippets.
+ // 1.2: With snippets.
section.setSuggestions(snippets, CategoryStatus.AVAILABLE);
assertEquals(0, section.getDismissSiblingPosDelta(section.getActionItem()));
+ assertEquals(0, section.getDismissSiblingPosDelta(section.getStatusItem()));
+ assertEquals(0, section.getDismissSiblingPosDelta(snippets.get(0)));
+
+ // Part 2: ShowMoreButton = false.
+ section = new SuggestionsSection(42, createInfo(false, true), observerMock, null);
+ section.setStatus(CategoryStatus.AVAILABLE);
+ assertNull(section.getActionItem());
+
+ // 2.1: Without snippets.
+ assertEquals(0, section.getDismissSiblingPosDelta(section.getStatusItem()));
+
+ // 2.2: With snippets.
+ section.setSuggestions(snippets, CategoryStatus.AVAILABLE);
assertEquals(0, section.getDismissSiblingPosDelta(section.getStatusItem()));
assertEquals(0, section.getDismissSiblingPosDelta(snippets.get(0)));
}
@@ -85,6 +96,7 @@
@Feature({"Ntp"})
public void testSetStatusNotification() {
ItemGroup.Observer observerMock = mock(ItemGroup.Observer.class);
+ final int emptySectionCount = 3;
final int suggestionCount = 5;
List<SnippetArticle> snippets = createDummySuggestions(suggestionCount);

Powered by Google App Engine
This is Rietveld 408576698