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

Unified Diff: tools/telemetry/telemetry/page/html_page_measurement_results_unittest.py

Issue 327583002: Move results related stuff to telemetry/results (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Change the directory name from test_results to results Created 6 years, 6 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/page/html_page_measurement_results_unittest.py
diff --git a/tools/telemetry/telemetry/page/html_page_measurement_results_unittest.py b/tools/telemetry/telemetry/page/html_page_measurement_results_unittest.py
deleted file mode 100644
index b7b8154309c90ee780face7dbd204b4d41cc3e31..0000000000000000000000000000000000000000
--- a/tools/telemetry/telemetry/page/html_page_measurement_results_unittest.py
+++ /dev/null
@@ -1,240 +0,0 @@
-# Copyright 2013 The Chromium Authors. All rights reserved.
-# Use of this source code is governed by a BSD-style license that can be
-# found in the LICENSE file.
-import os
-import StringIO
-import unittest
-
-from telemetry.page import html_page_measurement_results
-from telemetry.page import page_set
-
-
-def _MakePageSet():
- ps = page_set.PageSet(file_path=os.path.dirname(__file__))
- ps.AddPageWithDefaultRunNavigate('http://www.foo.com/')
- ps.AddPageWithDefaultRunNavigate('http://www.bar.com/')
- ps.AddPageWithDefaultRunNavigate('http://www.baz.com/')
- return ps
-
-
-class DeterministicHtmlPageMeasurementResults(
- html_page_measurement_results.HtmlPageMeasurementResults):
- def _GetBuildTime(self):
- return 'build_time'
-
- def _GetRevision(self):
- return 'revision'
-
-
-# Wrap string IO with a .name property so that it behaves more like a file.
-class StringIOFile(StringIO.StringIO):
- name = 'fake_output_file'
-
-
-class HtmlPageMeasurementResultsTest(unittest.TestCase):
-
- def test_basic_summary(self):
- test_page_set = _MakePageSet()
- output_file = StringIOFile()
-
- # Run the first time and verify the results are written to the HTML file.
- results = DeterministicHtmlPageMeasurementResults(
- output_file, 'test_name', False, False, 'browser_type')
- results.WillMeasurePage(test_page_set.pages[0])
- results.Add('a', 'seconds', 3)
- results.DidMeasurePage()
- results.AddSuccess(test_page_set.pages[0])
-
- results.WillMeasurePage(test_page_set.pages[1])
- results.Add('a', 'seconds', 7)
- results.DidMeasurePage()
- results.AddSuccess(test_page_set.pages[1])
-
- results.PrintSummary()
- results.GetResults()
- expected = {
- "platform": "browser_type",
- "buildTime": "build_time",
- "label": None,
- "tests": {
- "test_name": {
- "metrics": {
- "a": {
- "current": [3, 7],
- "units": "seconds",
- "important": True
- },
- "telemetry_page_measurement_results.num_failed": {
- "current": [0],
- "units": "count",
- "important": False
- },
- "a.http://www.bar.com/": {
- "current": [7],
- "units": "seconds",
- "important": False
- },
- "telemetry_page_measurement_results.num_errored": {
- "current": [0],
- "units": "count",
- "important": False
- },
- "a.http://www.foo.com/": {
- "current": [3],
- "units": "seconds",
- "important": False
- }
- }
- }
- },
- "revision": "revision"
- }
- self.assertEquals(expected, results.GetResults())
-
- # Run the second time and verify the results are appended to the HTML file.
- output_file.seek(0)
- results = DeterministicHtmlPageMeasurementResults(
- output_file, 'test_name', False, False, 'browser_type')
- results.WillMeasurePage(test_page_set.pages[0])
- results.Add('a', 'seconds', 4)
- results.DidMeasurePage()
- results.AddSuccess(test_page_set.pages[0])
-
- results.WillMeasurePage(test_page_set.pages[1])
- results.Add('a', 'seconds', 8)
- results.DidMeasurePage()
- results.AddSuccess(test_page_set.pages[1])
-
- results.PrintSummary()
- expected = [
- {
- "platform": "browser_type",
- "buildTime": "build_time",
- "label": None,
- "tests": {
- "test_name": {
- "metrics": {
- "a": {
- "current": [3, 7],
- "units": "seconds",
- "important": True
- },
- "telemetry_page_measurement_results.num_failed": {
- "current": [0],
- "units": "count",
- "important": False
- },
- "a.http://www.bar.com/": {
- "current": [7],
- "units": "seconds",
- "important": False
- },
- "telemetry_page_measurement_results.num_errored": {
- "current": [0],
- "units": "count",
- "important": False
- },
- "a.http://www.foo.com/": {
- "current": [3],
- "units": "seconds",
- "important": False
- }
- }
- }
- },
- "revision": "revision"
- },
- {
- "platform": "browser_type",
- "buildTime": "build_time",
- "label": None,
- "tests": {
- "test_name": {
- "metrics": {
- "a": {
- "current": [4, 8],
- "units": "seconds",
- "important": True
- },
- "telemetry_page_measurement_results.num_failed": {
- "current": [0],
- "units": "count",
- "important": False,
- },
- "a.http://www.bar.com/": {
- "current": [8],
- "units": "seconds",
- "important": False
- },
- "telemetry_page_measurement_results.num_errored": {
- "current": [0],
- "units": "count",
- "important": False
- },
- "a.http://www.foo.com/": {
- "current": [4],
- "units": "seconds",
- "important": False
- }
- }
- }
- },
- "revision": "revision"
- }]
- self.assertEquals(expected, results.GetCombinedResults())
- last_output_len = len(output_file.getvalue())
-
- # Now reset the results and verify the old ones are gone.
- output_file.seek(0)
- results = DeterministicHtmlPageMeasurementResults(
- output_file, 'test_name', True, False, 'browser_type')
- results.WillMeasurePage(test_page_set.pages[0])
- results.Add('a', 'seconds', 5)
- results.DidMeasurePage()
- results.AddSuccess(test_page_set.pages[0])
-
- results.WillMeasurePage(test_page_set.pages[1])
- results.Add('a', 'seconds', 9)
- results.DidMeasurePage()
- results.AddSuccess(test_page_set.pages[1])
-
- results.PrintSummary()
- expected = [{
- "platform": "browser_type",
- "buildTime": "build_time",
- "label": None,
- "tests": {
- "test_name": {
- "metrics": {
- "a": {
- "current": [5, 9],
- "units": "seconds",
- "important": True
- },
- "telemetry_page_measurement_results.num_failed": {
- "current": [0],
- "units": "count",
- "important": False
- },
- "a.http://www.bar.com/": {
- "current": [9],
- "units": "seconds",
- "important": False
- },
- "telemetry_page_measurement_results.num_errored": {
- "current": [0],
- "units": "count",
- "important": False
- },
- "a.http://www.foo.com/": {
- "current": [5],
- "units": "seconds",
- "important": False
- }
- }
- }
- },
- "revision": "revision"
- }]
- self.assertEquals(expected, results.GetCombinedResults())
- self.assertTrue(len(output_file.getvalue()) < last_output_len)

Powered by Google App Engine
This is Rietveld 408576698