Add Duration field to EventRateCounter
This can be better used to determine the length of test calls, rather than using the interval metric. Bug: webrtc:11017 Change-Id: I69f66fa750b061a7d010d591a718555e2b5b34b7 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/156087 Commit-Queue: Evan Shrubsole <eshr@google.com> Reviewed-by: Sebastian Jansson <srte@webrtc.org> Cr-Commit-Position: refs/heads/master@{#29413}
This commit is contained in:
parent
0169a3e5cc
commit
9ddd72989a
@ -162,6 +162,7 @@ if (rtc_include_tests) {
|
||||
rtc_source_set("scenario_unittests") {
|
||||
testonly = true
|
||||
sources = [
|
||||
"performance_stats_unittest.cc",
|
||||
"scenario_unittest.cc",
|
||||
"stats_collection_unittest.cc",
|
||||
"video_stream_unittest.cc",
|
||||
|
||||
@ -40,6 +40,13 @@ double EventRateCounter::Rate() const {
|
||||
return (event_count_ - 1) / (last_time_ - first_time_).seconds<double>();
|
||||
}
|
||||
|
||||
TimeDelta EventRateCounter::TotalDuration() const {
|
||||
if (first_time_.IsInfinite()) {
|
||||
return TimeDelta::Zero();
|
||||
}
|
||||
return last_time_ - first_time_;
|
||||
}
|
||||
|
||||
double SampleStats<double>::Max() {
|
||||
if (IsEmpty())
|
||||
return INFINITY;
|
||||
|
||||
@ -99,6 +99,7 @@ class EventRateCounter {
|
||||
bool IsEmpty() const;
|
||||
double Rate() const;
|
||||
SampleStats<TimeDelta>& interval() { return interval_; }
|
||||
TimeDelta TotalDuration() const;
|
||||
int Count() const { return event_count_; }
|
||||
|
||||
private:
|
||||
|
||||
27
test/scenario/performance_stats_unittest.cc
Normal file
27
test/scenario/performance_stats_unittest.cc
Normal file
@ -0,0 +1,27 @@
|
||||
/*
|
||||
* Copyright 2019 The WebRTC project authors. All Rights Reserved.
|
||||
*
|
||||
* Use of this source code is governed by a BSD-style license
|
||||
* that can be found in the LICENSE file in the root of the source
|
||||
* tree. An additional intellectual property rights grant can be found
|
||||
* in the file PATENTS. All contributing project authors may
|
||||
* be found in the AUTHORS file in the root of the source tree.
|
||||
*/
|
||||
#include "test/scenario/performance_stats.h"
|
||||
|
||||
#include "test/gtest.h"
|
||||
|
||||
namespace webrtc {
|
||||
namespace test {
|
||||
|
||||
TEST(EventRateCounter, ReturnsCorrectTotalDuration) {
|
||||
EventRateCounter event_rate_counter;
|
||||
EXPECT_EQ(event_rate_counter.TotalDuration(), TimeDelta::Zero());
|
||||
event_rate_counter.AddEvent(Timestamp::seconds(1));
|
||||
EXPECT_EQ(event_rate_counter.TotalDuration(), TimeDelta::Zero());
|
||||
event_rate_counter.AddEvent(Timestamp::seconds(2));
|
||||
EXPECT_EQ(event_rate_counter.TotalDuration(), TimeDelta::seconds(1));
|
||||
}
|
||||
|
||||
} // namespace test
|
||||
} // namespace webrtc
|
||||
Loading…
x
Reference in New Issue
Block a user