Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commit 9661530

Browse files
devversionjelbourn
authored andcommitted
build: run snapshot local browser tests with bazel (#16832)
Runs the snapshot local browser tests with Bazel. This improves our consistency and allows us to remove `gulp test` in the future.
1 parent 7c40679 commit 9661530

File tree

3 files changed

+11
-21
lines changed

3 files changed

+11
-21
lines changed

‎.circleci/config.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -388,19 +388,19 @@ jobs:
388388
snapshot_tests_local_browsers:
389389
docker: *docker-firefox-image
390390
resource_class: xlarge
391+
environment:
392+
GCP_DECRYPT_TOKEN: *gcp_decrypt_token
391393
steps:
392394
- *checkout_code
393395
- *restore_cache
396+
- *setup_bazel_ci_config
397+
- *setup_bazel_remote_execution
394398
- *yarn_download
395399
- *yarn_install
400+
- *setup_bazel_binary
396401

397402
- run: node ./scripts/circleci/setup-angular-snapshots.js --tag master
398-
# We need to re-run yarn install in order to pull the Angular snapshot builds. Note
399-
# that we can't use the "--frozen-lockfile" option since the setup snapshots script does
400-
# not update the lock file.
401-
- run: yarn install --non-interactive
402-
- run: ./scripts/circleci/run-local-browser-tests.sh
403-
403+
- run: bazel test src/... --build_tag_filters=-e2e --test_tag_filters=-e2e
404404

405405
# ----------------------------------------------------------------------------
406406
# Job that runs all Bazel tests against Ivy with the current Angular version

‎angular-tsconfig.json

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,11 @@
22
// This file is required because when using the Angular NPM packages and building
33
// with AOT compilation, NGC needs the "ngsummary.json" files.
44
{
5+
"angularCompilerOptions": {
6+
// In snapshot builds the compiler-cli will now use ngtsc by default. In
7+
// order to be able to build the summary files, we need to use ngc.
8+
"enableIvy": false
9+
},
510
"compilerOptions": {
611
"module": "umd",
712
"moduleResolution": "node",

‎scripts/circleci/run-local-browser-tests.sh

Lines changed: 0 additions & 15 deletions
This file was deleted.

0 commit comments

Comments
(0)

AltStyle によって変換されたページ (->オリジナル) /