fix(bazel): ts_web_test_suite now properly includes init_browser_spec.js (#27965)

There was an issue where init_browser_spec.js was being run out of order, this change adds it as a runtime dependency so it is executed when it needs to be preventing tests from bombing when they try to read from isNode from global scope before it is defined

PR Close #27965
This commit is contained in:
Ben Lesh 2019-01-03 17:46:14 -08:00 committed by Andrew Kushnir
parent 8ebdb437dc
commit ce51dfb499
1 changed files with 5 additions and 2 deletions

View File

@ -120,16 +120,19 @@ def npm_package(name, replacements = {}, **kwargs):
**kwargs
)
def ts_web_test_suite(bootstrap = [], deps = [], **kwargs):
def ts_web_test_suite(bootstrap = [], deps = [], runtime_deps = [], **kwargs):
"""Default values for ts_web_test_suite"""
if not bootstrap:
bootstrap = ["//:web_test_bootstrap_scripts"]
local_deps = [
"@ngdeps//node_modules/tslib:tslib.js",
"//tools/testing:browser",
] + deps
local_runtime_deps = [
"//tools/testing:browser",
] + runtime_deps
_ts_web_test_suite(
runtime_deps = local_runtime_deps,
bootstrap = bootstrap,
deps = local_deps,
karma = _DEFAULT_KARMA_BIN,