| Index: testing/chromoting/browser_tests_launcher.py
|
| diff --git a/testing/chromoting/browser_tests_launcher.py b/testing/chromoting/browser_tests_launcher.py
|
| index 8f4bba19f938692d2ec188005e0e31f00abbe166..434728716ee7c48159d052140d47ac305adad02b 100644
|
| --- a/testing/chromoting/browser_tests_launcher.py
|
| +++ b/testing/chromoting/browser_tests_launcher.py
|
| @@ -5,36 +5,112 @@
|
|
|
| """Utility script to launch browser-tests on the Chromoting bot."""
|
| import argparse
|
| +import hashlib
|
| +import os
|
| +from os.path import expanduser
|
| +import shutil
|
| +import socket
|
| import subprocess
|
|
|
| -PROD_DIR_ID = '$(PROD_DIR)'
|
| +BROWSER_TEST_ID = 'browser_tests'
|
| +PROD_DIR_ID = '#PROD_DIR#'
|
| +HOST_HASH_VALUE = hashlib.md5(socket.gethostname()).hexdigest()
|
| +SUCCESS_INDICATOR = 'SUCCESS: all tests passed.'
|
| +NATIVE_MESSAGING_DIR = 'NativeMessagingHosts'
|
| +CRD_ID = 'chrome-remote-desktop' # Used in a few file/folder names
|
|
|
|
|
| def LaunchCommand(command):
|
| -
|
| cmd_line = [command]
|
| try:
|
| p = subprocess.Popen(cmd_line, stdout=subprocess.PIPE, shell=True)
|
| results, err = p.communicate()
|
| - if 'SUCCESS: all tests passed.' not in results:
|
| - raise Exception('Test failed\n%s\n%s' % (results, err))
|
| + # Check that the test passed.
|
| + if SUCCESS_INDICATOR not in results:
|
| + raise Exception(
|
| + 'Test failed. Command:%s\nResults:%s\nError:%s\n' %
|
| + (command, results, err))
|
| except subprocess.CalledProcessError, e:
|
| raise Exception('Exception %s running command %s' % (e, command))
|
| else:
|
| print results
|
|
|
|
|
| -def main():
|
| +def InitialiseTestMachineForLinux(cfg_file, 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.
|
| + By default, the Linux me2me host expects the host-config file to be under
|
| + $HOME/.config/chrome-remote-desktop
|
| + 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.
|
| +
|
| + This function performs both the above tasks.
|
| +
|
| + TODO(anandc):
|
| + Once we have Linux machines in the swarming lab already installed with the
|
| + me2me host, this function should also perform the step of starting the host.
|
| + That is gated on this CL: https://chromereviews.googleplex.com/123957013/, and
|
| + then having base images in the chrome-labs be updated with it.
|
| +
|
| + Args:
|
| + cfg_file: location of test account's host-config file.
|
| + manifest_file: location of me2me host manifest file.
|
| + user_profile_dir: user-profile-dir to be used by the connect-to-host tests.
|
| + """
|
| +
|
| + # First get home directory on current machine.
|
| + home_dir = expanduser('~')
|
| + default_config_file_location = os.path.join(home_dir, '.config', CRD_ID)
|
| + if os.path.exists(default_config_file_location):
|
| + shutil.rmtree(default_config_file_location)
|
| + os.makedirs(default_config_file_location)
|
|
|
| + # Copy over test host-config to expected location, with expected file-name.
|
| + # The file-name should contain a hash-value that is machine-specific.
|
| + default_config_file_name = 'host#%s.json' % HOST_HASH_VALUE
|
| + config_file_src = os.path.join(os.getcwd(), cfg_file)
|
| + shutil.copyfile(
|
| + config_file_src,
|
| + os.path.join(default_config_file_location, default_config_file_name))
|
| +
|
| + # Next, create a user-profile dir, and place the me2me manifest.json file in
|
| + # the expected location for native-messating-host to work properly.
|
| + native_messaging_folder = os.path.join(user_profile_dir, NATIVE_MESSAGING_DIR)
|
| +
|
| + if os.path.exists(native_messaging_folder):
|
| + shutil.rmtree(native_messaging_folder)
|
| + 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)
|
| +
|
| +
|
| +def main():
|
| parser = argparse.ArgumentParser()
|
| - parser.add_argument('-f', '--file',
|
| + parser.add_argument('-f', '--commands_file',
|
| help='path to file listing commands to be launched.')
|
| parser.add_argument('-p', '--prod_dir',
|
| 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',
|
| + help='path to me2me 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()
|
|
|
| - with open(args.file) as f:
|
| + InitialiseTestMachineForLinux(args.cfg_file, args.manifest_file,
|
| + args.user_profile_dir)
|
| +
|
| + with open(args.commands_file) as f:
|
| for line in f:
|
| # Replace the PROD_DIR value in the command-line with
|
| # the passed in value.
|
|
|