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

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

Issue 3011923002: Clean up Stats class used in VideoProcessor tests. (Closed)
Patch Set: Fix VS compile. Created 3 years, 3 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) 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"
15 14
16 namespace webrtc { 15 namespace webrtc {
17 namespace test { 16 namespace test {
18 17
19 TEST(StatsTest, TestEmptyObject) { 18 TEST(StatsTest, TestEmptyObject) {
20 Stats stats; 19 Stats stats;
21 EXPECT_EQ(0u, stats.stats_.size()); 20 stats.PrintSummary(); // Should not crash.
22 stats.PrintSummary(); // should not crash
23 } 21 }
24 22
25 TEST(StatsTest, AddSingleFrame) { 23 TEST(StatsTest, AddSingleFrame) {
26 const int kFrameNumber = 0;
27 Stats stats; 24 Stats stats;
28 stats.NewFrame(kFrameNumber); 25 FrameStatistic* frame_stat = stats.AddFrame();
29 EXPECT_EQ(1u, stats.stats_.size()); 26 EXPECT_EQ(0, frame_stat->frame_number);
30 FrameStatistic* frame_stat = &stats.stats_[0]; 27 EXPECT_EQ(1u, stats.size());
31 EXPECT_EQ(kFrameNumber, frame_stat->frame_number);
32 } 28 }
33 29
34 TEST(StatsTest, AddMultipleFrames) { 30 TEST(StatsTest, AddMultipleFrames) {
35 Stats stats; 31 Stats stats;
36 const int kNumFrames = 1000; 32 const int kNumFrames = 1000;
37 for (int i = 0; i < kNumFrames; ++i) { 33 for (int i = 0; i < kNumFrames; ++i) {
38 FrameStatistic& frame_stat = stats.NewFrame(i); 34 FrameStatistic* frame_stat = stats.AddFrame();
39 EXPECT_EQ(i, frame_stat.frame_number); 35 EXPECT_EQ(i, frame_stat->frame_number);
40 } 36 }
41 EXPECT_EQ(kNumFrames, static_cast<int>(stats.stats_.size())); 37 EXPECT_EQ(kNumFrames, static_cast<int>(stats.size()));
42 38
43 stats.PrintSummary(); // should not crash 39 stats.PrintSummary(); // Should not crash.
44 } 40 }
45 41
46 } // namespace test 42 } // namespace test
47 } // namespace webrtc 43 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/modules/video_coding/codecs/test/stats.cc ('k') | webrtc/modules/video_coding/codecs/test/videoprocessor.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698