Commit 443bd15a authored by huapengl's avatar huapengl Committed by Commit bot

Add Feature tag to all ImportantSite tests

This will allow users to exclusively run or ignore
all ImportantSite tests at once.

BUG=None

Review-Url: https://codereview.chromium.org/2392303006
Cr-Commit-Position: refs/heads/master@{#423876}
parent d1c2a9ac
...@@ -19,6 +19,7 @@ import android.widget.ListView; ...@@ -19,6 +19,7 @@ import android.widget.ListView;
import org.chromium.base.ThreadUtils; import org.chromium.base.ThreadUtils;
import org.chromium.base.test.util.CommandLineFlags; import org.chromium.base.test.util.CommandLineFlags;
import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.RetryOnFailure; import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.chrome.browser.ChromeActivity; import org.chromium.chrome.browser.ChromeActivity;
import org.chromium.chrome.browser.ChromeFeatureList; import org.chromium.chrome.browser.ChromeFeatureList;
...@@ -524,6 +525,7 @@ public class ClearBrowsingDataPreferencesTest ...@@ -524,6 +525,7 @@ public class ClearBrowsingDataPreferencesTest
*/ */
@CommandLineFlags.Add({"enable-features=ImportantSitesInCBD", "enable-site-engagement"}) @CommandLineFlags.Add({"enable-features=ImportantSitesInCBD", "enable-site-engagement"})
@MediumTest @MediumTest
@Feature({"SiteEngagement"})
public void testImportantSitesDialogNoFiltering() throws Exception { public void testImportantSitesDialogNoFiltering() throws Exception {
// Sign in. // Sign in.
SigninTestUtil.addAndSignInTestAccount(); SigninTestUtil.addAndSignInTestAccount();
...@@ -570,6 +572,7 @@ public class ClearBrowsingDataPreferencesTest ...@@ -570,6 +572,7 @@ public class ClearBrowsingDataPreferencesTest
*/ */
@CommandLineFlags.Add({"enable-features=ImportantSitesInCBD", "enable-site-engagement"}) @CommandLineFlags.Add({"enable-features=ImportantSitesInCBD", "enable-site-engagement"})
@MediumTest @MediumTest
@Feature({"SiteEngagement"})
public void testImportantSitesDialogNoopOnCancel() throws Exception { public void testImportantSitesDialogNoopOnCancel() throws Exception {
// Sign in. // Sign in.
SigninTestUtil.addAndSignInTestAccount(); SigninTestUtil.addAndSignInTestAccount();
...@@ -608,6 +611,7 @@ public class ClearBrowsingDataPreferencesTest ...@@ -608,6 +611,7 @@ public class ClearBrowsingDataPreferencesTest
*/ */
@CommandLineFlags.Add({"enable-features=ImportantSitesInCBD", "enable-site-engagement"}) @CommandLineFlags.Add({"enable-features=ImportantSitesInCBD", "enable-site-engagement"})
@MediumTest @MediumTest
@Feature({"SiteEngagement"})
public void testImportantSitesDialog() throws Exception { public void testImportantSitesDialog() throws Exception {
// Sign in. // Sign in.
SigninTestUtil.addAndSignInTestAccount(); SigninTestUtil.addAndSignInTestAccount();
......
...@@ -14,6 +14,7 @@ import android.test.suitebuilder.annotation.SmallTest; ...@@ -14,6 +14,7 @@ import android.test.suitebuilder.annotation.SmallTest;
import org.chromium.base.ThreadUtils; import org.chromium.base.ThreadUtils;
import org.chromium.base.test.util.CommandLineFlags; import org.chromium.base.test.util.CommandLineFlags;
import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.MinAndroidSdkLevel; import org.chromium.base.test.util.MinAndroidSdkLevel;
import org.chromium.base.test.util.RetryOnFailure; import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.chrome.browser.ChromeActivity; import org.chromium.chrome.browser.ChromeActivity;
...@@ -109,6 +110,7 @@ public class ManageSpaceActivityTest extends ChromeActivityTestCaseBase<ChromeAc ...@@ -109,6 +110,7 @@ public class ManageSpaceActivityTest extends ChromeActivityTestCaseBase<ChromeAc
@MediumTest @MediumTest
@RetryOnFailure @RetryOnFailure
@Feature({"SiteEngagement"})
public void testClearUnimportantOnly() throws Exception { public void testClearUnimportantOnly() throws Exception {
final String cookiesUrl = final String cookiesUrl =
mTestServer.getURL("/chrome/test/data/android/storage_persistance.html"); mTestServer.getURL("/chrome/test/data/android/storage_persistance.html");
...@@ -144,6 +146,7 @@ public class ManageSpaceActivityTest extends ChromeActivityTestCaseBase<ChromeAc ...@@ -144,6 +146,7 @@ public class ManageSpaceActivityTest extends ChromeActivityTestCaseBase<ChromeAc
} }
@MediumTest @MediumTest
@Feature({"SiteEngagement"})
public void testClearUnimporantWithoutChromeStart() throws Exception { public void testClearUnimporantWithoutChromeStart() throws Exception {
ManageSpaceActivity manageSpaceActivity = startManageSpaceActivity(); ManageSpaceActivity manageSpaceActivity = startManageSpaceActivity();
// Click 'clear' in the CBD screen. // Click 'clear' in the CBD screen.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment