diff options
author | kelvinp <kelvinp@chromium.org> | 2015-01-16 12:09:49 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-01-16 20:11:04 +0000 |
commit | 70d6474255bc232ebc3e0c77dd0e7063b491619d (patch) | |
tree | 0de6428942576cd70eabddea522a6643bcb0a9e7 /testing | |
parent | 25c1109a382a17fa1401c2b596afd987296e4264 (diff) | |
download | chromium_src-70d6474255bc232ebc3e0c77dd0e7063b491619d.zip chromium_src-70d6474255bc232ebc3e0c77dd0e7063b491619d.tar.gz chromium_src-70d6474255bc232ebc3e0c77dd0e7063b491619d.tar.bz2 |
Enable It2me Browser Test on Chromoting waterfall
Summary of changes:
1. Copies the browser test JavaScript files to
remoting/browser_test_resources so we can copy the entire
directory on swarming bots.
2. Modifies browser_test_launcher.py to install the it2me
native messaging host manifests.
3. Adds the It2meBrowserTest on to the
browser_test_commands_linux.txt
BUG=446342
Review URL: https://codereview.chromium.org/856653002
Cr-Commit-Position: refs/heads/master@{#311940}
Diffstat (limited to 'testing')
-rw-r--r-- | testing/chromoting/browser_test_commands_linux.txt | 4 | ||||
-rw-r--r-- | testing/chromoting/browser_tests_launcher.py | 28 | ||||
-rw-r--r-- | testing/chromoting/chromoting_integration_tests.isolate | 6 |
3 files changed, 25 insertions, 13 deletions
diff --git a/testing/chromoting/browser_test_commands_linux.txt b/testing/chromoting/browser_test_commands_linux.txt index 2f5f2be..d6f2bf5 100644 --- a/testing/chromoting/browser_test_commands_linux.txt +++ b/testing/chromoting/browser_test_commands_linux.txt @@ -3,4 +3,6 @@ /usr/bin/python ../xvfb.py #PROD_DIR# #PROD_DIR#/browser_tests --gtest_filter=RemoteDesktopBrowserTest.MANUAL_Auth --run-manual --ui-test-action-timeout=100000 --webapp-unpacked=#PROD_DIR#/remoting/remoting.webapp --extension-name=Chromoting --accounts-file=../../remoting/tools/internal/test_accounts.json --account-type=gafyd /usr/bin/python ../xvfb.py #PROD_DIR# #PROD_DIR#/browser_tests --gtest_filter=RemoteDesktopBrowserTest.MANUAL_Auth --run-manual --ui-test-action-timeout=100000 --webapp-unpacked=#PROD_DIR#/remoting/remoting.webapp --extension-name=Chromoting --accounts-file=../../remoting/tools/internal/test_accounts.json --account-type=non-gmail /usr/bin/python ../xvfb.py #PROD_DIR# #PROD_DIR#/browser_tests --gtest_filter=RemoteDesktopBrowserTest.MANUAL_Launch --run-manual --ui-test-action-timeout=100000 --webapp-unpacked=#PROD_DIR#/remoting/remoting.webapp.v2 --extension-name=Chromoting -/usr/bin/python ../xvfb.py #PROD_DIR# #PROD_DIR#/browser_tests --gtest_filter=Me2MeBrowserTest.MANUAL_Me2Me_Connect_Local_Host --run-manual --ui-test-action-timeout=100000 --webapp-unpacked=#PROD_DIR#/remoting/remoting.webapp --extension-name=Chromoting --accounts-file=../../remoting/tools/internal/test_accounts.json --account-type=gmail --me2me-pin=123456 --override-user-data-dir=/tmp/chromoting_test_profile
\ No newline at end of file +/usr/bin/python ../xvfb.py #PROD_DIR# #PROD_DIR#/browser_tests --gtest_filter=Me2MeBrowserTest.MANUAL_Me2Me_Connect_Local_Host --run-manual --ui-test-action-timeout=100000 --webapp-unpacked=#PROD_DIR#/remoting/remoting.webapp --extension-name=Chromoting --accounts-file=../../remoting/tools/internal/test_accounts.json --account-type=gmail --me2me-pin=123456 --override-user-data-dir=/tmp/chromoting_test_profile +/usr/bin/python ../xvfb.py #PROD_DIR# #PROD_DIR#/browser_tests --gtest_filter=It2MeBrowserTest.MANUAL_Connect --run-manual --ui-test-action-timeout=100000 --webapp-unpacked=#PROD_DIR#/remoting/remoting.webapp --extension-name=Chromoting --accounts-file=../../remoting/tools/internal/test_accounts.json --account-type=gmail --override-user-data-dir=/tmp/chromoting_test_profile +/usr/bin/python ../xvfb.py #PROD_DIR# #PROD_DIR#/browser_tests --gtest_filter=It2MeBrowserTest.MANUAL_InvalidAccessCode --run-manual --ui-test-action-timeout=100000 --webapp-unpacked=#PROD_DIR#/remoting/remoting.webapp --extension-name=Chromoting --accounts-file=../../remoting/tools/internal/test_accounts.json --account-type=gmail --override-user-data-dir=/tmp/chromoting_test_profile
\ No newline at end of file diff --git a/testing/chromoting/browser_tests_launcher.py b/testing/chromoting/browser_tests_launcher.py index be66955..dab4c97 100644 --- a/testing/chromoting/browser_tests_launcher.py +++ b/testing/chromoting/browser_tests_launcher.py @@ -80,7 +80,8 @@ def TestCleanUp(user_profile_dir): shutil.rmtree(user_profile_dir) -def InitialiseTestMachineForLinux(cfg_file, manifest_file, user_profile_dir): +def InitialiseTestMachineForLinux(cfg_file, me2me_manifest_file, + it2me_manifest_file, user_profile_dir): """Sets up a Linux machine for connect-to-host browser-tests. Copy over me2me host-config and manifest files to expected locations. @@ -89,8 +90,8 @@ def InitialiseTestMachineForLinux(cfg_file, manifest_file, user_profile_dir): Its name is expected to have a hash that is specific to a machine. When a user launches the remoting web-app, the native-message host process is - started. For this to work, the manifest file for me2me host is expected to be - in a specific folder under the user-profile dir. + started. For this to work, the manifest file for me2me host and it2me host is + expected to be in a specific folder under the user-profile dir. This function performs both the above tasks. @@ -102,7 +103,8 @@ def InitialiseTestMachineForLinux(cfg_file, manifest_file, user_profile_dir): Args: cfg_file: location of test account's host-config file. - manifest_file: location of me2me host manifest file. + me2me_manifest_file: location of me2me host manifest file. + it2me_manifest_file: location of it2me host manifest file. user_profile_dir: user-profile-dir to be used by the connect-to-host tests. """ @@ -129,10 +131,12 @@ def InitialiseTestMachineForLinux(cfg_file, manifest_file, user_profile_dir): shutil.rmtree(user_profile_dir) os.makedirs(native_messaging_folder) - manifest_file_src = os.path.join(os.getcwd(), manifest_file) - manifest_file_dest = ( - os.path.join(native_messaging_folder, os.path.basename(manifest_file))) - shutil.copyfile(manifest_file_src, manifest_file_dest) + manifest_files = [me2me_manifest_file, it2me_manifest_file] + for manifest_file in manifest_files: + manifest_file_src = os.path.join(os.getcwd(), manifest_file) + manifest_file_dest = ( + os.path.join(native_messaging_folder, os.path.basename(manifest_file))) + shutil.copyfile(manifest_file_src, manifest_file_dest) # Finally, start chromoting host. RunCommandInSubProcess(CHROMOTING_HOST_PATH + ' --start') @@ -146,16 +150,18 @@ def main(): help='path to folder having product and test binaries.') parser.add_argument('-c', '--cfg_file', help='path to test host config file.') - parser.add_argument('-m', '--manifest_file', + parser.add_argument('--me2me_manifest_file', help='path to me2me host manifest file.') + parser.add_argument('--it2me_manifest_file', + help='path to it2me host manifest file.') parser.add_argument( '-u', '--user_profile_dir', help='path to user-profile-dir, used by connect-to-host tests.') args = parser.parse_args() - InitialiseTestMachineForLinux(args.cfg_file, args.manifest_file, - args.user_profile_dir) + InitialiseTestMachineForLinux(args.cfg_file, args.me2me_manifest_file, + args.it2me_manifest_file, args.user_profile_dir) with open(args.commands_file) as f: for line in f: diff --git a/testing/chromoting/chromoting_integration_tests.isolate b/testing/chromoting/chromoting_integration_tests.isolate index 83eb440..be04bd6 100644 --- a/testing/chromoting/chromoting_integration_tests.isolate +++ b/testing/chromoting/chromoting_integration_tests.isolate @@ -13,8 +13,10 @@ '<(PRODUCT_DIR)', '--cfg_file', '../../remoting/tools/internal/test-account-host-config.json', - '--manifest_file', + '--me2me_manifest_file', '<(PRODUCT_DIR)/remoting/com.google.chrome.remote_desktop.json', + '--it2me_manifest_file', + '<(PRODUCT_DIR)/remoting/com.google.chrome.remote_assistance.json', '--user_profile_dir', '/tmp/chromoting_test_profile', ], @@ -27,6 +29,7 @@ '<(PRODUCT_DIR)/nacl_irt_x86_64.nexe', '../../remoting/tools/internal/test-account-host-config.json', '<(PRODUCT_DIR)/remoting/com.google.chrome.remote_desktop.json', + '<(PRODUCT_DIR)/remoting/com.google.chrome.remote_assistance.json', ], }, }], @@ -44,6 +47,7 @@ '<(PRODUCT_DIR)/browser_tests<(EXECUTABLE_SUFFIX)', '<(PRODUCT_DIR)/remoting/remoting.webapp/', '<(PRODUCT_DIR)/remoting/remoting.webapp.v2/', + '<(PRODUCT_DIR)/remoting/browser_test_resources/', '<(PRODUCT_DIR)/resources.pak', '../../remoting/tools/internal/test_accounts.json', ], |