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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/UrlManagerTest.java

Issue 1821823003: Add a UrlInfo class for the Physical Web (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add VisibleForTesting to removeUrl Created 4 years, 8 months 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
« no previous file with comments | « chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/MockPwsClient.java ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/UrlManagerTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/UrlManagerTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/UrlManagerTest.java
index 81c399d3a85150b22cae05d2606ab134df944b1a..791be56c39c91905b43b55dcc8e629498896a579 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/UrlManagerTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/UrlManagerTest.java
@@ -16,7 +16,6 @@ import org.chromium.chrome.test.util.browser.notifications.MockNotificationManag
import java.util.ArrayList;
import java.util.Collection;
import java.util.List;
-import java.util.Set;
/**
* Tests for the UrlManager class.
@@ -62,11 +61,11 @@ public class UrlManagerTest extends InstrumentationTestCase {
getInstrumentation().waitForIdleSync();
// Make sure that a resolution was *not* attempted.
- List<Collection<String>> resolveCalls = mMockPwsClient.getResolveCalls();
+ List<Collection<UrlInfo>> resolveCalls = mMockPwsClient.getResolveCalls();
assertEquals(0, resolveCalls.size());
// Make sure that we have no resolved URLs.
- Set<String> urls = mUrlManager.getUrls();
+ List<UrlInfo> urls = mUrlManager.getUrls();
assertEquals(0, urls.size());
// Make sure that a notification was shown.
@@ -81,11 +80,11 @@ public class UrlManagerTest extends InstrumentationTestCase {
getInstrumentation().waitForIdleSync();
// Make sure that a resolution was attempted.
- List<Collection<String>> resolveCalls = mMockPwsClient.getResolveCalls();
+ List<Collection<UrlInfo>> resolveCalls = mMockPwsClient.getResolveCalls();
assertEquals(1, resolveCalls.size());
// Make sure that we have no resolved URLs.
- Set<String> urls = mUrlManager.getUrls();
+ List<UrlInfo> urls = mUrlManager.getUrls();
assertEquals(0, urls.size());
// Make sure that we do have unresolved URLs.
urls = mUrlManager.getUrls(true);
@@ -105,11 +104,11 @@ public class UrlManagerTest extends InstrumentationTestCase {
getInstrumentation().waitForIdleSync();
// Make sure that a resolution was attempted.
- List<Collection<String>> resolveCalls = mMockPwsClient.getResolveCalls();
+ List<Collection<UrlInfo>> resolveCalls = mMockPwsClient.getResolveCalls();
assertEquals(1, resolveCalls.size());
// Make sure that we have our resolved URLs.
- Set<String> urls = mUrlManager.getUrls();
+ List<UrlInfo> urls = mUrlManager.getUrls();
assertEquals(1, urls.size());
// Make sure that a notification was shown.
@@ -132,7 +131,7 @@ public class UrlManagerTest extends InstrumentationTestCase {
mUrlManager.removeUrl(URL1);
// Make sure the URL was removed.
- Set<String> urls = mUrlManager.getUrls(true);
+ List<UrlInfo> urls = mUrlManager.getUrls(true);
assertEquals(0, urls.size());
// Make sure no notification is shown.
@@ -155,7 +154,7 @@ public class UrlManagerTest extends InstrumentationTestCase {
mUrlManager.clearUrls();
// Make sure all URLs were removed.
- Set<String> urls = mUrlManager.getUrls(true);
+ List<UrlInfo> urls = mUrlManager.getUrls(true);
assertEquals(0, urls.size());
// Make sure no notification is shown.
« no previous file with comments | « chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/MockPwsClient.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698