[tools/perf] Add unittests coverage for system health benchmarks

tools/perf/benchmarks/system_health_unittest.py

Index: tools/perf/benchmarks/system_health_unittest.py
diff --git a/tools/perf/benchmarks/system_health_unittest.py b/tools/perf/benchmarks/system_health_unittest.py
new file mode 100644
index 0000000000000000000000000000000000000000..d52332034a05c05e5ee8db9ce419b973e9bcc966
--- /dev/null
+++ b/tools/perf/benchmarks/system_health_unittest.py
@@ -0,0 +1,46 @@
+# Copyright 2016 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 sys
+import unittest
+
+from benchmarks import system_health as system_health_benchmark
+from core import path_util
+from page_sets.system_health import system_health_stories
+
+from telemetry import benchmark as benchmark_module
+from telemetry.core import discover
+
+
+def _GetAllSystemHealthBenchmarks():
+  all_perf_benchmarks = discover.DiscoverClasses(
+      path_util.GetPerfBenchmarksDir(), path_util.GetPerfDir(),
+      benchmark_module.Benchmark,
+      index_by_class_name=True).values()
+  return [b for b in all_perf_benchmarks if
+          sys.modules[b.__module__] == system_health_benchmark]
+
+
+class TestSystemHealthBenchmarks(unittest.TestCase):
+
+  def testNaming(self):

[petrcermak, 2016/08/03 15:47:27]

nit: testNamePrefix might be better?

[nednguyen, 2016/08/03 17:50:06]

Done.

+    for b in _GetAllSystemHealthBenchmarks():
+      self.assertTrue(
+          b.Name().startswith('system_health.'),
+          '%r must have name started with "system_health." prefix' % b)

[petrcermak, 2016/08/03 15:47:27]

nit: s/started/starting/

[nednguyen, 2016/08/03 17:50:06]

Done.

+
+  def testShouldTearDownStateAfterEachStoryRunIsTrue(self):
+    for b in _GetAllSystemHealthBenchmarks():
+      self.assertTrue(
+          b.ShouldTearDownStateAfterEachStoryRun(),
+          '%r has ShouldTearDownStateAfterEachStoryRun set to False' % b)
+
+  def testSystemHealthStorySetAreUsed(self):

[petrcermak, 2016/08/03 15:47:27]

nit: s/Are/Is/

[nednguyen, 2016/08/03 17:50:06]

Done.

+    for b in _GetAllSystemHealthBenchmarks():
+      if b is system_health_benchmark.WebviewStartupSystemHealthBenchmark:
+        continue
+      self.assertIsInstance(
+          b().CreateStorySet(None),
+          system_health_stories.SystemHealthStorySet,
+          '%r does not use SystemHealthStorySet' % b)