mirror of
https://github.com/angular/angular.git
synced 2026-09-14 13:54:52 +08:00
Andrew Scott
dedac8d3f7
In order to test components and services which depend on router navigations, such as `ActivatedRoute` instances, tests currently need to provide a fair bit of boilerplate _or_ they can set up a stub for `ActivatedRoute` and list it in the `providers` to override it in `TestBed`. This approach of stubbing the `ActivatedRoute` creates a situation that can easily cause the test to break. The stub often only mocks out the dependencies that the component/service _currently_ needs. This dependencies might change over time and break the test in an unexpected way. In addition, it is difficult to get the structure of `ActivatedRoute` exactly correct. This change will allow unit tests to quickly set up routes, trigger real navigations in the Router, and get instances of component's to test along with real instances of `ActivatedRoute`. This all comes without needing to know that the component depends on `ActivatedRoute` at all. This becomes more important when considering that a component may be refactored in the future to use `@Input` rather than access data on the `ActivatedRoute` instance (see #18967). Tests which mock out `ActivatedRoute` would all break, but those which use `navigateForTest` would continue to work without needing any updates. resolves #15779 resolves #48608 PR Close #48552
107 lines
3.1 KiB
Python
107 lines
3.1 KiB
Python
# Copyright Google LLC All Rights Reserved.
|
|
#
|
|
# Use of this source code is governed by an MIT-style license that can be
|
|
# found in the LICENSE file at https://angular.io/license
|
|
"""Packages published to npm"""
|
|
|
|
def to_package_label(package_name):
|
|
"""Get a label corresponding to the npm_package target for the package name"""
|
|
if package_name == "angular-in-memory-web-api":
|
|
return "//packages/misc/angular-in-memory-web-api:npm_package"
|
|
|
|
return "//packages/{package_name}:npm_package".format(package_name = package_name.replace("@angular/", ""))
|
|
|
|
def _exclude_pkgs(packages, *args):
|
|
modified_packages = packages[:]
|
|
for pkg in args:
|
|
modified_packages.remove(pkg)
|
|
return modified_packages
|
|
|
|
# All framework packages published to NPM.
|
|
ALL_PACKAGES = [
|
|
"@angular/animations",
|
|
"@angular/bazel",
|
|
"@angular/benchpress",
|
|
"@angular/common",
|
|
"@angular/compiler",
|
|
"@angular/compiler-cli",
|
|
"@angular/core",
|
|
"@angular/elements",
|
|
"@angular/forms",
|
|
"@angular/language-service",
|
|
"@angular/localize",
|
|
"@angular/platform-browser",
|
|
"@angular/platform-browser-dynamic",
|
|
"@angular/platform-server",
|
|
"@angular/router",
|
|
"@angular/service-worker",
|
|
"@angular/upgrade",
|
|
"angular-in-memory-web-api",
|
|
"zone.js",
|
|
]
|
|
|
|
# Packages used by integration tests
|
|
INTEGRATION_PACKAGES = _exclude_pkgs(ALL_PACKAGES, "angular-in-memory-web-api")
|
|
|
|
# Packages used by example e2e tests
|
|
AIO_EXAMPLE_PACKAGES = _exclude_pkgs(ALL_PACKAGES, "@angular/benchpress")
|
|
|
|
# Package names under //packages that have `files_for_docgen` targets
|
|
# including files needed for AIO doc generation.
|
|
DOCS_ENTRYPOINTS = [
|
|
"animations",
|
|
"animations/browser",
|
|
"animations/browser/testing",
|
|
"common",
|
|
"common/http",
|
|
"common/http/testing",
|
|
"common/testing",
|
|
"common/upgrade",
|
|
"compiler",
|
|
"core",
|
|
"core/src/compiler",
|
|
"core/src/di/interface",
|
|
"core/src/interface",
|
|
"core/src/reflection",
|
|
"core/src/util",
|
|
"core/testing",
|
|
"elements",
|
|
"examples/common",
|
|
"examples/core",
|
|
"examples/core/di/ts/forward_ref",
|
|
"examples/core/testing/ts",
|
|
"examples/forms",
|
|
"examples/platform-browser",
|
|
"examples/router/activated-route",
|
|
"examples/router/testing",
|
|
"examples/service-worker/push",
|
|
"examples/service-worker/registration-options",
|
|
"examples/test-utils",
|
|
"examples/testing",
|
|
"examples/upgrade/static/ts/full",
|
|
"examples/upgrade/static/ts/lite",
|
|
"examples/upgrade/static/ts/lite-multi",
|
|
"examples/upgrade/static/ts/lite-multi-shared",
|
|
"forms",
|
|
"localize",
|
|
"localize/init",
|
|
"localize/src/localize",
|
|
"localize/src/utils",
|
|
"platform-browser",
|
|
"platform-browser-dynamic",
|
|
"platform-browser-dynamic/testing",
|
|
"platform-browser/animations",
|
|
"platform-browser/testing",
|
|
"platform-server",
|
|
"platform-server/init",
|
|
"platform-server/testing",
|
|
"router",
|
|
"router/testing",
|
|
"router/upgrade",
|
|
"service-worker",
|
|
"upgrade",
|
|
"upgrade/src/common",
|
|
"upgrade/static",
|
|
"upgrade/static/testing",
|
|
]
|