build: migrate all ts_library in packages/core/test (angular#61472)

This commit migrates all `ts_library` in `package/core/test` to
`ts_project`, and fixes deep module, or relative imports inside.

PR Close angular#61472

Author: devversion

packages/core/test/BUILD.bazel

@@ -1,4 +1,5 @@
-load("//tools:defaults.bzl", "jasmine_node_test", "karma_web_test_suite", "ts_library")
+load("//tools:defaults.bzl", "jasmine_node_test", "karma_web_test_suite")
+load("//tools:defaults2.bzl", "ts_project")
 
 package(default_visibility = ["//visibility:public"])
 
@@ -23,17 +24,19 @@ UTILS = [
     "linker/source_map_util.ts",
 ]
 
-ts_library(
+ts_project(
     name = "test_utils",
     testonly = True,
     srcs = UTILS,
-    deps = [
+    interop_deps = [
         "//packages/compiler",
-        "@npm//source-map",
+    ],
+    deps = [
+        "//:node_modules/source-map",
     ],
 )
 
-ts_library(
+ts_project(
     name = "test_lib",
     testonly = True,
     srcs = glob(
@@ -43,10 +46,7 @@ ts_library(
             "reflection/es2015_inheritance_fixture.ts",
         ],
     ),
-    # Visible to //:saucelabs_unit_tests_poc target
-    visibility = ["//:__pkg__"],
-    deps = [
-        ":test_utils",
+    interop_deps = [
         "//packages/animations",
         "//packages/animations/browser",
         "//packages/animations/browser/testing",
@@ -69,20 +69,23 @@ ts_library(
         "//packages/router",
         "//packages/router/testing",
         "//packages/zone.js/lib:zone_d_ts",
-        "@npm//rxjs",
+    ],
+    # Visible to //:saucelabs_unit_tests_poc target
+    visibility = ["//:__pkg__"],
+    deps = [
+        ":test_utils_rjs",
+        "//:node_modules/rxjs",
     ],
 )
 
-ts_library(
+ts_project(
     name = "test_node_only_lib",
     testonly = True,
     srcs = glob(
         ["**/*_node_only_spec.ts"],
         exclude = UTILS,
     ),
-    deps = [
-        ":test_lib",
-        ":test_utils",
+    interop_deps = [
         "//packages/compiler",
         "//packages/core",
         "//packages/core/src/compiler",
@@ -91,6 +94,10 @@ ts_library(
         "//packages/platform-server/testing",
         "//packages/private/testing",
     ],
+    deps = [
+        ":test_lib_rjs",
+        ":test_utils_rjs",
+    ],
 )
 
 jasmine_node_test(

packages/core/test/acceptance/BUILD.bazel

@@ -1,21 +1,20 @@
-load("//tools:defaults.bzl", "jasmine_node_test", "karma_web_test_suite", "ng_module", "ts_library")
+load("//tools:defaults.bzl", "jasmine_node_test", "karma_web_test_suite", "ng_module")
+load("//tools:defaults2.bzl", "ts_project")
 
 package(default_visibility = ["//visibility:private"])
 
 SPEC_FILES_WITH_FORWARD_REFS = [
     "di_forward_ref_spec.ts",
 ]
 
-ts_library(
+ts_project(
     name = "acceptance_lib",
     testonly = True,
     srcs = glob(
         ["**/*.ts"],
         exclude = SPEC_FILES_WITH_FORWARD_REFS,
     ),
-    # Visible to //:saucelabs_unit_tests
-    visibility = ["//:__pkg__"],
-    deps = [
+    interop_deps = [
         "//packages/animations",
         "//packages/animations/browser",
         "//packages/animations/browser/testing",
@@ -38,7 +37,11 @@ ts_library(
         "//packages/private/testing",
         "//packages/router",
         "//packages/zone.js/lib:zone_d_ts",
-        "@npm//rxjs",
+    ],
+    # Visible to //:saucelabs_unit_tests
+    visibility = ["//:__pkg__"],
+    deps = [
+        "//:node_modules/rxjs",
     ],
 )
 

packages/core/test/acceptance/after_render_hook_spec.ts

@@ -6,7 +6,10 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
-import {PLATFORM_BROWSER_ID, PLATFORM_SERVER_ID} from '@angular/common/src/platform_id';
+import {
+  ɵPLATFORM_BROWSER_ID as PLATFORM_BROWSER_ID,
+  ɵPLATFORM_SERVER_ID as PLATFORM_SERVER_ID,
+} from '@angular/common';
 import {
   AfterRenderRef,
   ApplicationRef,

packages/core/test/acceptance/authoring/BUILD.bazel

@@ -1,4 +1,5 @@
-load("//tools:defaults.bzl", "jasmine_node_test", "karma_web_test_suite", "ng_module", "nodejs_binary", "npm_package_bin", "ts_library")
+load("//tools:defaults.bzl", "jasmine_node_test", "karma_web_test_suite", "ng_module", "nodejs_binary", "npm_package_bin")
+load("//tools:defaults2.bzl", "ts_project")
 
 package(default_visibility = ["//visibility:private"])
 
@@ -13,20 +14,21 @@ TEST_DEPS = [
     "//packages/core/testing",
     "//packages/platform-browser",
     "//packages/core/primitives/signals",
-    "@npm//rxjs",
 ]
 
-ts_library(
+ts_project(
     name = "test_compiler_lib",
     testonly = True,
     srcs = ["authoring_test_compiler.ts"],
-    deps = [
+    interop_deps = [
         "//packages/compiler-cli",
         "//packages/compiler-cli/src/ngtsc/imports",
         "//packages/compiler-cli/src/ngtsc/partial_evaluator",
         "//packages/compiler-cli/src/ngtsc/reflection",
         "//packages/compiler-cli/src/ngtsc/transform/jit",
-        "@npm//typescript",
+    ],
+    deps = [
+        "//:node_modules/typescript",
     ],
 )
 
@@ -37,7 +39,7 @@ nodejs_binary(
         ":test_compiler_lib",
         "//packages:package_json",
     ],
-    entry_point = ":authoring_test_compiler.ts",
+    entry_point = ":authoring_test_compiler.js",
 )
 
 npm_package_bin(
@@ -49,18 +51,25 @@ npm_package_bin(
     tool = ":test_compiler",
 )
 
-ts_library(
+ts_project(
     name = "test_jit_lib",
     testonly = True,
     srcs = ["transformed_%s" % file for file in TEST_FILES],
-    deps = TEST_DEPS,
+    interop_deps = TEST_DEPS,
+    deps = [
+        # TODO: Only rely on `TEST_DEPS`. This should be removed during `rules_js` migration.
+        "//:node_modules/rxjs",
+    ],
 )
 
 ng_module(
     name = "test_lib",
     testonly = True,
     srcs = TEST_FILES,
-    deps = TEST_DEPS,
+    deps = TEST_DEPS + [
+        # TODO: Only rely on `TEST_DEPS`. This should be removed during `rules_js` migration.
+        "@npm//rxjs",
+    ],
 )
 
 jasmine_node_test(

packages/core/test/acceptance/change_detection_spec.ts

@@ -7,7 +7,7 @@
  */
 
 import {CommonModule} from '@angular/common';
-import {expect} from '@angular/platform-browser/testing/src/matchers';
+import {expect} from '@angular/private/testing/matchers';
 import {BehaviorSubject} from 'rxjs';
 import {
   ApplicationRef,

packages/core/test/acceptance/change_detection_transplanted_view_spec.ts

@@ -28,9 +28,9 @@ import {
   ViewChild,
   ViewContainerRef,
 } from '../../src/core';
-import {provideCheckNoChangesConfig} from '@angular/core/src/change_detection/provide_check_no_changes_config';
+import {provideCheckNoChangesConfig} from '../../src/change_detection/provide_check_no_changes_config';
 import {ComponentFixture, TestBed} from '../../testing';
-import {expect} from '@angular/platform-browser/testing/src/matchers';
+import {expect} from '@angular/private/testing/matchers';
 import {of} from 'rxjs';
 
 describe('change detection for transplanted views', () => {

packages/core/test/acceptance/component_spec.ts

@@ -33,7 +33,7 @@ import {
   ɵɵdefineComponent,
 } from '../../src/core';
 import {TestBed} from '../../testing';
-import {expect} from '@angular/platform-browser/testing/src/matchers';
+import {expect} from '@angular/private/testing/matchers';
 
 import {global} from '../../src/util/global';
 

packages/core/test/acceptance/content_spec.ts

@@ -22,7 +22,7 @@ import {
 } from '../../src/core';
 import {TestBed} from '../../testing';
 import {By} from '@angular/platform-browser';
-import {expect} from '@angular/platform-browser/testing/src/matchers';
+import {expect} from '@angular/private/testing/matchers';
 
 describe('projection', () => {
   function getElementHtml(element: HTMLElement) {

packages/core/test/acceptance/defer_spec.ts

@@ -41,10 +41,11 @@ import {getInjectorResolutionPath} from '../../src/render3/util/injector_discove
 import {ActivatedRoute, provideRouter, Router, RouterOutlet} from '@angular/router';
 import {ChainedInjector} from '../../src/render3/chained_injector';
 import {global} from '../../src/util/global';
-import {TimerScheduler} from '@angular/core/src/defer/timer_scheduler';
+import {TimerScheduler} from '../../src/defer/timer_scheduler';
 import {Console} from '../../src/console';
 import {formatRuntimeErrorCode, RuntimeErrorCode} from '../../src/errors';
 import {provideNgReflectAttributes} from '../../src/ng_reflect';
+import {isBrowser} from '@angular/private/testing';
 
 /**
  * Clears all associated directive defs from a given component class.

packages/core/test/acceptance/hmr_spec.ts

@@ -38,6 +38,7 @@ import {clearTranslations, loadTranslations} from '@angular/localize';
 import {computeMsgId} from '@angular/compiler';
 import {EVENT_MANAGER_PLUGINS} from '@angular/platform-browser';
 import {ComponentType} from '../../src/render3';
+import {isNode} from '@angular/private/testing';
 
 describe('hot module replacement', () => {
   it('should recreate a single usage of a basic component', () => {