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

Side by Side Diff: webrtc/modules/video_coding/codecs/test/stats_unittest.cc

Issue 2916883002: Small updates to test::Stats. (Closed)
Patch Set: Created 3 years, 6 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
« no previous file with comments | « webrtc/modules/video_coding/codecs/test/stats.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2011 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2011 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
11 #include "webrtc/modules/video_coding/codecs/test/stats.h" 11 #include "webrtc/modules/video_coding/codecs/test/stats.h"
12 12
13 #include "webrtc/test/gtest.h" 13 #include "webrtc/test/gtest.h"
14 #include "webrtc/typedefs.h" 14 #include "webrtc/typedefs.h"
15 15
16 namespace webrtc { 16 namespace webrtc {
17 namespace test { 17 namespace test {
18 18
19 class StatsTest : public testing::Test { 19 TEST(StatsTest, TestEmptyObject) {
20 protected: 20 Stats stats;
21 StatsTest() {} 21 EXPECT_EQ(0u, stats.stats_.size());
22 22 stats.PrintSummary(); // should not crash
23 virtual ~StatsTest() {}
24
25 void SetUp() { stats_ = new Stats(); }
26
27 void TearDown() { delete stats_; }
28
29 Stats* stats_;
30 };
31
32 // Test empty object
33 TEST_F(StatsTest, Uninitialized) {
34 EXPECT_EQ(0u, stats_->stats_.size());
35 stats_->PrintSummary(); // should not crash
36 } 23 }
37 24
38 // Add single frame stats and verify 25 TEST(StatsTest, AddSingleFrame) {
39 TEST_F(StatsTest, AddOne) { 26 const int kFrameNumber = 0;
40 stats_->NewFrame(0u); 27 Stats stats;
41 FrameStatistic* frameStat = &stats_->stats_[0]; 28 stats.NewFrame(kFrameNumber);
42 EXPECT_EQ(0, frameStat->frame_number); 29 EXPECT_EQ(1u, stats.stats_.size());
30 FrameStatistic* frame_stat = &stats.stats_[0];
31 EXPECT_EQ(kFrameNumber, frame_stat->frame_number);
43 } 32 }
44 33
45 // Add multiple frame stats and verify 34 TEST(StatsTest, AddMultipleFrames) {
46 TEST_F(StatsTest, AddMany) { 35 Stats stats;
47 int nbr_of_frames = 1000; 36 const int kNumFrames = 1000;
48 for (int i = 0; i < nbr_of_frames; ++i) { 37 for (int i = 0; i < kNumFrames; ++i) {
49 FrameStatistic& frameStat = stats_->NewFrame(i); 38 FrameStatistic& frame_stat = stats.NewFrame(i);
50 EXPECT_EQ(i, frameStat.frame_number); 39 EXPECT_EQ(i, frame_stat.frame_number);
51 } 40 }
52 EXPECT_EQ(nbr_of_frames, static_cast<int>(stats_->stats_.size())); 41 EXPECT_EQ(kNumFrames, static_cast<int>(stats.stats_.size()));
53 42
54 stats_->PrintSummary(); // should not crash 43 stats.PrintSummary(); // should not crash
55 } 44 }
56 45
57 } // namespace test 46 } // namespace test
58 } // namespace webrtc 47 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/modules/video_coding/codecs/test/stats.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698