diff options
| author | sergeyu@chromium.org <sergeyu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-02-29 01:19:25 +0000 |
|---|---|---|
| committer | sergeyu@chromium.org <sergeyu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-02-29 01:19:25 +0000 |
| commit | a05c0d82a604eee8c765c450ddc47bf603844642 (patch) | |
| tree | dbbbc2071d8a6610ee9729f3218b1a52243576b5 /remoting/protocol/jingle_session_unittest.cc | |
| parent | 667d3f576698a74703700a8cf0a005883a3ff624 (diff) | |
| download | chromium_src-a05c0d82a604eee8c765c450ddc47bf603844642.zip chromium_src-a05c0d82a604eee8c765c450ddc47bf603844642.tar.gz chromium_src-a05c0d82a604eee8c765c450ddc47bf603844642.tar.bz2 | |
Fix JingleSession to properly set initiator field in session.
Previously session-initiate messages generated by JingleSession had
an empty initiator attribute. Also added tests that verify that
initiator field is set.
Review URL: https://chromiumcodereview.appspot.com/9477010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@124081 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'remoting/protocol/jingle_session_unittest.cc')
| -rw-r--r-- | remoting/protocol/jingle_session_unittest.cc | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/remoting/protocol/jingle_session_unittest.cc b/remoting/protocol/jingle_session_unittest.cc index b5984aa..9361da4 100644 --- a/remoting/protocol/jingle_session_unittest.cc +++ b/remoting/protocol/jingle_session_unittest.cc @@ -312,6 +312,16 @@ TEST_F(JingleSessionTest, RejectConnection) { TEST_F(JingleSessionTest, Connect) { CreateSessionManagers(1, FakeAuthenticator::ACCEPT); InitiateConnection(1, FakeAuthenticator::ACCEPT, false); + + // Verify that the client specified correct initiator value. + ASSERT_GT(host_signal_strategy_->received_messages().size(), 0U); + const buzz::XmlElement* initiate_xml = + host_signal_strategy_->received_messages().front(); + const buzz::XmlElement* jingle_element = + initiate_xml->FirstNamed(buzz::QName(kJingleNamespace, "jingle")); + ASSERT_TRUE(jingle_element); + ASSERT_EQ(kClientJid, + jingle_element->Attr(buzz::QName("", "initiator"))); } // Verify that we can connect two endpoints with multi-step authentication. |