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

Unified Diff: chrome/test/chromedriver/test/run_py_tests.py

Issue 63373003: [chromedriver] Add a WebView shell app for testing. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Don't run the test just yet Created 7 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: chrome/test/chromedriver/test/run_py_tests.py
diff --git a/chrome/test/chromedriver/test/run_py_tests.py b/chrome/test/chromedriver/test/run_py_tests.py
index bf6bf73a93d9ae14a2d2025850568a6c3f0a629e..fc2756f347e5ad40a05600cc9e22b9443d4521e9 100755
--- a/chrome/test/chromedriver/test/run_py_tests.py
+++ b/chrome/test/chromedriver/test/run_py_tests.py
@@ -34,6 +34,7 @@ _TEST_DATA_DIR = os.path.join(chrome_paths.GetTestData(), 'chromedriver')
if util.IsLinux():
sys.path.insert(0, os.path.join(chrome_paths.GetSrc(), 'build', 'android'))
from pylib import android_commands
+ from pylib import constants
from pylib import forwarder
from pylib import valgrind_tools
@@ -91,7 +92,7 @@ def _GetDesktopNegativeFilter(version_name):
return filter
_ANDROID_NEGATIVE_FILTER = {}
-_ANDROID_NEGATIVE_FILTER['com.google.android.apps.chrome'] = (
+_ANDROID_NEGATIVE_FILTER['chrome'] = (
_NEGATIVE_FILTER + [
# TODO(chrisgao): fix hang of tab crash test on android.
'ChromeDriverTest.testTabCrash',
@@ -116,18 +117,20 @@ _ANDROID_NEGATIVE_FILTER['com.google.android.apps.chrome'] = (
'ChromeDriverTest.testShouldHandleNewWindowLoadingProperly',
]
)
-_ANDROID_NEGATIVE_FILTER['com.android.chrome'] = (
- _ANDROID_NEGATIVE_FILTER['com.google.android.apps.chrome'])
-_ANDROID_NEGATIVE_FILTER['com.chrome.beta'] = (
- _ANDROID_NEGATIVE_FILTER['com.google.android.apps.chrome'])
-_ANDROID_NEGATIVE_FILTER['org.chromium.chrome.testshell'] = (
- _ANDROID_NEGATIVE_FILTER['com.google.android.apps.chrome'] + [
+_ANDROID_NEGATIVE_FILTER['chrome_stable'] = (
+ _ANDROID_NEGATIVE_FILTER['chrome'])
+_ANDROID_NEGATIVE_FILTER['chrome_beta'] = (
+ _ANDROID_NEGATIVE_FILTER['chrome'])
+_ANDROID_NEGATIVE_FILTER['chromium_test_shell'] = (
+ _ANDROID_NEGATIVE_FILTER['chrome'] + [
# ChromiumTestShell doesn't support multiple tabs.
'ChromeDriverTest.testGetWindowHandles',
'ChromeDriverTest.testSwitchToWindow',
'ChromeDriverTest.testShouldHandleNewWindowLoadingProperly',
]
)
+_ANDROID_NEGATIVE_FILTER['chromedriver_webview_shell'] = (
+ _ANDROID_NEGATIVE_FILTER['chromium_test_shell'])
class ChromeDriverBaseTest(unittest.TestCase):
@@ -147,9 +150,21 @@ class ChromeDriverBaseTest(unittest.TestCase):
def CreateDriver(self, server_url=None, **kwargs):
if server_url is None:
server_url = _CHROMEDRIVER_SERVER_URL
+
+ android_package = None
+ android_activity = None
+ android_process = None
+ if _ANDROID_PACKAGE_KEY:
+ android_package = constants.PACKAGE_INFO[_ANDROID_PACKAGE_KEY].package
+ if _ANDROID_PACKAGE_KEY == 'chromedriver_webview_shell':
+ android_activity = constants.PACKAGE_INFO[_ANDROID_PACKAGE_KEY].activity
+ android_process = '%s:main' % android_package
+
driver = chromedriver.ChromeDriver(server_url,
chrome_binary=_CHROME_BINARY,
- android_package=_ANDROID_PACKAGE,
+ android_package=android_package,
+ android_activity=android_activity,
+ android_process=android_process,
**kwargs)
self._drivers += [driver]
return driver
@@ -163,7 +178,7 @@ class ChromeDriverTest(ChromeDriverBaseTest):
ChromeDriverTest._http_server = webserver.WebServer(
chrome_paths.GetTestData())
ChromeDriverTest._sync_server = webserver.SyncWebServer()
- if _ANDROID_PACKAGE:
+ if _ANDROID_PACKAGE_KEY:
ChromeDriverTest._adb = android_commands.AndroidCommands()
http_host_port = ChromeDriverTest._http_server._server.server_port
sync_host_port = ChromeDriverTest._sync_server._server.server_port
@@ -173,7 +188,7 @@ class ChromeDriverTest(ChromeDriverBaseTest):
@staticmethod
def GlobalTearDown():
- if _ANDROID_PACKAGE:
+ if _ANDROID_PACKAGE_KEY:
forwarder.Forwarder.UnmapAllDevicePorts(ChromeDriverTest._adb)
ChromeDriverTest._http_server.Shutdown()
@@ -833,13 +848,19 @@ if __name__ == '__main__':
help=('Filter for specifying what tests to run, "*" will run all. E.g., '
'*testStartStop'))
parser.add_option(
- '', '--android-package', help='Android package name')
+ '', '--android-package',
+ help=('Android package key. Possible values: ' +
+ str(_ANDROID_NEGATIVE_FILTER.keys())))
options, args = parser.parse_args()
if not options.chromedriver or not os.path.exists(options.chromedriver):
parser.error('chromedriver is required or the given path is invalid.' +
'Please run "%s --help" for help' % __file__)
+ if (options.android_package and
+ options.android_package not in _ANDROID_NEGATIVE_FILTER):
+ parser.error('Invalid --android-package')
+
chromedriver_server = server.Server(os.path.abspath(options.chromedriver),
options.log_path)
global _CHROMEDRIVER_SERVER_URL
@@ -854,12 +875,12 @@ if __name__ == '__main__':
else:
_CHROME_BINARY = None
- global _ANDROID_PACKAGE
- _ANDROID_PACKAGE = options.android_package
+ global _ANDROID_PACKAGE_KEY
+ _ANDROID_PACKAGE_KEY = options.android_package
if options.filter == '*':
- if _ANDROID_PACKAGE:
- negative_filter = _ANDROID_NEGATIVE_FILTER[_ANDROID_PACKAGE]
+ if _ANDROID_PACKAGE_KEY:
+ negative_filter = _ANDROID_NEGATIVE_FILTER[_ANDROID_PACKAGE_KEY]
else:
negative_filter = _GetDesktopNegativeFilter(options.chrome_version)
options.filter = '*-' + ':__main__.'.join([''] + negative_filter)
« no previous file with comments | « chrome/test/chromedriver/test/run_java_tests.py ('k') | chrome/test/chromedriver/test/webview_shell/java/AndroidManifest.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698