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

Unified Diff: tracing/tracing/metrics/system_health/tti_power_metric_test.html

Issue 2334233003: Merge NumericValue into Histogram (Closed)
Patch Set: fix rail_power_metric Created 4 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 side-by-side diff with in-line comments
Download patch
Index: tracing/tracing/metrics/system_health/tti_power_metric_test.html
diff --git a/tracing/tracing/metrics/system_health/tti_power_metric_test.html b/tracing/tracing/metrics/system_health/tti_power_metric_test.html
index 6a08cce776a11a0c39074acd4d49787f0e776aa4..c8ea6f17e972dfa288851e418279d6b877cc8a41 100644
--- a/tracing/tracing/metrics/system_health/tti_power_metric_test.html
+++ b/tracing/tracing/metrics/system_health/tti_power_metric_test.html
@@ -12,12 +12,6 @@ found in the LICENSE file.
<script>
'use strict';
-function getMetricValue(valueSet, name) {
- for (var value of valueSet)
- if (value.name === name)
- return value.numeric.average;
-}
-
tr.b.unittest.testSuite(function() {
test('ttiPowerMetric_oneInterval', function() {
@@ -101,10 +95,10 @@ tr.b.unittest.testSuite(function() {
// Energy for first load is 20 W * 15.2 s
// (interval from 0.2 s to 15.4 s)
- assert.closeTo(
- getMetricValue(valueSet, 'energy:load'), 304, 0.1);
- assert.closeTo(
- getMetricValue(valueSet, 'power:after_load'), 10, 0.01);
+ assert.closeTo(tr.b.getOnlyElement(valueSet.getValuesNamed(
+ 'energy:load')).average, 304, 0.1);
+ assert.closeTo(tr.b.getOnlyElement(valueSet.getValuesNamed(
+ 'power:after_load')).average, 10, 0.01);
});
});
</script>

Powered by Google App Engine
This is Rietveld 408576698