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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/ntp/cards/NewTabPageRecyclerViewTest.java

Issue 2532953002: [Android NTP] Reduce API surface of NewTabPageAdapter. (Closed)
Patch Set: move Created 4 years 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/javatests/src/org/chromium/chrome/browser/ntp/cards/NewTabPageRecyclerViewTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/ntp/cards/NewTabPageRecyclerViewTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/ntp/cards/NewTabPageRecyclerViewTest.java
index 2c5ecadbed99fa26ace4e22770f969c85b678c98..fba2acfc58a96e66bc3c984261f2ed358ae4f8c9 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/ntp/cards/NewTabPageRecyclerViewTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/ntp/cards/NewTabPageRecyclerViewTest.java
@@ -117,7 +117,7 @@ public class NewTabPageRecyclerViewTest extends ChromeTabbedActivityTestBase {
// Scroll the last suggestion into view and click it.
SnippetArticle suggestion = suggestions.get(suggestions.size() - 1);
- int suggestionPosition = getAdapter().getSuggestionPosition(suggestion);
+ int suggestionPosition = getSuggestionPosition(suggestion);
scrollToPosition(suggestionPosition);
final View suggestionView = waitForView(suggestionPosition);
ChromeTabUtils.waitForTabPageLoaded(mTab, new Runnable() {
@@ -135,12 +135,13 @@ public class NewTabPageRecyclerViewTest extends ChromeTabbedActivityTestBase {
public void testAllDismissed() throws InterruptedException, TimeoutException {
setSuggestionsAndWaitForUpdate(3);
assertEquals(3, mSource.getSuggestionsForCategory(KnownCategories.ARTICLES).size());
- assertFalse(getAdapter().hasAllBeenDismissed());
+ assertEquals(RecyclerView.NO_POSITION,
+ getAdapter().getFirstPositionForType(ItemViewType.ALL_DISMISSED));
assertEquals(1, mSource.getCategories().length);
assertEquals(KnownCategories.ARTICLES, mSource.getCategories()[0]);
// Dismiss the sign in promo.
- int signinPromoPosition = getAdapter().getSignInPromoPosition();
+ int signinPromoPosition = getAdapter().getFirstPositionForType(ItemViewType.PROMO);
scrollToPosition(signinPromoPosition);
View signinPromoView = waitForView(signinPromoPosition);
getAdapter().dismissItem(signinPromoPosition);
@@ -155,11 +156,11 @@ public class NewTabPageRecyclerViewTest extends ChromeTabbedActivityTestBase {
waitForViewToDetach(cardView);
cardPosition = getAdapter().getFirstCardPosition();
}
- assertTrue(getAdapter().hasAllBeenDismissed());
assertEquals(0, mSource.getCategories().length);
// Click the refresh button on the all dismissed item.
- int allDismissedPosition = getAdapter().getLastContentItemPosition();
+ int allDismissedPosition = getAdapter().getFirstPositionForType(ItemViewType.ALL_DISMISSED);
+ assertTrue(allDismissedPosition != RecyclerView.NO_POSITION);
scrollToPosition(allDismissedPosition);
View allDismissedView = waitForView(allDismissedPosition);
singleClickView(allDismissedView.findViewById(R.id.action_button));
@@ -178,8 +179,7 @@ public class NewTabPageRecyclerViewTest extends ChromeTabbedActivityTestBase {
assertEquals(10, suggestions.size());
// Scroll a suggestion into view.
- int suggestionPosition =
- getAdapter().getSuggestionPosition(suggestions.get(suggestions.size() - 1));
+ int suggestionPosition = getSuggestionPosition(suggestions.get(suggestions.size() - 1));
scrollToPosition(suggestionPosition);
View suggestionView = waitForView(suggestionPosition);
@@ -246,6 +246,15 @@ public class NewTabPageRecyclerViewTest extends ChromeTabbedActivityTestBase {
return getRecyclerView().getNewTabPageAdapter();
}
+ private int getSuggestionPosition(SnippetArticle article) {
+ NewTabPageAdapter adapter = getAdapter();
+ for (int i = 0; i < adapter.getItemCount(); i++) {
+ SnippetArticle articleToCheck = adapter.getSuggestionAt(i);
+ if (articleToCheck != null && articleToCheck.equals(article)) return i;
+ }
+ return RecyclerView.NO_POSITION;
+ }
+
private void scrollToPosition(final int position) {
final NewTabPageRecyclerView recyclerView = getRecyclerView();

Powered by Google App Engine
This is Rietveld 408576698