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

Unified Diff: tools/telemetry/telemetry/internal/results/csv_pivot_table_output_formatter_unittest.py

Issue 1313243003: [Telemetry] Introduce SummarizableValue. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 5 years, 4 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: tools/telemetry/telemetry/internal/results/csv_pivot_table_output_formatter_unittest.py
diff --git a/tools/telemetry/telemetry/internal/results/csv_pivot_table_output_formatter_unittest.py b/tools/telemetry/telemetry/internal/results/csv_pivot_table_output_formatter_unittest.py
index 8aa53aec0c762be25a9d6ce96f5ceaa56c9a4e01..9df2f39d5be523a529dcca415cbb5847f7f991ba 100644
--- a/tools/telemetry/telemetry/internal/results/csv_pivot_table_output_formatter_unittest.py
+++ b/tools/telemetry/telemetry/internal/results/csv_pivot_table_output_formatter_unittest.py
@@ -9,6 +9,7 @@ from telemetry import story
from telemetry.internal.results import csv_pivot_table_output_formatter
from telemetry.internal.results import page_test_results
from telemetry import page as page_module
+from telemetry.value import improvement_direction
from telemetry.value import scalar
@@ -59,7 +60,9 @@ class CsvPivotTableOutputFormatterTest(unittest.TestCase):
def testSimple(self):
# Test a simple benchmark with only one value:
self.SimulateBenchmarkRun({
- self._story_set[0]: [scalar.ScalarValue(None, 'foo', 'seconds', 3)]})
+ self._story_set[0]: [scalar.ScalarValue(
+ None, 'foo', 'seconds', 3,
+ improvement_direction=improvement_direction.DOWN)]})
expected = self._LINE_SEPARATOR.join([
'story_set,page,name,value,units,run_index',
'story_set,http://www.foo.com/,foo,3,seconds,0',
@@ -69,10 +72,20 @@ class CsvPivotTableOutputFormatterTest(unittest.TestCase):
def testMultiplePagesAndValues(self):
self.SimulateBenchmarkRun({
- self._story_set[0]: [scalar.ScalarValue(None, 'foo', 'seconds', 4)],
- self._story_set[1]: [scalar.ScalarValue(None, 'foo', 'seconds', 3.4),
- scalar.ScalarValue(None, 'bar', 'km', 10),
- scalar.ScalarValue(None, 'baz', 'count', 5)]})
+ self._story_set[0]: [
+ scalar.ScalarValue(
+ None, 'foo', 'seconds', 4,
+ improvement_direction=improvement_direction.DOWN)],
+ self._story_set[1]: [
+ scalar.ScalarValue(
+ None, 'foo', 'seconds', 3.4,
+ improvement_direction=improvement_direction.DOWN),
+ scalar.ScalarValue(
+ None, 'bar', 'km', 10,
+ improvement_direction=improvement_direction.DOWN),
+ scalar.ScalarValue(
+ None, 'baz', 'count', 5,
+ improvement_direction=improvement_direction.DOWN)]})
# Parse CSV output into list of lists.
csv_string = self.Format()
@@ -86,8 +99,13 @@ class CsvPivotTableOutputFormatterTest(unittest.TestCase):
def testTraceTag(self):
self.MakeFormatter(trace_tag='date,option')
self.SimulateBenchmarkRun({
- self._story_set[0]: [scalar.ScalarValue(None, 'foo', 'seconds', 3),
- scalar.ScalarValue(None, 'bar', 'tons', 5)]})
+ self._story_set[0]: [
+ scalar.ScalarValue(
+ None, 'foo', 'seconds', 3,
+ improvement_direction=improvement_direction.DOWN),
+ scalar.ScalarValue(
+ None, 'bar', 'tons', 5,
+ improvement_direction=improvement_direction.DOWN)]})
output = self.Format().split(self._LINE_SEPARATOR)
self.assertTrue(output[0].endswith(',trace_tag_0,trace_tag_1'))

Powered by Google App Engine
This is Rietveld 408576698