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

Side by Side Diff: webrtc/modules/rtp_rtcp/source/rtcp_packet/extended_jitter_report_unittest.cc

Issue 2348623003: Unify rtcp packet setters (Closed)
Patch Set: +call/rtc_event_log_unittest Created 4 years, 2 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 13 matching lines...) Expand all
24 constexpr uint32_t kJitter2 = 0x22242628; 24 constexpr uint32_t kJitter2 = 0x22242628;
25 } // namespace 25 } // namespace
26 26
27 TEST(RtcpPacketExtendedJitterReportTest, CreateAndParseWithoutItems) { 27 TEST(RtcpPacketExtendedJitterReportTest, CreateAndParseWithoutItems) {
28 ExtendedJitterReport ij; 28 ExtendedJitterReport ij;
29 rtc::Buffer raw = ij.Build(); 29 rtc::Buffer raw = ij.Build();
30 30
31 ExtendedJitterReport parsed; 31 ExtendedJitterReport parsed;
32 EXPECT_TRUE(test::ParseSinglePacket(raw, &parsed)); 32 EXPECT_TRUE(test::ParseSinglePacket(raw, &parsed));
33 33
34 EXPECT_THAT(parsed.jitters(), IsEmpty()); 34 EXPECT_THAT(parsed.jitter_values(), IsEmpty());
35 } 35 }
36 36
37 TEST(RtcpPacketExtendedJitterReportTest, CreateAndParseWithOneItem) { 37 TEST(RtcpPacketExtendedJitterReportTest, CreateAndParseWithOneItem) {
38 ExtendedJitterReport ij; 38 ExtendedJitterReport ij;
39 EXPECT_TRUE(ij.WithJitter(kJitter1)); 39 EXPECT_TRUE(ij.SetJitterValues({kJitter1}));
40 rtc::Buffer raw = ij.Build(); 40 rtc::Buffer raw = ij.Build();
41 41
42 ExtendedJitterReport parsed; 42 ExtendedJitterReport parsed;
43 EXPECT_TRUE(test::ParseSinglePacket(raw, &parsed)); 43 EXPECT_TRUE(test::ParseSinglePacket(raw, &parsed));
44 44
45 EXPECT_THAT(parsed.jitters(), ElementsAre(kJitter1)); 45 EXPECT_THAT(parsed.jitter_values(), ElementsAre(kJitter1));
46 } 46 }
47 47
48 TEST(RtcpPacketExtendedJitterReportTest, CreateAndParseWithTwoItems) { 48 TEST(RtcpPacketExtendedJitterReportTest, CreateAndParseWithTwoItems) {
49 ExtendedJitterReport ij; 49 ExtendedJitterReport ij;
50 EXPECT_TRUE(ij.WithJitter(kJitter1)); 50 EXPECT_TRUE(ij.SetJitterValues({kJitter1, kJitter2}));
51 EXPECT_TRUE(ij.WithJitter(kJitter2));
52 rtc::Buffer raw = ij.Build(); 51 rtc::Buffer raw = ij.Build();
53 52
54 ExtendedJitterReport parsed; 53 ExtendedJitterReport parsed;
55 EXPECT_TRUE(test::ParseSinglePacket(raw, &parsed)); 54 EXPECT_TRUE(test::ParseSinglePacket(raw, &parsed));
56 55
57 EXPECT_THAT(parsed.jitters(), ElementsAre(kJitter1, kJitter2)); 56 EXPECT_THAT(parsed.jitter_values(), ElementsAre(kJitter1, kJitter2));
58 } 57 }
59 58
60 TEST(RtcpPacketExtendedJitterReportTest, CreateWithTooManyItems) { 59 TEST(RtcpPacketExtendedJitterReportTest, CreateWithTooManyItems) {
61 ExtendedJitterReport ij; 60 ExtendedJitterReport ij;
62 const int kMaxIjItems = (1 << 5) - 1; 61 const int kMaxItems = ExtendedJitterReport::kMaxNumberOfJitterValues;
63 for (int i = 0; i < kMaxIjItems; ++i) { 62 EXPECT_FALSE(
64 EXPECT_TRUE(ij.WithJitter(i)); 63 ij.SetJitterValues(std::vector<uint32_t>(kMaxItems + 1, kJitter1)));
65 } 64 EXPECT_TRUE(ij.SetJitterValues(std::vector<uint32_t>(kMaxItems, kJitter1)));
66 EXPECT_FALSE(ij.WithJitter(kMaxIjItems));
67 } 65 }
68 66
69 TEST(RtcpPacketExtendedJitterReportTest, ParseFailsWithTooManyItems) { 67 TEST(RtcpPacketExtendedJitterReportTest, ParseFailsWithTooManyItems) {
70 ExtendedJitterReport ij; 68 ExtendedJitterReport ij;
71 ij.WithJitter(kJitter1); 69 ij.SetJitterValues({kJitter1});
72 rtc::Buffer raw = ij.Build(); 70 rtc::Buffer raw = ij.Build();
73 raw[0]++; // Damage packet: increase jitter count by 1. 71 raw[0]++; // Damage packet: increase jitter count by 1.
74 ExtendedJitterReport parsed; 72 ExtendedJitterReport parsed;
75 EXPECT_FALSE(test::ParseSinglePacket(raw, &parsed)); 73 EXPECT_FALSE(test::ParseSinglePacket(raw, &parsed));
76 } 74 }
77 75
78 } // namespace webrtc 76 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698