Commit 77c4a104 authored by Wei-Yin Chen (陳威尹)'s avatar Wei-Yin Chen (陳威尹) Committed by Commit Bot

Re-enable tests in PartnerHomepageUnitTest

Bug: 836700, 836110
Change-Id: Id22b0e22d95f3b0429c20799f59a95062a99ac5d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2143691Reviewed-by: default avatarTheresa  <twellington@chromium.org>
Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#758343}
parent 5de5acf5
...@@ -83,12 +83,10 @@ public class PartnerHomepageUnitTest { ...@@ -83,12 +83,10 @@ public class PartnerHomepageUnitTest {
/** /**
* Everything is enabled for using partner homepage, except that there is no flag file. * Everything is enabled for using partner homepage, except that there is no flag file.
* Flaky: crbug.com/836700
*/ */
@Test @Test
@SmallTest @SmallTest
@Feature({"Homepage"}) @Feature({"Homepage"})
@DisabledTest(message = "crbug.com/836700")
public void testProviderNotFromSystemPackage() throws InterruptedException { public void testProviderNotFromSystemPackage() throws InterruptedException {
mHomepageManager.setPrefHomepageEnabled(true); mHomepageManager.setPrefHomepageEnabled(true);
mHomepageManager.setHomepagePreferences(false, true, TEST_CUSTOM_HOMEPAGE_URI); mHomepageManager.setHomepagePreferences(false, true, TEST_CUSTOM_HOMEPAGE_URI);
...@@ -115,12 +113,10 @@ public class PartnerHomepageUnitTest { ...@@ -115,12 +113,10 @@ public class PartnerHomepageUnitTest {
/** /**
* Everything is enabled for using partner homepage, except that there is no actual provider. * Everything is enabled for using partner homepage, except that there is no actual provider.
* Flaky : http://crbug.com/836110
*/ */
@Test @Test
@SmallTest @SmallTest
@Feature({"Homepage"}) @Feature({"Homepage"})
@DisabledTest(message = "crbug.com/836110")
public void testNoProvider() throws InterruptedException { public void testNoProvider() throws InterruptedException {
mHomepageManager.setPrefHomepageEnabled(true); mHomepageManager.setPrefHomepageEnabled(true);
mHomepageManager.setHomepagePreferences(false, true, TEST_CUSTOM_HOMEPAGE_URI); mHomepageManager.setHomepagePreferences(false, true, TEST_CUSTOM_HOMEPAGE_URI);
...@@ -148,7 +144,6 @@ public class PartnerHomepageUnitTest { ...@@ -148,7 +144,6 @@ public class PartnerHomepageUnitTest {
@Test @Test
@SmallTest @SmallTest
@Feature({"Homepage"}) @Feature({"Homepage"})
@RetryOnFailure
public void testHomepageDisabled() throws InterruptedException { public void testHomepageDisabled() throws InterruptedException {
mHomepageManager.setPrefHomepageEnabled(false); mHomepageManager.setPrefHomepageEnabled(false);
mHomepageManager.setHomepagePreferences(false, true, TEST_CUSTOM_HOMEPAGE_URI); mHomepageManager.setHomepagePreferences(false, true, TEST_CUSTOM_HOMEPAGE_URI);
......
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