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

Unified Diff: tools/telemetry/telemetry/core/chrome/android_browser_finder_unittest.py

Issue 23072018: [telemetry] Move telemetry/core/chrome/ to telemetry/core/backends/chrome/ (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix perf smoothness_unittest. Created 7 years, 4 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/chrome/android_browser_finder_unittest.py
diff --git a/tools/telemetry/telemetry/core/chrome/android_browser_finder_unittest.py b/tools/telemetry/telemetry/core/chrome/android_browser_finder_unittest.py
deleted file mode 100644
index 26df16949f007b0372fc76453e50b1477779e872..0000000000000000000000000000000000000000
--- a/tools/telemetry/telemetry/core/chrome/android_browser_finder_unittest.py
+++ /dev/null
@@ -1,89 +0,0 @@
-# Copyright (c) 2012 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 unittest
-
-from telemetry.core import browser_options
-from telemetry.core.chrome import android_browser_finder
-from telemetry.unittest import system_stub
-
-class LoggingStub(object):
- def __init__(self):
- self.warnings = []
-
- def info(self, msg, *args):
- pass
-
- def warn(self, msg, *args):
- self.warnings.append(msg % args)
-
-class AndroidBrowserFinderTest(unittest.TestCase):
- def setUp(self):
- self._stubs = system_stub.Override(android_browser_finder,
- ['adb_commands', 'subprocess'])
- android_browser_finder.adb_works = None # Blow cache between runs.
-
- def tearDown(self):
- self._stubs.Restore()
-
- def test_no_adb(self):
- finder_options = browser_options.BrowserFinderOptions()
-
- def NoAdb(*args, **kargs): # pylint: disable=W0613
- raise OSError('not found')
- self._stubs.subprocess.Popen = NoAdb
- browsers = android_browser_finder.FindAllAvailableBrowsers(finder_options)
- self.assertEquals(0, len(browsers))
-
- def test_adb_no_devices(self):
- finder_options = browser_options.BrowserFinderOptions()
-
- browsers = android_browser_finder.FindAllAvailableBrowsers(finder_options)
- self.assertEquals(0, len(browsers))
-
-
- def test_adb_permissions_error(self):
- finder_options = browser_options.BrowserFinderOptions()
-
- self._stubs.subprocess.Popen.communicate_result = (
- """List of devices attached
-????????????\tno permissions""",
- """* daemon not running. starting it now on port 5037 *
-* daemon started successfully *
-""")
-
- log_stub = LoggingStub()
- browsers = android_browser_finder.FindAllAvailableBrowsers(
- finder_options, log_stub)
- self.assertEquals(3, len(log_stub.warnings))
- self.assertEquals(0, len(browsers))
-
-
- def test_adb_two_devices(self):
- finder_options = browser_options.BrowserFinderOptions()
-
- self._stubs.adb_commands.attached_devices = ['015d14fec128220c',
- '015d14fec128220d']
-
- log_stub = LoggingStub()
- browsers = android_browser_finder.FindAllAvailableBrowsers(
- finder_options, log_stub)
- self.assertEquals(1, len(log_stub.warnings))
- self.assertEquals(0, len(browsers))
-
- def test_adb_one_device(self):
- finder_options = browser_options.BrowserFinderOptions()
-
- self._stubs.adb_commands.attached_devices = ['015d14fec128220c']
-
- def OnPM(args):
- assert args[0] == 'pm'
- assert args[1] == 'list'
- assert args[2] == 'packages'
- return ['package:org.chromium.content_shell_apk',
- 'package.com.google.android.setupwizard']
-
- self._stubs.adb_commands.shell_command_handlers['pm'] = OnPM
-
- browsers = android_browser_finder.FindAllAvailableBrowsers(finder_options)
- self.assertEquals(1, len(browsers))

Powered by Google App Engine
This is Rietveld 408576698