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

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

Issue 2557593003: Store device addresses in UrlInfo (Closed)
Patch Set: Created 4 years 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/UrlInfoTest.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 39102a08d1becee1ca783616aaed5d1308f91214..cb1be3fa129c66b95cb2257725f2d6b2feb9dc29 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
@@ -263,6 +263,30 @@ public class UrlManagerTest extends InstrumentationTestCase {
}
@SmallTest
+ public void testAddUrlUpdatesCache() throws Exception {
+ addEmptyPwsResult();
+ addEmptyPwsResult();
+
+ UrlInfo urlInfo = new UrlInfo(URL1);
+ mUrlManager.addUrl(urlInfo);
+ List<UrlInfo> urls = mUrlManager.getUrls(true);
+ assertEquals(1, urls.size());
+ assertEquals(urlInfo.getDistance(), urls.get(0).getDistance());
+ assertEquals(urlInfo.getDeviceAddress(), urls.get(0).getDeviceAddress());
+ assertEquals(urlInfo.getScanTimestamp(), urls.get(0).getScanTimestamp());
+
+ urlInfo = new UrlInfo(URL1)
+ .setDistance(100.0)
+ .setDeviceAddress("00:11:22:33:AA:BB");
+ mUrlManager.addUrl(urlInfo);
+ urls = mUrlManager.getUrls(true);
+ assertEquals(1, urls.size());
+ assertEquals(urlInfo.getDistance(), urls.get(0).getDistance());
+ assertEquals(urlInfo.getDeviceAddress(), urls.get(0).getDeviceAddress());
+ assertEquals(urlInfo.getScanTimestamp(), urls.get(0).getScanTimestamp());
+ }
+
+ @SmallTest
@RetryOnFailure
public void testAddUrlTwiceWorks() throws Exception {
// Add and remove an old URL twice and add new URL twice before removing.
« no previous file with comments | « chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/UrlInfoTest.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698