Revision 2664bc2b3ef4ee5fd671f915828cfcc274a36c77 authored by Paul Gschwendtner on 20 November 2021, 16:23:07 UTC, committed by Alex Rickabaugh on 08 December 2021, 18:42:42 UTC
Switches the integration tests form Puppeteer/webdriver-manager to the
Bazel-managed Chromium/Chromedriver. This is now possible with the new
integration test rule for which we can consult the
`dev-infra/bazel/browsers` toolchain and setup environment variables.
This has been configured already in a previous commit.

This commit also includes some additional small cleanups necessary for
the new integration test rule:

* The `test.sh` scripts have been renamed as they would conflict with
  the `test.sh` scripts generated by the integration test rule.
  Previously this was not an issue because tests were declared at a
  higher-level. As mentioned though this has other downsides and it is
  trivial to rename the file.

* Related to the point above, since tests are now declared witin the
  actual test folder (for perf e.g.), `package.json` files setting
  `"type": "module"` will accidentally cause the `nodejs_test`-generated
  files to be considered ESM. This is not correct and likely needs to be
  fixed upstream in `rules_nodejs` where explicit `.cjs` extensions
  should be used. This is only happening **once** in the `injectable-def`
  test so it is acceptable doing that for now.

PR Close #44238
1 parent 5837fbd
History
File Mode Size
aio-builds-setup
content
scripts
src
tests
tools
.browserslistrc -rw-r--r-- 600 bytes
.eslintrc.json -rw-r--r-- 2.9 KB
.gitignore -rw-r--r-- 664 bytes
.npmrc -rw-r--r-- 21 bytes
README.md -rw-r--r-- 7.7 KB
angular.json -rw-r--r-- 6.8 KB
firebase.json -rw-r--r-- 16.5 KB
karma.conf.js -rw-r--r-- 2.4 KB
ngsw-config.template.json -rw-r--r-- 1.8 KB
package.json -rw-r--r-- 8.5 KB
security-exemptions.json -rw-r--r-- 302 bytes
tsconfig.app.json -rw-r--r-- 357 bytes
tsconfig.json -rw-r--r-- 1.4 KB
tsconfig.spec.json -rw-r--r-- 297 bytes
tsconfig.worker.json -rw-r--r-- 230 bytes
yarn.lock -rw-r--r-- 554.5 KB

README.md

back to top