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

Side by Side Diff: tools/telemetry/telemetry/page/csv_page_measurement_results_unittest.py

Issue 22754003: Refactor BenchmarkResult.Add to take enum for data_type (Closed) Base URL: http://src.chromium.org/svn/trunk/src/
Patch Set: Created 7 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 import StringIO 4 import StringIO
5 import csv 5 import csv
6 import os 6 import os
7 import unittest 7 import unittest
8 8
9 from telemetry.page import csv_page_measurement_results 9 from telemetry.page import csv_page_measurement_results
10 from telemetry.page import page_set 10 from telemetry.page import page_set
11 from telemetry.page import result_data_type
11 12
12 def _MakePageSet(): 13 def _MakePageSet():
13 return page_set.PageSet.FromDict({ 14 return page_set.PageSet.FromDict({
14 "description": "hello", 15 "description": "hello",
15 "pages": [ 16 "pages": [
16 {"url": "http://www.foo.com/"}, 17 {"url": "http://www.foo.com/"},
17 {"url": "http://www.bar.com/"} 18 {"url": "http://www.bar.com/"}
18 ] 19 ]
19 }, os.path.dirname(__file__)) 20 }, os.path.dirname(__file__))
20 21
(...skipping 79 matching lines...) Expand 10 before | Expand all | Expand 10 after
100 self.assertEquals( 101 self.assertEquals(
101 self.output_data_rows, 102 self.output_data_rows,
102 [[self._page_set[0].url, '-', '3'], 103 [[self._page_set[0].url, '-', '3'],
103 [self._page_set[1].url, '4', '-']]) 104 [self._page_set[1].url, '4', '-']])
104 105
105 def test_histogram(self): 106 def test_histogram(self):
106 results = NonPrintingCsvPageMeasurementResults(self._output, False) 107 results = NonPrintingCsvPageMeasurementResults(self._output, False)
107 results.WillMeasurePage(self._page_set[0]) 108 results.WillMeasurePage(self._page_set[0])
108 results.Add('a', '', 109 results.Add('a', '',
109 '{"buckets": [{"low": 1, "high": 2, "count": 1}]}', 110 '{"buckets": [{"low": 1, "high": 2, "count": 1}]}',
110 data_type='histogram') 111 data_type=result_data_type.HISTOGRAM)
111 results.DidMeasurePage() 112 results.DidMeasurePage()
112 113
113 results.WillMeasurePage(self._page_set[1]) 114 results.WillMeasurePage(self._page_set[1])
114 results.Add('a', '', 115 results.Add('a', '',
115 '{"buckets": [{"low": 2, "high": 3, "count": 1}]}', 116 '{"buckets": [{"low": 2, "high": 3, "count": 1}]}',
116 data_type='histogram') 117 data_type=result_data_type.HISTOGRAM)
117 results.DidMeasurePage() 118 results.DidMeasurePage()
118 119
119 results.PrintSummary() 120 results.PrintSummary()
120 121
121 self.assertEquals( 122 self.assertEquals(
122 self.output_header_row, 123 self.output_header_row,
123 ['url', 'a ()']) 124 ['url', 'a ()'])
124 self.assertEquals( 125 self.assertEquals(
125 self.output_data_rows, 126 self.output_data_rows,
126 [[self._page_set[0].url, '1.5'], 127 [[self._page_set[0].url, '1.5'],
127 [self._page_set[1].url, '2.5']]) 128 [self._page_set[1].url, '2.5']])
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698