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

Unified Diff: tools/telemetry/telemetry/core/chrome/cros_interface_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/cros_interface_unittest.py
diff --git a/tools/telemetry/telemetry/core/chrome/cros_interface_unittest.py b/tools/telemetry/telemetry/core/chrome/cros_interface_unittest.py
deleted file mode 100644
index 994b05192c0d8dfaa00679f1466021a656af4638..0000000000000000000000000000000000000000
--- a/tools/telemetry/telemetry/core/chrome/cros_interface_unittest.py
+++ /dev/null
@@ -1,158 +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.
-
-# TODO(nduca): Rewrite what some of these tests to use mocks instead of
-# actually talking to the device. This would improve our coverage quite
-# a bit.
-import unittest
-import socket
-import sys
-
-from telemetry.core import util
-from telemetry.core.chrome import cros_browser_backend
-from telemetry.core.chrome import cros_interface
-from telemetry.unittest import options_for_unittests
-from telemetry.unittest import RequiresBrowserOfType
-
-class CrOSInterfaceTest(unittest.TestCase):
- @RequiresBrowserOfType('cros-chrome')
- def testPushContents(self):
- remote = options_for_unittests.GetCopy().cros_remote
- cri = cros_interface.CrOSInterface(
- remote,
- options_for_unittests.GetCopy().cros_ssh_identity)
- cri.RunCmdOnDevice(['rm', '-rf', '/tmp/testPushContents'])
- cri.PushContents('hello world', '/tmp/testPushContents')
- contents = cri.GetFileContents('/tmp/testPushContents')
- self.assertEquals(contents, 'hello world')
-
- @RequiresBrowserOfType('cros-chrome')
- def testExists(self):
- remote = options_for_unittests.GetCopy().cros_remote
- cri = cros_interface.CrOSInterface(
- remote,
- options_for_unittests.GetCopy().cros_ssh_identity)
- self.assertTrue(cri.FileExistsOnDevice('/proc/cpuinfo'))
- self.assertTrue(cri.FileExistsOnDevice('/etc/passwd'))
- self.assertFalse(cri.FileExistsOnDevice('/etc/sdlfsdjflskfjsflj'))
-
- def testExistsLocal(self):
- if not sys.platform.startswith('linux'):
- return
-
- cri = cros_interface.CrOSInterface()
- self.assertTrue(cri.FileExistsOnDevice('/proc/cpuinfo'))
- self.assertTrue(cri.FileExistsOnDevice('/etc/passwd'))
- self.assertFalse(cri.FileExistsOnDevice('/etc/sdlfsdjflskfjsflj'))
-
- @RequiresBrowserOfType('cros-chrome')
- def testGetFileContents(self): # pylint: disable=R0201
- remote = options_for_unittests.GetCopy().cros_remote
- cri = cros_interface.CrOSInterface(
- remote,
- options_for_unittests.GetCopy().cros_ssh_identity)
- hosts = cri.GetFileContents('/etc/hosts')
- assert hosts.startswith('# /etc/hosts')
-
- @RequiresBrowserOfType('cros-chrome')
- def testGetFileContentsForSomethingThatDoesntExist(self):
- remote = options_for_unittests.GetCopy().cros_remote
- cri = cros_interface.CrOSInterface(
- remote,
- options_for_unittests.GetCopy().cros_ssh_identity)
- self.assertRaises(
- OSError,
- lambda: cri.GetFileContents('/tmp/209fuslfskjf/dfsfsf'))
-
- @RequiresBrowserOfType('cros-chrome')
- def testIsServiceRunning(self):
- remote = options_for_unittests.GetCopy().cros_remote
- cri = cros_interface.CrOSInterface(
- remote,
- options_for_unittests.GetCopy().cros_ssh_identity)
-
- self.assertTrue(cri.IsServiceRunning('openssh-server'))
-
- def testIsServiceRunningLocal(self):
- if not sys.platform.startswith('linux'):
- return
- cri = cros_interface.CrOSInterface()
- self.assertTrue(cri.IsServiceRunning('dbus'))
-
- @RequiresBrowserOfType('cros-chrome')
- def testGetRemotePortAndIsHTTPServerRunningOnPort(self):
- remote = options_for_unittests.GetCopy().cros_remote
- cri = cros_interface.CrOSInterface(
- remote,
- options_for_unittests.GetCopy().cros_ssh_identity)
-
- # Create local server.
- sock = socket.socket()
- sock.bind(('', 0))
- port = sock.getsockname()[1]
- sock.listen(0)
-
- # Get remote port and ensure that it was unused.
- remote_port = cri.GetRemotePort()
- self.assertFalse(cri.IsHTTPServerRunningOnPort(remote_port))
-
- # Forward local server's port to remote device's remote_port.
- forwarder = cros_browser_backend.SSHForwarder(
- cri, 'R', util.PortPair(port, remote_port))
-
- # At this point, remote device should be able to connect to local server.
- self.assertTrue(cri.IsHTTPServerRunningOnPort(remote_port))
-
- # Next remote port shouldn't be the same as remote_port, since remote_port
- # is now in use.
- self.assertTrue(cri.GetRemotePort() != remote_port)
-
- # Close forwarder and local server ports.
- forwarder.Close()
- sock.close()
-
- # Device should no longer be able to connect to remote_port since it is no
- # longer in use.
- self.assertFalse(cri.IsHTTPServerRunningOnPort(remote_port))
-
- @RequiresBrowserOfType('cros-chrome')
- def testGetRemotePortReservedPorts(self):
- remote = options_for_unittests.GetCopy().cros_remote
- cri = cros_interface.CrOSInterface(
- remote,
- options_for_unittests.GetCopy().cros_ssh_identity)
-
- # Should return 2 separate ports even though the first one isn't technically
- # being used yet.
- remote_port_1 = cri.GetRemotePort()
- remote_port_2 = cri.GetRemotePort()
-
- self.assertTrue(remote_port_1 != remote_port_2)
-
- # TODO(tengs): It would be best if we can filter this test and other tests
- # that need to be run locally based on the platform of the system browser.
- def testEscapeCmdArguments(self):
- ''' Commands and their arguments that are executed through the cros
- interface should follow bash syntax. This test needs to run on remotely
- and locally on the device to check for consistency.
- '''
- if not sys.platform.startswith('linux'):
- return
-
- cri = cros_interface.CrOSInterface(
- options_for_unittests.GetCopy().cros_remote,
- options_for_unittests.GetCopy().cros_ssh_identity)
-
- # Check arguments with no special characters
- stdout, _ = cri.RunCmdOnDevice(['echo', '--arg1=value1', '--arg2=value2',
- '--arg3="value3"'])
- assert(stdout.strip() == '--arg1=value1 --arg2=value2 --arg3=value3')
-
- # Check argument with special characters escaped
- stdout, _ = cri.RunCmdOnDevice(['echo', '--arg=A\\; echo \\"B\\"'])
- assert(stdout.strip() == '--arg=A; echo "B"')
-
- # Check argument with special characters in quotes
- stdout, _ = cri.RunCmdOnDevice(['echo', "--arg='$HOME;;$PATH'"])
- assert(stdout.strip() == "--arg=$HOME;;$PATH")
« no previous file with comments | « tools/telemetry/telemetry/core/chrome/cros_interface.py ('k') | tools/telemetry/telemetry/core/chrome/crx_id.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698