2018-03-01 13:41:35 -05:00
|
|
|
# Copyright 2017 The Bazel Authors. All rights reserved.
|
|
|
|
#
|
|
|
|
# Licensed under the Apache License, Version 2.0 (the "License");
|
|
|
|
# you may not use this file except in compliance with the License.
|
|
|
|
# You may obtain a copy of the License at
|
|
|
|
#
|
|
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
|
|
#
|
|
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
|
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
|
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
|
|
# See the License for the specific language governing permissions and
|
|
|
|
# limitations under the License.
|
|
|
|
|
|
|
|
"""Runs ts_api_guardian
|
|
|
|
"""
|
2018-08-07 19:15:45 -04:00
|
|
|
|
2019-10-25 04:13:43 -04:00
|
|
|
load("@build_bazel_rules_nodejs//:index.bzl", "nodejs_binary", "nodejs_test")
|
2018-03-01 13:41:35 -05:00
|
|
|
|
fix(core): make QueryList implement Iterable in the type system (#33536)
Originally, QueryList implemented Iterable and provided a Symbol.iterator
on its prototype. This caused issues with tree-shaking, so QueryList was
refactored and the Symbol.iterator added in its constructor instead. As
part of this change, QueryList no longer implemented Iterable directly.
Unfortunately, this meant that QueryList was no longer assignable to
Iterable or, consequently, NgIterable. NgIterable is used for NgFor's input,
so this meant that QueryList was not usable (in a type sense) for NgFor
iteration. View Engine's template type checking would not catch this, but
Ivy's did.
As a fix, this commit adds the declaration (but not the implementation) of
the Symbol.iterator function back to QueryList. This has no runtime effect,
so it doesn't affect tree-shaking of QueryList, but it ensures that
QueryList is assignable to NgIterable and thus usable with NgFor.
Fixes #29842
PR Close #33536
2019-11-01 15:31:56 -04:00
|
|
|
COMMON_MODULE_IDENTIFIERS = ["angular", "jasmine", "protractor", "Symbol"]
|
2018-03-01 13:41:35 -05:00
|
|
|
|
2018-10-28 14:05:20 -04:00
|
|
|
def ts_api_guardian_test(
|
|
|
|
name,
|
|
|
|
golden,
|
|
|
|
actual,
|
|
|
|
data = [],
|
2020-07-24 12:49:19 -04:00
|
|
|
strip_export_pattern = [],
|
2018-10-28 14:05:20 -04:00
|
|
|
allow_module_identifiers = COMMON_MODULE_IDENTIFIERS,
|
|
|
|
use_angular_tag_rules = True,
|
|
|
|
**kwargs):
|
2018-08-07 19:15:45 -04:00
|
|
|
"""Runs ts_api_guardian
|
|
|
|
"""
|
|
|
|
data += [
|
2019-04-19 02:37:01 -04:00
|
|
|
# Locally we need to add the TS build target
|
2019-04-23 08:47:18 -04:00
|
|
|
# But it will replaced to @npm//ts-api-guardian when publishing
|
2018-08-28 14:04:21 -04:00
|
|
|
"@angular//tools/ts-api-guardian:lib",
|
2021-01-08 07:42:32 -05:00
|
|
|
# BEGIN-INTERNAL
|
2019-04-23 13:12:00 -04:00
|
|
|
"@angular//tools/ts-api-guardian:bin",
|
2021-01-08 07:42:32 -05:00
|
|
|
# END-INTERNAL
|
2019-04-26 03:17:32 -04:00
|
|
|
# The below are required during runtime
|
|
|
|
"@npm//chalk",
|
|
|
|
"@npm//diff",
|
|
|
|
"@npm//minimist",
|
|
|
|
"@npm//typescript",
|
2018-08-07 19:15:45 -04:00
|
|
|
]
|
2018-03-01 13:41:35 -05:00
|
|
|
|
2018-08-07 19:15:45 -04:00
|
|
|
args = [
|
|
|
|
# Needed so that node doesn't walk back to the source directory.
|
|
|
|
# From there, the relative imports would point to .ts files.
|
|
|
|
"--node_options=--preserve-symlinks",
|
2021-01-08 07:42:32 -05:00
|
|
|
# Since version 3, monkey-patch the implementation of require() in NodeJS is opt-in
|
|
|
|
# https://github.com/bazelbuild/rules_nodejs/wiki#--bazel_patch_module_resolver-now-defaults-to-false-2324
|
|
|
|
"--bazel_patch_module_resolver",
|
2018-08-07 19:15:45 -04:00
|
|
|
]
|
2018-09-19 07:40:21 -04:00
|
|
|
|
|
|
|
for i in strip_export_pattern:
|
2020-07-24 12:49:19 -04:00
|
|
|
# Quote the regexp before passing it via the command line.
|
|
|
|
quoted_pattern = "\"%s\"" % i
|
|
|
|
args += ["--stripExportPattern", quoted_pattern]
|
2018-09-19 07:40:21 -04:00
|
|
|
|
2018-09-07 11:31:45 -04:00
|
|
|
for i in allow_module_identifiers:
|
2018-08-07 19:15:45 -04:00
|
|
|
args += ["--allowModuleIdentifiers", i]
|
2018-03-01 13:41:35 -05:00
|
|
|
|
2018-10-28 14:05:20 -04:00
|
|
|
if use_angular_tag_rules:
|
|
|
|
args += ["--useAngularTagRules"]
|
|
|
|
|
2018-08-07 19:15:45 -04:00
|
|
|
nodejs_test(
|
|
|
|
name = name,
|
|
|
|
data = data,
|
2021-01-08 07:42:32 -05:00
|
|
|
entry_point = Label("@angular//tools/ts-api-guardian:bin/ts-api-guardian"),
|
2020-02-26 12:09:35 -05:00
|
|
|
tags = kwargs.pop("tags", []) + ["api_guard"],
|
2018-08-07 19:15:45 -04:00
|
|
|
templated_args = args + ["--verify", golden, actual],
|
|
|
|
**kwargs
|
|
|
|
)
|
2018-03-01 13:41:35 -05:00
|
|
|
|
2018-08-07 19:15:45 -04:00
|
|
|
nodejs_binary(
|
|
|
|
name = name + ".accept",
|
2018-08-14 19:18:26 -04:00
|
|
|
testonly = True,
|
2018-08-07 19:15:45 -04:00
|
|
|
data = data,
|
2021-01-08 07:42:32 -05:00
|
|
|
entry_point = Label("@angular//tools/ts-api-guardian:bin/ts-api-guardian"),
|
2020-02-26 12:09:35 -05:00
|
|
|
tags = kwargs.pop("tags", []) + ["api_guard"],
|
2018-08-07 19:15:45 -04:00
|
|
|
templated_args = args + ["--out", golden, actual],
|
|
|
|
**kwargs
|
|
|
|
)
|
2020-02-25 16:05:49 -05:00
|
|
|
|
|
|
|
def ts_api_guardian_test_npm_package(
|
|
|
|
name,
|
|
|
|
goldenDir,
|
|
|
|
actualDir,
|
|
|
|
data = [],
|
2020-07-24 12:49:19 -04:00
|
|
|
strip_export_pattern = ["^ɵ(?!ɵdefineInjectable|ɵinject|ɵInjectableDef)"],
|
2020-02-25 16:05:49 -05:00
|
|
|
allow_module_identifiers = COMMON_MODULE_IDENTIFIERS,
|
|
|
|
use_angular_tag_rules = True,
|
|
|
|
**kwargs):
|
|
|
|
"""Runs ts_api_guardian
|
|
|
|
"""
|
|
|
|
data += [
|
|
|
|
# Locally we need to add the TS build target
|
|
|
|
# But it will replaced to @npm//ts-api-guardian when publishing
|
|
|
|
"@angular//tools/ts-api-guardian:lib",
|
|
|
|
"@angular//tools/ts-api-guardian:bin",
|
|
|
|
# The below are required during runtime
|
|
|
|
"@npm//chalk",
|
|
|
|
"@npm//diff",
|
|
|
|
"@npm//minimist",
|
|
|
|
"@npm//typescript",
|
|
|
|
]
|
|
|
|
|
|
|
|
args = [
|
|
|
|
# Needed so that node doesn't walk back to the source directory.
|
|
|
|
# From there, the relative imports would point to .ts files.
|
|
|
|
"--node_options=--preserve-symlinks",
|
|
|
|
# We automatically discover the enpoints for our NPM package.
|
|
|
|
"--autoDiscoverEntrypoints",
|
|
|
|
]
|
|
|
|
|
|
|
|
for i in strip_export_pattern:
|
2020-07-24 12:49:19 -04:00
|
|
|
# Quote the regexp before passing it via the command line.
|
|
|
|
quoted_pattern = "\"%s\"" % i
|
|
|
|
args += ["--stripExportPattern", quoted_pattern]
|
2020-02-25 16:05:49 -05:00
|
|
|
|
|
|
|
for i in allow_module_identifiers:
|
|
|
|
args += ["--allowModuleIdentifiers", i]
|
|
|
|
|
|
|
|
if use_angular_tag_rules:
|
|
|
|
args += ["--useAngularTagRules"]
|
|
|
|
|
|
|
|
nodejs_test(
|
|
|
|
name = name,
|
|
|
|
data = data,
|
|
|
|
entry_point = "@angular//tools/ts-api-guardian:bin/ts-api-guardian",
|
2020-02-26 12:09:35 -05:00
|
|
|
tags = kwargs.pop("tags", []) + ["api_guard"],
|
2020-03-12 07:36:28 -04:00
|
|
|
templated_args = args + ["--autoDiscoverEntrypoints", "--verifyDir", goldenDir, "--rootDir", "$(rlocation %s)" % actualDir],
|
2020-02-25 16:05:49 -05:00
|
|
|
**kwargs
|
|
|
|
)
|
|
|
|
|
|
|
|
nodejs_binary(
|
|
|
|
name = name + ".accept",
|
|
|
|
testonly = True,
|
|
|
|
data = data,
|
|
|
|
entry_point = "@angular//tools/ts-api-guardian:bin/ts-api-guardian",
|
2020-02-26 12:09:35 -05:00
|
|
|
tags = kwargs.pop("tags", []) + ["api_guard"],
|
2020-03-12 07:36:28 -04:00
|
|
|
templated_args = args + ["--autoDiscoverEntrypoints", "--outDir", goldenDir, "--rootDir", "$(rlocation %s)" % actualDir],
|
2020-02-25 16:05:49 -05:00
|
|
|
**kwargs
|
|
|
|
)
|