Commit 92e87bfc authored by Peter Kotwicz's avatar Peter Kotwicz Committed by Commit Bot

[Android] Make WebappInfoTests JUnit tests

BUG=None

Change-Id: I640ee75a03a6b9bce6a5200b50b2d3baad47e482
Reviewed-on: https://chromium-review.googlesource.com/574568Reviewed-by: default avatarYaron Friedman <yfriedman@chromium.org>
Commit-Queue: Yaron Friedman <yfriedman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487099}
parent 6b22c715
...@@ -1680,7 +1680,6 @@ chrome_test_java_sources = [ ...@@ -1680,7 +1680,6 @@ chrome_test_java_sources = [
"javatests/src/org/chromium/chrome/browser/webapps/WebappAuthenticatorTest.java", "javatests/src/org/chromium/chrome/browser/webapps/WebappAuthenticatorTest.java",
"javatests/src/org/chromium/chrome/browser/webapps/WebappDeferredStartupTest.java", "javatests/src/org/chromium/chrome/browser/webapps/WebappDeferredStartupTest.java",
"javatests/src/org/chromium/chrome/browser/webapps/WebappDirectoryManagerTest.java", "javatests/src/org/chromium/chrome/browser/webapps/WebappDirectoryManagerTest.java",
"javatests/src/org/chromium/chrome/browser/webapps/WebappInfoTest.java",
"javatests/src/org/chromium/chrome/browser/webapps/WebappModeTest.java", "javatests/src/org/chromium/chrome/browser/webapps/WebappModeTest.java",
"javatests/src/org/chromium/chrome/browser/webapps/WebappNavigationTest.java", "javatests/src/org/chromium/chrome/browser/webapps/WebappNavigationTest.java",
"javatests/src/org/chromium/chrome/browser/webapps/WebappSplashScreenBackgroundColorTest.java", "javatests/src/org/chromium/chrome/browser/webapps/WebappSplashScreenBackgroundColorTest.java",
...@@ -1814,6 +1813,7 @@ chrome_junit_test_java_sources = [ ...@@ -1814,6 +1813,7 @@ chrome_junit_test_java_sources = [
"junit/src/org/chromium/chrome/browser/tabstate/TabStateUnitTest.java", "junit/src/org/chromium/chrome/browser/tabstate/TabStateUnitTest.java",
"junit/src/org/chromium/chrome/browser/webapps/WebappDataStorageTest.java", "junit/src/org/chromium/chrome/browser/webapps/WebappDataStorageTest.java",
"junit/src/org/chromium/chrome/browser/webapps/WebappRegistryTest.java", "junit/src/org/chromium/chrome/browser/webapps/WebappRegistryTest.java",
"junit/src/org/chromium/chrome/browser/webapps/WebappInfoTest.java",
"junit/src/org/chromium/chrome/browser/webapps/WebApkInfoTest.java", "junit/src/org/chromium/chrome/browser/webapps/WebApkInfoTest.java",
"junit/src/org/chromium/chrome/browser/webapps/WebApkUpdateManagerTest.java", "junit/src/org/chromium/chrome/browser/webapps/WebApkUpdateManagerTest.java",
"junit/src/org/chromium/chrome/browser/widget/selection/SelectionDelegateTest.java", "junit/src/org/chromium/chrome/browser/widget/selection/SelectionDelegateTest.java",
......
...@@ -5,27 +5,25 @@ ...@@ -5,27 +5,25 @@
package org.chromium.chrome.browser.webapps; package org.chromium.chrome.browser.webapps;
import android.content.Intent; import android.content.Intent;
import android.support.test.filters.SmallTest;
import org.junit.Assert; import org.junit.Assert;
import org.junit.Test; import org.junit.Test;
import org.junit.runner.RunWith; import org.junit.runner.RunWith;
import org.robolectric.annotation.Config;
import org.chromium.base.test.util.Feature;
import org.chromium.blink_public.platform.WebDisplayMode; import org.chromium.blink_public.platform.WebDisplayMode;
import org.chromium.chrome.browser.ShortcutHelper; import org.chromium.chrome.browser.ShortcutHelper;
import org.chromium.chrome.browser.ShortcutSource; import org.chromium.chrome.browser.ShortcutSource;
import org.chromium.chrome.test.ChromeJUnit4ClassRunner;
import org.chromium.content_public.common.ScreenOrientationValues; import org.chromium.content_public.common.ScreenOrientationValues;
import org.chromium.testing.local.LocalRobolectricTestRunner;
/** /**
* Tests the WebappInfo class's ability to parse various URLs. * Tests the WebappInfo class's ability to parse various URLs.
*/ */
@RunWith(ChromeJUnit4ClassRunner.class) @RunWith(LocalRobolectricTestRunner.class)
@Config(manifest = Config.NONE)
public class WebappInfoTest { public class WebappInfoTest {
@Test @Test
@SmallTest
@Feature({"Webapps"})
public void testAbout() { public void testAbout() {
String id = "webapp id"; String id = "webapp id";
String name = "longName"; String name = "longName";
...@@ -40,8 +38,6 @@ public class WebappInfoTest { ...@@ -40,8 +38,6 @@ public class WebappInfoTest {
} }
@Test @Test
@SmallTest
@Feature({"Webapps"})
public void testRandomUrl() { public void testRandomUrl() {
String id = "webapp id"; String id = "webapp id";
String name = "longName"; String name = "longName";
...@@ -56,8 +52,6 @@ public class WebappInfoTest { ...@@ -56,8 +52,6 @@ public class WebappInfoTest {
} }
@Test @Test
@SmallTest
@Feature({"Webapps"})
public void testSpacesInUrl() { public void testSpacesInUrl() {
String id = "webapp id"; String id = "webapp id";
String name = "longName"; String name = "longName";
...@@ -75,8 +69,6 @@ public class WebappInfoTest { ...@@ -75,8 +69,6 @@ public class WebappInfoTest {
} }
@Test @Test
@SmallTest
@Feature({"Webapps"})
public void testIntentTitleFallBack() { public void testIntentTitleFallBack() {
String title = "webapp title"; String title = "webapp title";
...@@ -89,8 +81,6 @@ public class WebappInfoTest { ...@@ -89,8 +81,6 @@ public class WebappInfoTest {
} }
@Test @Test
@SmallTest
@Feature({"Webapps"})
public void testIntentNameBlankNoTitle() { public void testIntentNameBlankNoTitle() {
String shortName = "name"; String shortName = "name";
...@@ -103,8 +93,6 @@ public class WebappInfoTest { ...@@ -103,8 +93,6 @@ public class WebappInfoTest {
} }
@Test @Test
@SmallTest
@Feature({"Webapps"})
public void testIntentShortNameFallBack() { public void testIntentShortNameFallBack() {
String title = "webapp title"; String title = "webapp title";
String shortName = "name"; String shortName = "name";
...@@ -119,8 +107,6 @@ public class WebappInfoTest { ...@@ -119,8 +107,6 @@ public class WebappInfoTest {
} }
@Test @Test
@SmallTest
@Feature({"Webapps"})
public void testIntentNameShortname() { public void testIntentNameShortname() {
String name = "longName"; String name = "longName";
String shortName = "name"; String shortName = "name";
...@@ -135,8 +121,6 @@ public class WebappInfoTest { ...@@ -135,8 +121,6 @@ public class WebappInfoTest {
} }
@Test @Test
@SmallTest
@Feature({"Webapps"})
public void testDisplayModeAndOrientationAndSource() { public void testDisplayModeAndOrientationAndSource() {
String id = "webapp id"; String id = "webapp id";
String name = "longName"; String name = "longName";
...@@ -153,8 +137,6 @@ public class WebappInfoTest { ...@@ -153,8 +137,6 @@ public class WebappInfoTest {
} }
@Test @Test
@SmallTest
@Feature({"Webapps"})
public void testNormalColors() { public void testNormalColors() {
String id = "webapp id"; String id = "webapp id";
String name = "longName"; String name = "longName";
...@@ -171,8 +153,6 @@ public class WebappInfoTest { ...@@ -171,8 +153,6 @@ public class WebappInfoTest {
} }
@Test @Test
@SmallTest
@Feature({"Webapps"})
public void testInvalidOrMissingColors() { public void testInvalidOrMissingColors() {
String id = "webapp id"; String id = "webapp id";
String name = "longName"; String name = "longName";
...@@ -189,8 +169,6 @@ public class WebappInfoTest { ...@@ -189,8 +169,6 @@ public class WebappInfoTest {
} }
@Test @Test
@SmallTest
@Feature({"Webapps"})
public void testColorsIntentCreation() { public void testColorsIntentCreation() {
long themeColor = 0xFF00FF00L; long themeColor = 0xFF00FF00L;
long backgroundColor = 0xFF0000FFL; long backgroundColor = 0xFF0000FFL;
...@@ -205,8 +183,6 @@ public class WebappInfoTest { ...@@ -205,8 +183,6 @@ public class WebappInfoTest {
} }
@Test @Test
@SmallTest
@Feature({"Webapps"})
public void testScopeIntentCreation() { public void testScopeIntentCreation() {
String scope = "https://www.foo.com"; String scope = "https://www.foo.com";
Intent intent = createIntentWithUrlAndId(); Intent intent = createIntentWithUrlAndId();
...@@ -216,8 +192,6 @@ public class WebappInfoTest { ...@@ -216,8 +192,6 @@ public class WebappInfoTest {
} }
@Test @Test
@SmallTest
@Feature({"Webapps"})
public void testIntentScopeFallback() { public void testIntentScopeFallback() {
String url = "https://www.foo.com/homepage.html"; String url = "https://www.foo.com/homepage.html";
Intent intent = createIntentWithUrlAndId(); Intent intent = createIntentWithUrlAndId();
...@@ -227,8 +201,6 @@ public class WebappInfoTest { ...@@ -227,8 +201,6 @@ public class WebappInfoTest {
} }
@Test @Test
@SmallTest
@Feature({"Webapps"})
public void testIntentDisplayMode() { public void testIntentDisplayMode() {
Intent intent = createIntentWithUrlAndId(); Intent intent = createIntentWithUrlAndId();
intent.putExtra(ShortcutHelper.EXTRA_DISPLAY_MODE, WebDisplayMode.MINIMAL_UI); intent.putExtra(ShortcutHelper.EXTRA_DISPLAY_MODE, WebDisplayMode.MINIMAL_UI);
...@@ -237,8 +209,6 @@ public class WebappInfoTest { ...@@ -237,8 +209,6 @@ public class WebappInfoTest {
} }
@Test @Test
@SmallTest
@Feature({"Webapps"})
public void testIntentOrientation() { public void testIntentOrientation() {
Intent intent = createIntentWithUrlAndId(); Intent intent = createIntentWithUrlAndId();
intent.putExtra(ShortcutHelper.EXTRA_ORIENTATION, ScreenOrientationValues.LANDSCAPE); intent.putExtra(ShortcutHelper.EXTRA_ORIENTATION, ScreenOrientationValues.LANDSCAPE);
...@@ -247,8 +217,6 @@ public class WebappInfoTest { ...@@ -247,8 +217,6 @@ public class WebappInfoTest {
} }
@Test @Test
@SmallTest
@Feature({"Webapps"})
public void testIntentGeneratedIcon() { public void testIntentGeneratedIcon() {
String id = "webapp id"; String id = "webapp id";
String name = "longName"; String name = "longName";
......
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