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

Unified Diff: telemetry/telemetry/internal/platform/tracing_agent/battor_tracing_agent_unittest.py

Issue 2889583003: [Telemetry][CodeHealth] Make unittests with fake platforms use fakes.FakePlatform (Closed)
Patch Set: [Telemetry][CodeHealth] Make unittests with fake platforms use fakes.FakePlatform Created 3 years, 7 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: telemetry/telemetry/internal/platform/tracing_agent/battor_tracing_agent_unittest.py
diff --git a/telemetry/telemetry/internal/platform/tracing_agent/battor_tracing_agent_unittest.py b/telemetry/telemetry/internal/platform/tracing_agent/battor_tracing_agent_unittest.py
index 1c6dc065f95f5fa39b3deff074b1e8f355747d99..440d9036fc6a27b628cdf0b21db4deea0c0a7b54 100644
--- a/telemetry/telemetry/internal/platform/tracing_agent/battor_tracing_agent_unittest.py
+++ b/telemetry/telemetry/internal/platform/tracing_agent/battor_tracing_agent_unittest.py
@@ -8,6 +8,7 @@ from battor import battor_error
from battor import battor_wrapper
from devil.android import battery_utils
from telemetry.internal.platform.tracing_agent import battor_tracing_agent
+from telemetry.testing import fakes
from telemetry.timeline import trace_data
from telemetry.timeline import tracing_config
from tracing.trace_data import trace_data
@@ -28,28 +29,6 @@ class FakeBatteryUtils(object):
return self._charging_state
-class FakePlatformBackend(object):
- def GetOSName(self):
- return ''
-
-
-class FakeAndroidPlatformBackend(FakePlatformBackend):
- def __init__(self):
- super(FakeAndroidPlatformBackend, self).__init__()
- self.device = 'fake_device'
-
- def GetOSName(self):
- return 'android'
-
-
-class FakeDesktopPlatformBackend(FakePlatformBackend):
- def __init__(self):
- self.platform = 'win'
-
- def GetOSName(self):
- return self.platform
-
-
class FakeBattOr(object):
def __init__(self, test_platform, android_device=None, battor_path=None,
battor_map=None, serial_log_bucket=None):
@@ -103,8 +82,10 @@ class BattOrTracingAgentTest(unittest.TestCase):
battery_utils.BatteryUtils = FakeBatteryUtils
# Agents and backends.
- self.android_backend = FakeAndroidPlatformBackend()
- self.desktop_backend = FakeDesktopPlatformBackend()
+ self.android_backend = fakes.FakePlatform()
+ self.android_backend.SetOSName('android')
+ self.desktop_backend = fakes.FakePlatform()
+ self.desktop_backend.SetOSName('win')
self.android_agent = (
battor_tracing_agent.BattOrTracingAgent(self.android_backend))
self.desktop_agent = (
@@ -115,10 +96,9 @@ class BattOrTracingAgentTest(unittest.TestCase):
battery_utils.BatteryUtils = self._battery_utils
def testInit(self):
- self.assertTrue(isinstance(self.android_agent._platform_backend,
- FakeAndroidPlatformBackend))
- self.assertTrue(isinstance(self.desktop_agent._platform_backend,
- FakeDesktopPlatformBackend))
+ self.assertEqual(
+ self.android_agent._platform_backend.GetOSName(), 'android')
+ self.assertEqual(self.desktop_agent._platform_backend.GetOSName(), 'win')
def testIsSupportedAndroid(self):
self.assertTrue(battor_tracing_agent.BattOrTracingAgent.IsSupported(
« no previous file with comments | « telemetry/telemetry/internal/browser/tab_unittest.py ('k') | telemetry/telemetry/internal/story_runner_unittest.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698