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

Unified Diff: remoting/test/test_chromoting_client_unittest.cc

Issue 1237093004: Support for connecting to localhost on the chromoting test driver. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Edited merge errors, comments, and optimized code. Created 5 years, 5 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
Index: remoting/test/test_chromoting_client_unittest.cc
diff --git a/remoting/test/test_chromoting_client_unittest.cc b/remoting/test/test_chromoting_client_unittest.cc
index 9f232891905ce0420b1c4a28b8fa71146e2e324c..166f96c61073a8ba49ee35752e7e4451a2be659e 100644
--- a/remoting/test/test_chromoting_client_unittest.cc
+++ b/remoting/test/test_chromoting_client_unittest.cc
@@ -112,8 +112,8 @@ void TestChromotingClientTest::ConnectionReady(bool ready) {
}
TEST_F(TestChromotingClientTest, StartConnectionAndDisconnect) {
- test_chromoting_client_->StartConnection(kTestUserName, kAccessToken,
- remote_host_info_);
+ test_chromoting_client_->StartConnection(
+ remote_host_info_.GenerateConnectionSetupInfo(kAccessToken, kTestUserName));
EXPECT_EQ(protocol::ConnectionToHost::State::CONNECTING, connection_state_);
EXPECT_EQ(protocol::OK, error_code_);
EXPECT_FALSE(is_connected_to_host_);
@@ -146,8 +146,8 @@ TEST_F(TestChromotingClientTest, StartConnectionAndDisconnect) {
TEST_F(TestChromotingClientTest,
StartConnectionThenFailWithAuthenticationError) {
- test_chromoting_client_->StartConnection(kTestUserName, kAccessToken,
- remote_host_info_);
+ test_chromoting_client_->StartConnection(
+ remote_host_info_.GenerateConnectionSetupInfo(kAccessToken, kTestUserName));
EXPECT_EQ(protocol::ConnectionToHost::State::CONNECTING, connection_state_);
EXPECT_EQ(protocol::OK, error_code_);
EXPECT_FALSE(is_connected_to_host_);
@@ -167,8 +167,8 @@ TEST_F(TestChromotingClientTest,
}
TEST_F(TestChromotingClientTest, StartConnectionThenFailWithUnknownError) {
- test_chromoting_client_->StartConnection(kTestUserName, kAccessToken,
- remote_host_info_);
+ test_chromoting_client_->StartConnection(
+ remote_host_info_.GenerateConnectionSetupInfo(kAccessToken, kTestUserName));
EXPECT_EQ(protocol::ConnectionToHost::State::CONNECTING, connection_state_);
EXPECT_EQ(protocol::OK, error_code_);
EXPECT_FALSE(is_connected_to_host_);
« remoting/test/test_chromoting_client.cc ('K') | « remoting/test/test_chromoting_client.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698