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

Unified Diff: tools/telemetry/telemetry/core/platform/mac_platform_backend_unittest.py

Issue 163183003: [Telemetry] Small fixes to power monitoring (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 10 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/core/platform/mac_platform_backend_unittest.py
diff --git a/tools/telemetry/telemetry/core/platform/mac_platform_backend_unittest.py b/tools/telemetry/telemetry/core/platform/mac_platform_backend_unittest.py
index f7e98e04ca7a7aa702c43daa8ebb56acc2c2a382..82b84224d9a54ba8dc94aa98285beab3bc145b0a 100644
--- a/tools/telemetry/telemetry/core/platform/mac_platform_backend_unittest.py
+++ b/tools/telemetry/telemetry/core/platform/mac_platform_backend_unittest.py
@@ -39,8 +39,9 @@ class MacPlatformBackendTest(unittest.TestCase):
@test.Enabled('mac')
def testCanMonitorPowerUsage(self):
- mavericks_or_later = int(os.uname()[2].split('.')[0]) >= 13
backend = mac_platform_backend.MacPlatformBackend()
+ mavericks_or_later = (
+ backend.GetOSVersionName() >= mac_platform_backend.MAVERICKS)
# Should always be able to monitor power usage on OS Version >= 10.9 .
self.assertEqual(backend.CanMonitorPowerAsync(), mavericks_or_later,
"Error checking powermetrics availability: '%s'" % '|'.join(os.uname()))

Powered by Google App Engine
This is Rietveld 408576698