From 893505d0fb41a840be5e4a44a1250dba83d79bf5 Mon Sep 17 00:00:00 2001 From: Taylor Brandstetter Date: Thu, 7 Jan 2016 15:12:48 -0800 Subject: [PATCH] Adding unit test to ensure TURN server priorities are unique. BUG=webrtc:5209 R=pthatcher@webrtc.org Review URL: https://codereview.webrtc.org/1570563002 . Cr-Commit-Position: refs/heads/master@{#11177} --- talk/app/webrtc/peerconnection_unittest.cc | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/talk/app/webrtc/peerconnection_unittest.cc b/talk/app/webrtc/peerconnection_unittest.cc index a2780a36e1..8d0793e25f 100644 --- a/talk/app/webrtc/peerconnection_unittest.cc +++ b/talk/app/webrtc/peerconnection_unittest.cc @@ -2022,4 +2022,17 @@ TEST_F(IceServerParsingTest, ParseMultipleUrls) { EXPECT_EQ(1U, turn_servers_.size()); } +// Ensure that TURN servers are given unique priorities, +// so that their resulting candidates have unique priorities. +TEST_F(IceServerParsingTest, TurnServerPrioritiesUnique) { + PeerConnectionInterface::IceServers servers; + PeerConnectionInterface::IceServer server; + server.urls.push_back("turn:hostname"); + server.urls.push_back("turn:hostname2"); + servers.push_back(server); + EXPECT_TRUE(webrtc::ParseIceServers(servers, &stun_servers_, &turn_servers_)); + EXPECT_EQ(2U, turn_servers_.size()); + EXPECT_NE(turn_servers_[0].priority, turn_servers_[1].priority); +} + #endif // if !defined(THREAD_SANITIZER)