| Index: chrome/android/javatests/src/org/chromium/chrome/browser/ntp/snippets/ArticleSnippetsTest.java
|
| diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/ntp/snippets/ArticleSnippetsTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/ntp/snippets/ArticleSnippetsTest.java
|
| index b4c346da828b43b04091726f72d4813bc192bb77..0943a21ddcb6e3d9745f8231b346ed5f516a27e4 100644
|
| --- a/chrome/android/javatests/src/org/chromium/chrome/browser/ntp/snippets/ArticleSnippetsTest.java
|
| +++ b/chrome/android/javatests/src/org/chromium/chrome/browser/ntp/snippets/ArticleSnippetsTest.java
|
| @@ -5,16 +5,24 @@
|
| package org.chromium.chrome.browser.ntp.snippets;
|
|
|
| import android.graphics.BitmapFactory;
|
| +import android.support.test.InstrumentationRegistry;
|
| import android.support.test.filters.MediumTest;
|
| import android.util.TypedValue;
|
| import android.view.ViewGroup;
|
| import android.widget.FrameLayout;
|
|
|
| +import org.junit.Before;
|
| +import org.junit.Rule;
|
| +import org.junit.Test;
|
| +import org.junit.runner.RunWith;
|
| +
|
| import org.chromium.base.ThreadUtils;
|
| +import org.chromium.base.test.util.CommandLineFlags;
|
| import org.chromium.base.test.util.Feature;
|
| import org.chromium.base.test.util.RetryOnFailure;
|
| import org.chromium.chrome.R;
|
| import org.chromium.chrome.browser.ChromeActivity;
|
| +import org.chromium.chrome.browser.ChromeSwitches;
|
| import org.chromium.chrome.browser.favicon.FaviconHelper.FaviconImageCallback;
|
| import org.chromium.chrome.browser.favicon.FaviconHelper.IconAvailabilityCallback;
|
| import org.chromium.chrome.browser.favicon.LargeIconBridge.LargeIconCallback;
|
| @@ -31,7 +39,8 @@ import org.chromium.chrome.browser.suggestions.SuggestionsUiDelegate;
|
| import org.chromium.chrome.browser.widget.displaystyle.HorizontalDisplayStyle;
|
| import org.chromium.chrome.browser.widget.displaystyle.UiConfig;
|
| import org.chromium.chrome.browser.widget.displaystyle.VerticalDisplayStyle;
|
| -import org.chromium.chrome.test.ChromeActivityTestCaseBase;
|
| +import org.chromium.chrome.test.ChromeActivityTestRule;
|
| +import org.chromium.chrome.test.ChromeJUnit4ClassRunner;
|
| import org.chromium.chrome.test.util.RenderUtils.ViewRenderer;
|
| import org.chromium.chrome.test.util.browser.suggestions.DummySuggestionsMetricsReporter;
|
| import org.chromium.chrome.test.util.browser.suggestions.FakeSuggestionsSource;
|
| @@ -42,7 +51,14 @@ import java.util.Arrays;
|
| /**
|
| * Tests for the appearance of Article Snippets.
|
| */
|
| -public class ArticleSnippetsTest extends ChromeActivityTestCaseBase<ChromeActivity> {
|
| +@RunWith(ChromeJUnit4ClassRunner.class)
|
| +@CommandLineFlags.Add({ChromeSwitches.DISABLE_FIRST_RUN_EXPERIENCE,
|
| + ChromeActivityTestRule.DISABLE_NETWORK_PREDICTION_FLAG})
|
| +public class ArticleSnippetsTest {
|
| + @Rule
|
| + public ChromeActivityTestRule<ChromeActivity> mActivityTestRule =
|
| + new ChromeActivityTestRule<>(ChromeActivity.class);
|
| +
|
| private ViewRenderer mViewRenderer;
|
|
|
| private SuggestionsUiDelegate mUiDelegate;
|
| @@ -53,10 +69,7 @@ public class ArticleSnippetsTest extends ChromeActivityTestCaseBase<ChromeActivi
|
| private FrameLayout mContentView;
|
| private UiConfig mUiConfig;
|
|
|
| - public ArticleSnippetsTest() {
|
| - super(ChromeActivity.class);
|
| - }
|
| -
|
| + @Test
|
| @MediumTest
|
| @Feature({"ArticleSnippets", "RenderTest"})
|
| @RetryOnFailure
|
| @@ -66,12 +79,12 @@ public class ArticleSnippetsTest extends ChromeActivityTestCaseBase<ChromeActivi
|
| public void run() {
|
| setupTestData();
|
|
|
| - mContentView = new FrameLayout(getActivity());
|
| + mContentView = new FrameLayout(mActivityTestRule.getActivity());
|
| mUiConfig = new UiConfig(mContentView);
|
|
|
| - getActivity().setContentView(mContentView);
|
| + mActivityTestRule.getActivity().setContentView(mContentView);
|
|
|
| - mRecyclerView = new SuggestionsRecyclerView(getActivity());
|
| + mRecyclerView = new SuggestionsRecyclerView(mActivityTestRule.getActivity());
|
| mContentView.addView(mRecyclerView);
|
|
|
| mAdapter = new NewTabPageAdapter(mUiDelegate, /* aboveTheFold = */ null, mUiConfig,
|
| @@ -82,7 +95,7 @@ public class ArticleSnippetsTest extends ChromeActivityTestCaseBase<ChromeActivi
|
| }
|
| });
|
|
|
| - getInstrumentation().waitForIdleSync();
|
| + InstrumentationRegistry.getInstrumentation().waitForIdleSync();
|
|
|
| int first = mAdapter.getFirstCardPosition();
|
| mViewRenderer.renderAndCompare(mRecyclerView.getChildAt(first), "short_snippet");
|
| @@ -111,7 +124,7 @@ public class ArticleSnippetsTest extends ChromeActivityTestCaseBase<ChromeActivi
|
| }
|
| });
|
|
|
| - getInstrumentation().waitForIdleSync();
|
| + InstrumentationRegistry.getInstrumentation().waitForIdleSync();
|
|
|
| mViewRenderer.renderAndCompare(mRecyclerView.getChildAt(first), "short_snippet_narrow");
|
| mViewRenderer.renderAndCompare(mRecyclerView.getChildAt(first + 1), "long_snippet_narrow");
|
| @@ -132,8 +145,9 @@ public class ArticleSnippetsTest extends ChromeActivityTestCaseBase<ChromeActivi
|
| 1466614774, // Publish timestamp
|
| 10f, // Score
|
| 1466634774); // Fetch timestamp
|
| - shortSnippet.setThumbnailBitmap(BitmapFactory.decodeResource(getActivity().getResources(),
|
| - R.drawable.signin_promo_illustration));
|
| + shortSnippet.setThumbnailBitmap(
|
| + BitmapFactory.decodeResource(mActivityTestRule.getActivity().getResources(),
|
| + R.drawable.signin_promo_illustration));
|
|
|
| SnippetArticle longSnippet = new SnippetArticle(fullCategory, "id2",
|
| new String(new char[20]).replace("\0", "Snippet "),
|
| @@ -175,16 +189,11 @@ public class ArticleSnippetsTest extends ChromeActivityTestCaseBase<ChromeActivi
|
| minimalCategory, Arrays.asList(minimalSnippet, minimalSnippet2));
|
| }
|
|
|
| - @Override
|
| - public void startMainActivity() throws InterruptedException {
|
| - startMainActivityOnBlankPage();
|
| - mViewRenderer = new ViewRenderer(getActivity(),
|
| + @Before
|
| + public void setUp() throws Exception {
|
| + mActivityTestRule.startMainActivityOnBlankPage();
|
| + mViewRenderer = new ViewRenderer(mActivityTestRule.getActivity(),
|
| "chrome/test/data/android/render_tests", this.getClass().getSimpleName());
|
| - }
|
| -
|
| - @Override
|
| - protected void setUp() throws Exception {
|
| - super.setUp();
|
| mUiDelegate = new MockUiDelegate();
|
| mSnippetsSource = new FakeSuggestionsSource();
|
| }
|
| @@ -205,8 +214,9 @@ public class ArticleSnippetsTest extends ChromeActivityTestCaseBase<ChromeActivi
|
| public void run() {
|
| // Return an arbitrary drawable.
|
| faviconCallback.onFaviconAvailable(
|
| - BitmapFactory.decodeResource(getActivity().getResources(),
|
| - R.drawable.star_green),
|
| + BitmapFactory.decodeResource(
|
| + mActivityTestRule.getActivity().getResources(),
|
| + R.drawable.star_green),
|
| url);
|
| }
|
| });
|
|
|