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

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

Issue 675843004: Telemetry: Implement idle wakeups counting on Android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address nit Created 6 years, 1 month 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/linux_based_platform_backend_unittest.py
diff --git a/tools/telemetry/telemetry/core/platform/linux_based_platform_backend_unittest.py b/tools/telemetry/telemetry/core/platform/linux_based_platform_backend_unittest.py
index d227e300d3642d50661aa7a6c9ee2e68c82a5ea4..2962b7ba8fe5d24d433db2be0019aa94d28f65f9 100644
--- a/tools/telemetry/telemetry/core/platform/linux_based_platform_backend_unittest.py
+++ b/tools/telemetry/telemetry/core/platform/linux_based_platform_backend_unittest.py
@@ -1,6 +1,7 @@
# 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 logging
import os
import unittest
@@ -26,6 +27,9 @@ class TestBackend(linux_based_platform_backend.LinuxBasedPlatformBackend):
def GetClockTicks(self):
return 41
+ def GetIdleWakeupCount(self, pid):
+ return self._GetIdleWakeupCount(pid)
+
class LinuxBasedPlatformBackendTest(unittest.TestCase):
@@ -33,6 +37,21 @@ class LinuxBasedPlatformBackendTest(unittest.TestCase):
with open(os.path.join(util.GetUnittestDataDir(), real_file)) as f:
backend.SetMockFile(mock_file, f.read())
+ def testGetIdleWakeupCount(self):
+ backend = TestBackend()
+ self.SetMockFileInBackend(backend, 'timer_stats_empty', '/proc/timer_stats')
+ result = backend.GetIdleWakeupCount(31735).total_sum()
+ self.assertEquals(result, 0)
+
+ self.SetMockFileInBackend(backend, 'timer_stats', '/proc/timer_stats')
+ result = backend.GetIdleWakeupCount(31735).total_sum()
+ self.assertEquals(result, 14)
+
+ self.SetMockFileInBackend(backend, 'timer_stats_dup_pid',
+ '/proc/timer_stats')
+ result = backend.GetIdleWakeupCount(31735).total_sum()
+ self.assertEquals(result, 1303)
+
def testGetCpuStatsBasic(self):
if not linux_based_platform_backend.resource:
logging.warning('Test not supported')
@@ -40,8 +59,10 @@ class LinuxBasedPlatformBackendTest(unittest.TestCase):
backend = TestBackend()
self.SetMockFileInBackend(backend, 'stat', '/proc/1/stat')
+ self.SetMockFileInBackend(backend, 'timer_stats_empty', '/proc/timer_stats')
result = backend.GetCpuStats(1)
- self.assertEquals(result, {'CpuProcessTime': 22.0})
+ self.assertTrue('CpuProcessTime'in result)
+ self.assertEquals(result['CpuProcessTime'], 22.0)
def testGetCpuTimestampBasic(self):
if not linux_based_platform_backend.resource:

Powered by Google App Engine
This is Rietveld 408576698