petebacondarwin · Mar 22, 2019 · Mar 22, 2019 · Mar 23, 2019 · Mar 27, 2019 · Mar 27, 2019
diff --git a/.codefresh/codefresh.yml b/.codefresh/codefresh.yml
@@ -17,8 +17,6 @@ steps:
     commands:
     # Install dependencies
     - yarn install --frozen-lockfile --non-interactive --network-timeout 100000 --no-progress
-    # Create symlinks needed for Windows.
-    - scripts\windows\create-symlinks.cmd
     # Add Bazel CI config
     - copy .codefresh\bazel.rc %ProgramData%\bazel.bazelrc
     # Run tests

diff --git a/aio/README.md b/aio/README.md
@@ -41,16 +41,6 @@ Here are the most important tasks you might need to use:
   - `yarn example-e2e --filter=foo` - limit e2e tests to those containing the word "foo"
   - `yarn example-e2e --setup --local` - run e2e tests with the local version of Angular contained in the "dist" folder
 
-## Developing on Windows
-The `packages/` directory may contain Linux-specific symlinks, which are not recognized by Windows.
-These unresolved links cause the docs generation process to fail because it cannot locate certain files.
-
-> Hint: The following steps require administration rights or [Windows Developer Mode](https://docs.microsoft.com/en-us/windows/uwp/get-started/enable-your-device-for-development) enabled!
-
-To fix this problem, run `scripts/windows/create-symlinks.sh`. This command creates temporary files where the symlinks used to be. Make sure not to commit those files with your documentation changes.
-When you are done making and testing your documentation changes, you can restore the original symlinks and delete the temporary files by running `scripts/windows/remove-symlinks.sh`.
-
-It's necessary to remove the temporary files, because otherwise they're displayed as local changes in your git working copy and certain operations are blocked.
 
 ## Using ServiceWorker locally
 

diff --git a/docs/DEVELOPER.md b/docs/DEVELOPER.md
@@ -59,18 +59,6 @@ Next, install the JavaScript modules needed to build and test Angular:
 yarn install
 ```
 
-## Windows only
-
-In order to create the right symlinks, run **as administrator**:
-```shell
-./scripts/windows/create-symlinks.sh
-```
-
-Before submitting a PR, do not forget to remove them:
-```shell
- ./scripts/windows/remove-symlinks.sh
- ```
-
 ## Building
 
 To build Angular run:

diff --git a/packages/upgrade/BUILD.bazel b/packages/upgrade/BUILD.bazel
@@ -7,14 +7,13 @@ ng_module(
     srcs = glob(
         [
             "*.ts",
-            "src/common/**/*.ts",
-            "src/dynamic/**/*.ts",
+            "src/dynamic/src/*.ts",
         ],
     ),
     deps = [
         "//packages/core",
-        "//packages/platform-browser",
         "//packages/platform-browser-dynamic",
+        "//packages/upgrade/src/common",
         "@npm//zone.js",
     ],
 )

diff --git a/packages/upgrade/public_api.ts b/packages/upgrade/public_api.ts
@@ -12,7 +12,7 @@
  * Entry point for all public APIs of this package. allowing
  * Angular 1 and Angular 2+ to run side by side in the same application.
  */
-export {VERSION} from './src/common/version';
-export {UpgradeAdapter, UpgradeAdapterRef} from './src/dynamic/upgrade_adapter';
+export {VERSION} from './src/common/src/version';
+export {UpgradeAdapter, UpgradeAdapterRef} from './src/dynamic/src/upgrade_adapter';
 
 // This file only re-exports content of the `src` folder. Keep it that way.
diff --git a/packages/upgrade/src/common/BUILD.bazel b/packages/upgrade/src/common/BUILD.bazel
@@ -0,0 +1,16 @@
+load("//tools:defaults.bzl", "ts_library")
+
+package(default_visibility = [
+    "//packages/upgrade:__subpackages__",
+    "//tools/public_api_guard:__subpackages__",
+])
+
+ts_library(
+    name = "common",
+    srcs = glob([
+        "src/**/*.ts",
+    ]),
+    deps = [
+        "//packages/core",
+    ],
+)
diff --git a/packages/upgrade/src/common/angular1.ts → packages/upgrade/src/common/src/angular1.ts b/packages/upgrade/src/common/angular1.ts → packages/upgrade/src/common/src/angular1.ts
diff --git a/...ages/upgrade/src/common/component_info.ts → .../upgrade/src/common/src/component_info.ts b/...ages/upgrade/src/common/component_info.ts → .../upgrade/src/common/src/component_info.ts
diff --git a/packages/upgrade/src/common/constants.ts → packages/upgrade/src/common/src/constants.ts b/packages/upgrade/src/common/constants.ts → packages/upgrade/src/common/src/constants.ts
diff --git a/...upgrade/src/common/downgrade_component.ts → ...ade/src/common/src/downgrade_component.ts b/...upgrade/src/common/downgrade_component.ts → ...ade/src/common/src/downgrade_component.ts
diff --git a/...src/common/downgrade_component_adapter.ts → ...common/src/downgrade_component_adapter.ts b/...src/common/downgrade_component_adapter.ts → ...common/src/downgrade_component_adapter.ts
diff --git a/...pgrade/src/common/downgrade_injectable.ts → ...de/src/common/src/downgrade_injectable.ts b/...pgrade/src/common/downgrade_injectable.ts → ...de/src/common/src/downgrade_injectable.ts
diff --git a/...ages/upgrade/src/common/upgrade_helper.ts → .../upgrade/src/common/src/upgrade_helper.ts b/...ages/upgrade/src/common/upgrade_helper.ts → .../upgrade/src/common/src/upgrade_helper.ts
diff --git a/packages/upgrade/src/common/util.ts → packages/upgrade/src/common/src/util.ts b/packages/upgrade/src/common/util.ts → packages/upgrade/src/common/src/util.ts
diff --git a/packages/upgrade/src/common/version.ts → packages/upgrade/src/common/src/version.ts b/packages/upgrade/src/common/version.ts → packages/upgrade/src/common/src/version.ts
diff --git a/packages/upgrade/src/common/test/BUILD.bazel b/packages/upgrade/src/common/test/BUILD.bazel
@@ -0,0 +1,23 @@
+load("//tools:defaults.bzl", "ts_library", "ts_web_test_suite")
+
+ts_library(
+    name = "test_lib",
+    testonly = True,
+    srcs = glob(["**/*.ts"]),
+    deps = [
+        "//packages/core",
+        "//packages/core/testing",
+        "//packages/upgrade/src/common",
+        "//packages/upgrade/src/common/test/helpers",
+    ],
+)
+
+ts_web_test_suite(
+    name = "test",
+    static_files = [
+        "//:angularjs_scripts",
+    ],
+    deps = [
+        ":test_lib",
+    ],
+)
diff --git a/...pgrade/test/common/component_info_spec.ts → ...de/src/common/test/component_info_spec.ts b/...pgrade/test/common/component_info_spec.ts → ...de/src/common/test/component_info_spec.ts
@@ -6,7 +6,7 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import {PropertyBinding} from '@angular/upgrade/src/common/component_info';
+import {PropertyBinding} from '../src/component_info';
 
 {
   describe('PropertyBinding', () => {

diff --git a/...ommon/downgrade_component_adapter_spec.ts → .../test/downgrade_component_adapter_spec.ts b/...ommon/downgrade_component_adapter_spec.ts → .../test/downgrade_component_adapter_spec.ts
@@ -7,10 +7,10 @@
  */
 import {Compiler, Component, ComponentFactory, Injector, NgModule, TestabilityRegistry} from '@angular/core';
 import {TestBed} from '@angular/core/testing';
-import * as angular from '@angular/upgrade/src/common/angular1';
-import {DowngradeComponentAdapter, groupNodesBySelector} from '@angular/upgrade/src/common/downgrade_component_adapter';
+import * as angular from '../src/angular1';
+import {DowngradeComponentAdapter, groupNodesBySelector} from '../src/downgrade_component_adapter';
 
-import {nodes, withEachNg1Version} from './test_helpers';
+import {nodes, withEachNg1Version} from './helpers/common_test_helpers';
 
 withEachNg1Version(() => {
   describe('DowngradeComponentAdapter', () => {

diff --git a/.../test/common/downgrade_injectable_spec.ts → .../common/test/downgrade_injectable_spec.ts b/.../test/common/downgrade_injectable_spec.ts → .../common/test/downgrade_injectable_spec.ts
@@ -7,10 +7,10 @@
  */
 
 import {Injector} from '@angular/core';
-import * as angular from '@angular/upgrade/src/common/angular1';
-import {$INJECTOR, INJECTOR_KEY, UPGRADE_APP_TYPE_KEY} from '@angular/upgrade/src/common/constants';
-import {downgradeInjectable} from '@angular/upgrade/src/common/downgrade_injectable';
-import {UpgradeAppType} from '@angular/upgrade/src/common/util';
+import * as angular from '../src/angular1';
+import {$INJECTOR, INJECTOR_KEY, UPGRADE_APP_TYPE_KEY} from '../src/constants';
+import {downgradeInjectable} from '../src/downgrade_injectable';
+import {UpgradeAppType} from '../src/util';
 
 describe('downgradeInjectable', () => {
   const setupMockInjectors = (downgradedModule = '') => {

diff --git a/packages/upgrade/src/common/test/helpers/BUILD.bazel b/packages/upgrade/src/common/test/helpers/BUILD.bazel
@@ -0,0 +1,14 @@
+load("//tools:defaults.bzl", "ts_library")
+
+package(default_visibility = ["//packages/upgrade:__subpackages__"])
+
+ts_library(
+    name = "helpers",
+    srcs = glob([
+        "*.ts",
+    ]),
+    deps = [
+        "//packages/core/testing",
+        "//packages/upgrade/src/common",
+    ],
+)
diff --git a/packages/upgrade/test/common/test_helpers.ts → ...ommon/test/helpers/common_test_helpers.ts b/packages/upgrade/test/common/test_helpers.ts → ...ommon/test/helpers/common_test_helpers.ts
@@ -5,7 +5,7 @@
  * 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
  */
-import {setAngularJSGlobal} from '@angular/upgrade/src/common/angular1';
+import {setAngularJSGlobal} from '../../src/angular1';
 
 // Whether the upgrade tests should run against AngularJS minified or not. This can be
 // temporarily switched to "false" in order to make it easy to debug AngularJS locally.

diff --git a/...es/upgrade/src/dynamic/upgrade_adapter.ts → ...pgrade/src/dynamic/src/upgrade_adapter.ts b/...es/upgrade/src/dynamic/upgrade_adapter.ts → ...pgrade/src/dynamic/src/upgrade_adapter.ts
@@ -9,11 +9,11 @@
 import {Compiler, CompilerOptions, Injector, NgModule, NgModuleRef, NgZone, StaticProvider, Testability, Type, resolveForwardRef} from '@angular/core';
 import {platformBrowserDynamic} from '@angular/platform-browser-dynamic';
 
-import {IAngularBootstrapConfig, IAugmentedJQuery, IInjectorService, IModule, IProvideService, IRootScopeService, ITestabilityService, bootstrap, element as angularElement, module as angularModule} from '../common/angular1';
-import {$$TESTABILITY, $COMPILE, $INJECTOR, $ROOT_SCOPE, COMPILER_KEY, INJECTOR_KEY, LAZY_MODULE_REF, NG_ZONE_KEY, UPGRADE_APP_TYPE_KEY} from '../common/constants';
-import {downgradeComponent} from '../common/downgrade_component';
-import {downgradeInjectable} from '../common/downgrade_injectable';
-import {Deferred, LazyModuleRef, UpgradeAppType, controllerKey, onError} from '../common/util';
+import {IAngularBootstrapConfig, IAugmentedJQuery, IInjectorService, IModule, IProvideService, IRootScopeService, ITestabilityService, bootstrap, element as angularElement, module as angularModule} from '../../common/src/angular1';
+import {$$TESTABILITY, $COMPILE, $INJECTOR, $ROOT_SCOPE, COMPILER_KEY, INJECTOR_KEY, LAZY_MODULE_REF, NG_ZONE_KEY, UPGRADE_APP_TYPE_KEY} from '../../common/src/constants';
+import {downgradeComponent} from '../../common/src/downgrade_component';
+import {downgradeInjectable} from '../../common/src/downgrade_injectable';
+import {Deferred, LazyModuleRef, UpgradeAppType, controllerKey, onError} from '../../common/src/util';
 
 import {UpgradeNg1ComponentAdapterBuilder} from './upgrade_ng1_adapter';
 

diff --git a/...pgrade/src/dynamic/upgrade_ng1_adapter.ts → ...de/src/dynamic/src/upgrade_ng1_adapter.ts b/...pgrade/src/dynamic/upgrade_ng1_adapter.ts → ...de/src/dynamic/src/upgrade_ng1_adapter.ts
@@ -8,10 +8,10 @@
 
 import {Directive, DoCheck, ElementRef, EventEmitter, Inject, Injector, OnChanges, OnDestroy, OnInit, SimpleChange, SimpleChanges, Type} from '@angular/core';
 
-import {IAttributes, IDirective, IDirectivePrePost, IInjectorService, ILinkFn, IScope, ITranscludeFunction} from '../common/angular1';
-import {$SCOPE} from '../common/constants';
-import {IBindingDestination, IControllerInstance, UpgradeHelper} from '../common/upgrade_helper';
-import {isFunction, strictEquals} from '../common/util';
+import {IAttributes, IDirective, IDirectivePrePost, IInjectorService, ILinkFn, IScope, ITranscludeFunction} from '../../common/src/angular1';
+import {$SCOPE} from '../../common/src/constants';
+import {IBindingDestination, IControllerInstance, UpgradeHelper} from '../../common/src/upgrade_helper';
+import {isFunction, strictEquals} from '../../common/src/util';
 
 
 const CAMEL_CASE = /([A-Z])/g;

diff --git a/packages/upgrade/src/dynamic/test/BUILD.bazel b/packages/upgrade/src/dynamic/test/BUILD.bazel
@@ -0,0 +1,28 @@
+load("//tools:defaults.bzl", "ts_library", "ts_web_test_suite")
+
+ts_library(
+    name = "test_lib",
+    testonly = True,
+    srcs = glob([
+        "**/*.ts",
+    ]),
+    deps = [
+        "//packages/core",
+        "//packages/core/testing",
+        "//packages/platform-browser",
+        "//packages/platform-browser-dynamic",
+        "//packages/upgrade",
+        "//packages/upgrade/src/common",
+        "//packages/upgrade/src/common/test/helpers",
+    ],
+)
+
+ts_web_test_suite(
+    name = "test",
+    static_files = [
+        "//:angularjs_scripts",
+    ],
+    deps = [
+        ":test_lib",
+    ],
+)
diff --git a/...ages/upgrade/test/dynamic/upgrade_spec.ts → .../upgrade/src/dynamic/test/upgrade_spec.ts b/...ages/upgrade/test/dynamic/upgrade_spec.ts → .../upgrade/src/dynamic/test/upgrade_spec.ts
@@ -10,11 +10,13 @@ import {ChangeDetectorRef, Component, EventEmitter, Input, NO_ERRORS_SCHEMA, NgM
 import {async, fakeAsync, flushMicrotasks, tick} from '@angular/core/testing';
 import {BrowserModule} from '@angular/platform-browser';
 import {platformBrowserDynamic} from '@angular/platform-browser-dynamic';
-import * as angular from '@angular/upgrade/src/common/angular1';
-import {$EXCEPTION_HANDLER} from '@angular/upgrade/src/common/constants';
-import {UpgradeAdapter, UpgradeAdapterRef} from '@angular/upgrade/src/dynamic/upgrade_adapter';
 
-import {$apply, $digest, html, multiTrim, withEachNg1Version} from './test_helpers';
+import * as angular from '../../common/src/angular1';
+import {$EXCEPTION_HANDLER, $ROOT_SCOPE} from '../../common/src/constants';
+import {html, multiTrim, withEachNg1Version} from '../../common/test/helpers/common_test_helpers';
+
+import {UpgradeAdapter, UpgradeAdapterRef} from '../src/upgrade_adapter';
+
 
 declare global {
   export var inject: Function;
@@ -3280,3 +3282,13 @@ withEachNg1Version(() => {
     });
   });
 });
+
+function $apply(adapter: UpgradeAdapterRef, exp: angular.Ng1Expression) {
+  const $rootScope = adapter.ng1Injector.get($ROOT_SCOPE) as angular.IRootScopeService;
+  $rootScope.$apply(exp);
+}
+
+function $digest(adapter: UpgradeAdapterRef) {
+  const $rootScope = adapter.ng1Injector.get($ROOT_SCOPE) as angular.IRootScopeService;
+  $rootScope.$digest();
+}
diff --git a/packages/upgrade/static/BUILD.bazel b/packages/upgrade/static/BUILD.bazel
@@ -6,18 +6,15 @@ exports_files(["package.json"])
 
 ng_module(
     name = "static",
-    # Note: There is Bazel issue where Windows symlinks
-    #       aka. junctions are not traversed by glob.
     srcs = glob(
         [
             "*.ts",
-            "src/common/**/*.ts",
-            "src/static/**/*.ts",
+            "src/*.ts",
         ],
     ),
     deps = [
         "//packages/core",
         "//packages/platform-browser",
-        "//packages/platform-browser-dynamic",
+        "//packages/upgrade/src/common",
     ],
 )
diff --git a/packages/upgrade/static/public_api.ts b/packages/upgrade/static/public_api.ts
@@ -6,19 +6,13 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-/**
- * @module
- * @description
- * Entry point for all public APIs of this package. allowing
- * Angular 1 and Angular 2+ to run side by side in the same application.
- */
-export {getAngularJSGlobal, getAngularLib, setAngularJSGlobal, setAngularLib} from './src/common/angular1';
-export {downgradeComponent} from './src/common/downgrade_component';
-export {downgradeInjectable} from './src/common/downgrade_injectable';
-export {VERSION} from './src/common/version';
-export {downgradeModule} from './src/static/downgrade_module';
-export {UpgradeComponent} from './src/static/upgrade_component';
-export {UpgradeModule} from './src/static/upgrade_module';
+export {getAngularJSGlobal, getAngularLib, setAngularJSGlobal, setAngularLib} from '../src/common/src/angular1';
+export {downgradeComponent} from '../src/common/src/downgrade_component';
+export {downgradeInjectable} from '../src/common/src/downgrade_injectable';
+export {VERSION} from '../src/common/src/version';
+export {downgradeModule} from './src/downgrade_module';
+export {UpgradeComponent} from './src/upgrade_component';
+export {UpgradeModule} from './src/upgrade_module';
 
 
-// This file only re-exports content of the `src` folder. Keep it that way.
+// This file only re-exports items to appear in the public api. Keep it that way.
diff --git a/packages/upgrade/static/src b/packages/upgrade/static/src
diff --git a/.../upgrade/src/static/angular1_providers.ts → .../upgrade/static/src/angular1_providers.ts b/.../upgrade/src/static/angular1_providers.ts → .../upgrade/static/src/angular1_providers.ts
@@ -7,7 +7,7 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import {IInjectorService} from '../common/angular1';
+import {IInjectorService} from '../../src/common/src/angular1';
 
 // We have to do a little dance to get the ng1 injector into the module injector.
 // We store the ng1 injector so that the provider in the module injector can access it

diff --git a/...es/upgrade/src/static/downgrade_module.ts → ...es/upgrade/static/src/downgrade_module.ts b/...es/upgrade/src/static/downgrade_module.ts → ...es/upgrade/static/src/downgrade_module.ts
@@ -9,9 +9,9 @@
 import {Injector, NgModuleFactory, NgModuleRef, StaticProvider} from '@angular/core';
 import {platformBrowser} from '@angular/platform-browser';
 
-import {IInjectorService, IProvideService, module as angularModule} from '../common/angular1';
-import {$INJECTOR, $PROVIDE, DOWNGRADED_MODULE_COUNT_KEY, INJECTOR_KEY, LAZY_MODULE_REF, UPGRADE_APP_TYPE_KEY, UPGRADE_MODULE_NAME} from '../common/constants';
-import {LazyModuleRef, UpgradeAppType, getDowngradedModuleCount, isFunction} from '../common/util';
+import {IInjectorService, IProvideService, module as angularModule} from '../../src/common/src/angular1';
+import {$INJECTOR, $PROVIDE, DOWNGRADED_MODULE_COUNT_KEY, INJECTOR_KEY, LAZY_MODULE_REF, UPGRADE_APP_TYPE_KEY, UPGRADE_MODULE_NAME} from '../../src/common/src/constants';
+import {LazyModuleRef, UpgradeAppType, getDowngradedModuleCount, isFunction} from '../../src/common/src/util';
 
 import {angular1Providers, setTempInjectorRef} from './angular1_providers';
 import {NgAdapterInjector} from './util';

diff --git a/...s/upgrade/src/static/upgrade_component.ts → ...s/upgrade/static/src/upgrade_component.ts b/...s/upgrade/src/static/upgrade_component.ts → ...s/upgrade/static/src/upgrade_component.ts
@@ -8,10 +8,10 @@
 
 import {DoCheck, ElementRef, EventEmitter, Injector, OnChanges, OnDestroy, OnInit, SimpleChanges, ɵlooseIdentical as looseIdentical} from '@angular/core';
 
-import {IAttributes, IAugmentedJQuery, IDirective, IDirectivePrePost, IInjectorService, ILinkFn, IScope, ITranscludeFunction} from '../common/angular1';
-import {$SCOPE} from '../common/constants';
-import {IBindingDestination, IControllerInstance, UpgradeHelper} from '../common/upgrade_helper';
-import {isFunction} from '../common/util';
+import {IAttributes, IAugmentedJQuery, IDirective, IDirectivePrePost, IInjectorService, ILinkFn, IScope, ITranscludeFunction} from '../../src/common/src/angular1';
+import {$SCOPE} from '../../src/common/src/constants';
+import {IBindingDestination, IControllerInstance, UpgradeHelper} from '../../src/common/src/upgrade_helper';
+import {isFunction} from '../../src/common/src/util';
 
 const NOT_SUPPORTED: any = 'NOT_SUPPORTED';
 const INITIAL_VALUE = {

diff --git a/...ages/upgrade/src/static/upgrade_module.ts → ...ages/upgrade/static/src/upgrade_module.ts b/...ages/upgrade/src/static/upgrade_module.ts → ...ages/upgrade/static/src/upgrade_module.ts
@@ -8,9 +8,9 @@
 
 import {Injector, NgModule, NgZone, Testability} from '@angular/core';
 
-import {IInjectorService, IIntervalService, IProvideService, ITestabilityService, bootstrap, element as angularElement, module as angularModule} from '../common/angular1';
-import {$$TESTABILITY, $DELEGATE, $INJECTOR, $INTERVAL, $PROVIDE, INJECTOR_KEY, LAZY_MODULE_REF, UPGRADE_APP_TYPE_KEY, UPGRADE_MODULE_NAME} from '../common/constants';
-import {LazyModuleRef, UpgradeAppType, controllerKey} from '../common/util';
+import {IInjectorService, IIntervalService, IProvideService, ITestabilityService, bootstrap, element as angularElement, module as angularModule} from '../../src/common/src/angular1';
+import {$$TESTABILITY, $DELEGATE, $INJECTOR, $INTERVAL, $PROVIDE, INJECTOR_KEY, LAZY_MODULE_REF, UPGRADE_APP_TYPE_KEY, UPGRADE_MODULE_NAME} from '../../src/common/src/constants';
+import {LazyModuleRef, UpgradeAppType, controllerKey} from '../../src/common/src/util';
 
 import {angular1Providers, setTempInjectorRef} from './angular1_providers';
 import {NgAdapterInjector} from './util';

diff --git a/packages/upgrade/src/static/util.ts → packages/upgrade/static/src/util.ts b/packages/upgrade/src/static/util.ts → packages/upgrade/static/src/util.ts
diff --git a/packages/upgrade/test/BUILD.bazel → packages/upgrade/static/test/BUILD.bazel b/packages/upgrade/test/BUILD.bazel → packages/upgrade/static/test/BUILD.bazel
@@ -3,22 +3,23 @@ load("//tools:defaults.bzl", "ts_library", "ts_web_test_suite")
 ts_library(
     name = "test_lib",
     testonly = True,
-    srcs = glob(["**/*.ts"]),
+    srcs = glob([
+        "**/*.ts",
+    ]),
     deps = [
-        "//packages:types",
         "//packages/core",
         "//packages/core/testing",
         "//packages/platform-browser",
         "//packages/platform-browser-dynamic",
         "//packages/platform-browser/testing",
-        "//packages/upgrade",
+        "//packages/upgrade/src/common",
+        "//packages/upgrade/src/common/test/helpers",
         "//packages/upgrade/static",
-        "@npm//rxjs",
     ],
 )
 
 ts_web_test_suite(
-    name = "test_web",
+    name = "test",
     static_files = [
         "//:angularjs_scripts",
     ],