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

Unified Diff: tools/perf/measurements/rasterize_and_record_unittest.py

Issue 100443007: telemetry: handle tests that are not supported on a platform. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Catch new exception in TestTracingCleanedUp. Created 6 years, 9 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/perf/measurements/rasterize_and_record_unittest.py
diff --git a/tools/perf/measurements/rasterize_and_record_unittest.py b/tools/perf/measurements/rasterize_and_record_unittest.py
index cc50b43614922a7f504ada5d4537b31eeb050b47..9a7afb8b7fd228d69e99b92ce2c09b09826942f4 100644
--- a/tools/perf/measurements/rasterize_and_record_unittest.py
+++ b/tools/perf/measurements/rasterize_and_record_unittest.py
@@ -2,10 +2,13 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
+import logging
+
from measurements import rasterize_and_record
from telemetry import test
from telemetry.core import wpr_modes
from telemetry.page import page_measurement_unittest_base
+from telemetry.page import page_test
from telemetry.unittest import options_for_unittests
@@ -29,7 +32,11 @@ class RasterizeAndRecordUnitTest(
def testRasterizeAndRecord(self):
ps = self.CreatePageSetFromFileInUnittestDataDir('blank.html')
measurement = rasterize_and_record.RasterizeAndRecord()
- results = self.RunMeasurement(measurement, ps, options=self._options)
+ try:
+ results = self.RunMeasurement(measurement, ps, options=self._options)
+ except page_test.TestNotSupportedOnPlatformFailure as failure:
+ logging.warning(str(failure))
+ return
self.assertEquals(0, len(results.failures))
rasterize_time = results.FindAllPageSpecificValuesNamed('rasterize_time')

Powered by Google App Engine
This is Rietveld 408576698