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: components/cloud_devices/common/cloud_devices_urls_unittest.cc

Issue 258623004: Fixed url to manage specific cloud printers. (Closed) Base URL: http://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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 | « components/cloud_devices/common/cloud_devices_urls.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/cloud_devices/common/cloud_devices_urls_unittest.cc
diff --git a/components/cloud_devices/common/cloud_devices_urls_unittest.cc b/components/cloud_devices/common/cloud_devices_urls_unittest.cc
index 4a285c6b56cda1c664b3cc7f69e5bdb11d627e84..bd4da91522236c47f2c3ed7b98cd29155cb3c3ea 100644
--- a/components/cloud_devices/common/cloud_devices_urls_unittest.cc
+++ b/components/cloud_devices/common/cloud_devices_urls_unittest.cc
@@ -53,8 +53,8 @@ TEST(CloudPrintURLTest, GetCloudPrintEnableURL) {
EXPECT_THAT(enable_url, HasSubstr("/cloudprint/"));
}
-TEST(CloudPrintURLTest, GetCloudPrintEnableURLWithSignin) {
- std::string enable_url = GetCloudPrintEnableURLWithSignin("123123").spec();
+TEST(CloudPrintURLTest, GetCloudPrintEnableWithSigninURL) {
+ std::string enable_url = GetCloudPrintEnableWithSigninURL("123123").spec();
EXPECT_THAT(enable_url, HasSubstr("accounts.google.com"));
EXPECT_THAT(enable_url, HasSubstr("/ServiceLogin"));
EXPECT_THAT(enable_url, HasSubstr("service=cloudprint"));
@@ -64,6 +64,13 @@ TEST(CloudPrintURLTest, GetCloudPrintEnableURLWithSignin) {
EXPECT_THAT(enable_url, HasSubstr("%2Fcloudprint%2F"));
}
+TEST(CloudPrintURLTest, GetCloudPrintManageDeviceURL) {
+ std::string manage_url = GetCloudPrintManageDeviceURL("123").spec();
+ EXPECT_THAT(manage_url, HasSubstr("www.google.com"));
+ EXPECT_THAT(manage_url, HasSubstr("/cloudprint"));
+ EXPECT_THAT(manage_url, HasSubstr("#printers/123"));
+}
+
TEST(CloudPrintURLTest, GetCloudPrintSigninURL) {
std::string signin_url = GetCloudPrintSigninURL().spec();
EXPECT_THAT(signin_url, HasSubstr("accounts.google.com"));
« no previous file with comments | « components/cloud_devices/common/cloud_devices_urls.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698