Example #1
0
def _checkWithNodeScript(input_api,
                         output_api,
                         script_path,
                         script_arguments=None):  # pylint: disable=invalid-name
    original_sys_path = sys.path
    try:
        sys.path = sys.path + [
            input_api.os_path.join(input_api.PresubmitLocalPath(), "scripts")
        ]
        import local_node
    finally:
        sys.path = original_sys_path

    node_path = local_node.node_path()

    if script_arguments is None:
        script_arguments = []

    process = input_api.subprocess.Popen([node_path, script_path] +
                                         script_arguments,
                                         stdout=input_api.subprocess.PIPE,
                                         stderr=input_api.subprocess.STDOUT)
    out, _ = process.communicate()

    if process.returncode != 0:
        return [output_api.PresubmitError(out)]
    return [output_api.PresubmitNotifyResult(out)]
Example #2
0
def _CheckFormat(input_api, output_api):
    def popen(args):
        return input_api.subprocess.Popen(args=args,
                                          stdout=input_api.subprocess.PIPE,
                                          stderr=input_api.subprocess.STDOUT)

    affected_files = _getAffectedJSFiles(input_api)
    if len(affected_files) == 0:
        return []
    original_sys_path = sys.path
    try:
        sys.path = sys.path + [
            input_api.os_path.join(input_api.PresubmitLocalPath(), "scripts")
        ]
        import local_node
    finally:
        sys.path = original_sys_path

    ignore_files = []
    eslint_ignore_path = input_api.os_path.join(input_api.PresubmitLocalPath(),
                                                '.eslintignore')
    with open(eslint_ignore_path, 'r') as ignore_manifest:
        for line in ignore_manifest:
            ignore_files.append(line.strip())
    formattable_files = [
        affected_file for affected_file in affected_files if all(
            ignore_file not in affected_file for ignore_file in ignore_files)
    ]

    check_formatting_process = popen(
        ['git', 'cl', 'format', '--js', '--dry-run'] + formattable_files)
    check_formatting_process.communicate()
    if check_formatting_process.returncode == 0:
        return []

    format_args = ['git', 'cl', 'format', '--js'] + formattable_files
    format_process = popen(format_args)
    format_out, _ = format_process.communicate()
    if format_process.returncode != 0:
        return [output_api.PresubmitError(format_out)]

    # Use eslint to autofix the braces.
    # Also fix semicolon to avoid confusing clang-format.
    eslint_process = popen([
        local_node.node_path(),
        local_node.eslint_path(), '--no-eslintrc', '--fix', '--env=es6',
        '--rule={"curly": [2, "multi-or-nest", "consistent"], "semi": 2}'
    ] + affected_files)
    eslint_process.communicate()

    # Need to run clang-format again to align the braces
    popen(format_args).communicate()

    return [
        output_api.PresubmitError(
            "ERROR: Found formatting violations in third_party/WebKit/Source/devtools.\n"
            "Ran clang-format on diff\n"
            "Use git status to check the formatting changes"),
        output_api.PresubmitError(format_out),
    ]
Example #3
0
def js_lint(files_list=None):
    eslint_errors_found = False

    if files_list is None:
        files_list = [devtools_frontend_path]
    files_list = [file_name for file_name in files_list if not file_name.endswith(".eslintrc.js")]

    eslintconfig_path = path.join(devtools_path, ".eslintrc.js")
    eslintignore_path = path.join(devtools_path, ".eslintignore")
    exec_command = [
        local_node.node_path(),
        local_node.eslint_path(),
        "--config",
        to_platform_path_exact(eslintconfig_path),
        "--ignore-path",
        to_platform_path_exact(eslintignore_path),
    ] + files_list

    eslint_proc = popen(exec_command, cwd=devtools_path)
    (eslint_proc_out, _) = eslint_proc.communicate()
    if eslint_proc.returncode != 0:
        eslint_errors_found = True
    else:
        print("eslint exited successfully")

    print(eslint_proc_out)
    return eslint_errors_found
def _checkWithNodeScript(input_api, output_api, script_path):
    original_sys_path = sys.path
    try:
        sys.path = sys.path + [input_api.os_path.join(input_api.PresubmitLocalPath(), "scripts")]
        import local_node
    finally:
        sys.path = original_sys_path

    node_path = local_node.node_path()

    process = input_api.subprocess.Popen(
        [node_path, script_path], stdout=input_api.subprocess.PIPE, stderr=input_api.subprocess.STDOUT)
    out, _ = process.communicate()

    if process.returncode != 0:
        return [output_api.PresubmitError(out)]
    return [output_api.PresubmitNotifyResult(out)]
Example #5
0
def run_tests():
    karma_errors_found = False

    karmaconfig_path = os.path.join(devtools_path, "karma.conf.js")
    exec_command = [local_node.node_path(), local_node.karma_path(), "start", to_platform_path_exact(karmaconfig_path)]

    env = {'NODE_PATH': local_node.node_modules_path()}
    if (chrome_binary is not None):
        env['CHROME_BIN'] = chrome_binary

    karma_proc = popen(exec_command, cwd=devtools_path, env=env)

    (karma_proc_out, _) = karma_proc.communicate()
    if karma_proc.returncode != 0:
        karma_errors_found = True
    else:
        print("Karma exited successfully")

    print(karma_proc_out)
    return karma_errors_found