Skip to content

Commit c0e1ce5

Browse files
Merge master into release
2 parents 3337f59 + e30d7a6 commit c0e1ce5

File tree

51 files changed

+1257
-490
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

51 files changed

+1257
-490
lines changed

.github/workflows/test-all.yml

Lines changed: 10 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,10 @@ on:
77
env:
88
# make chromedriver detect installed Chrome version and download the corresponding driver
99
DETECT_CHROMEDRIVER_VERSION: true
10+
# The default behavior of chromedriver uses the older Chrome download URLs. We need to override
11+
# the beahvior to use the new URLs.
12+
CHROMEDRIVER_CDNURL: https://googlechromelabs.github.io/
13+
CHROMEDRIVER_CDNBINARIESURL: https://edgedl.me.gvt1.com/edgedl/chrome/chrome-for-testing/
1014
artifactRetentionDays: 14
1115

1216
jobs:
@@ -17,14 +21,8 @@ jobs:
1721
# Install Chrome so the correct version of webdriver can be installed by chromedriver when
1822
# setting up the repo. This must be done to build and execute Auth properly.
1923
- name: install Chrome stable
20-
# Pin Chrome version 114.0.5735.90-1 to avoid install failures like "No such object: chromedriver/LATEST_RELEASE_115.0.5790".
21-
# The failure happens because https://chromedriver.chromium.org/downloads only goes up to version 114.
22-
# TODO(b/297380444) Update script to install the latest Chrome and ChromeDriver.
2324
run: |
24-
sudo apt-get update
25-
sudo apt-get install wget
26-
sudo wget http://dl.google.com/linux/chrome/deb/pool/main/g/google-chrome-stable/google-chrome-stable_114.0.5735.90-1_amd64.deb
27-
sudo apt-get install -f ./google-chrome-stable_114.0.5735.90-1_amd64.deb --allow-downgrades
25+
npx @puppeteer/browsers install chrome@stable
2826
- uses: actions/checkout@v3
2927
- name: Set up Node (16)
3028
uses: actions/setup-node@v3
@@ -101,16 +99,11 @@ jobs:
10199
needs: build
102100
runs-on: ubuntu-latest
103101
steps:
104-
# install Chrome so the correct version of webdriver can be installed by chromedriver when setting up the repo
105-
- name: install Chrome stable
106-
# Pin Chrome version 114.0.5735.90-1 to avoid install failures like "No such object: chromedriver/LATEST_RELEASE_115.0.5790".
107-
# The failure happens because https://chromedriver.chromium.org/downloads only goes up to version 114.
108-
# TODO(b/297380444) Update script to install the latest Chrome and ChromeDriver.
109-
run: |
110-
sudo apt-get update
111-
sudo apt-get install wget
112-
sudo wget http://dl.google.com/linux/chrome/deb/pool/main/g/google-chrome-stable/google-chrome-stable_114.0.5735.90-1_amd64.deb
113-
sudo apt-get install -f ./google-chrome-stable_114.0.5735.90-1_amd64.deb --allow-downgrades
102+
# install Chrome first, so the correct version of webdriver can be installed by chromedriver
103+
# when setting up the repo
104+
- name: install Chrome stable
105+
run: |
106+
npx @puppeteer/browsers install chrome@stable
114107
- name: Download build archive
115108
uses: actions/download-artifact@v3
116109
with:

.github/workflows/test-changed-auth.yml

Lines changed: 11 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -5,32 +5,31 @@ on: pull_request
55
env:
66
# make chromedriver detect installed Chrome version and download the corresponding driver
77
DETECT_CHROMEDRIVER_VERSION: true
8+
# The default behavior of chromedriver uses the older Chrome download URLs. We need to override
9+
# the beahvior to use the new URLs.
10+
CHROMEDRIVER_CDNURL: https://googlechromelabs.github.io/
11+
CHROMEDRIVER_CDNBINARIESURL: https://edgedl.me.gvt1.com/edgedl/chrome/chrome-for-testing/
812

913
jobs:
1014
test-chrome:
1115
name: Test Auth on Chrome and Node If Changed
1216
runs-on: ubuntu-latest
1317

1418
steps:
15-
# install Chrome first, so the correct version of webdriver can be installed by chromedriver when setting up the repo
19+
# install Chrome first, so the correct version of webdriver can be installed by chromedriver
20+
# when setting up the repo
1621
- name: install Chrome stable
17-
# Pin Chrome version 114.0.5735.90-1 to avoid install failures like "No such object: chromedriver/LATEST_RELEASE_115.0.5790".
18-
# The failure happens because https://chromedriver.chromium.org/downloads only goes up to version 114.
19-
# TODO(b/297380444) Update script to install the latest Chrome and ChromeDriver.
2022
run: |
21-
sudo apt-get update
22-
sudo apt-get install wget
23-
sudo wget http://dl.google.com/linux/chrome/deb/pool/main/g/google-chrome-stable/google-chrome-stable_114.0.5735.90-1_amd64.deb
24-
sudo apt-get install -f ./google-chrome-stable_114.0.5735.90-1_amd64.deb --allow-downgrades
23+
npx @puppeteer/browsers install chrome@stable
2524
- name: Checkout Repo
2625
uses: actions/checkout@master
2726
with:
2827
# This makes Actions fetch all Git history so run-changed script can diff properly.
2928
fetch-depth: 0
30-
- name: Set up Node (14)
29+
- name: Set up Node (16)
3130
uses: actions/setup-node@v3
3231
with:
33-
node-version: 14.x
32+
node-version: 16.x
3433
- name: Bump Node memory limit
3534
run: echo "NODE_OPTIONS=--max_old_space_size=4096" >> $GITHUB_ENV
3635
- name: Test setup and yarn install
@@ -49,27 +48,22 @@ jobs:
4948

5049
runs-on: ubuntu-20.04
5150

52-
# Pin Chrome version 114.0.5735.90-1 to avoid install failures like "No such object: chromedriver/LATEST_RELEASE_115.0.5790".
53-
# The failure happens because https://chromedriver.chromium.org/downloads only goes up to version 114.
54-
# TODO(b/297380444) Update script to install the latest Chrome and ChromeDriver.
5551
steps:
5652
- name: install Firefox stable
5753
run: |
5854
sudo apt-get update
5955
sudo apt-get install firefox
6056
sudo apt-get install wget
61-
sudo wget http://dl.google.com/linux/chrome/deb/pool/main/g/google-chrome-stable/google-chrome-stable_114.0.5735.90-1_amd64.deb
62-
sudo apt-get install -f ./google-chrome-stable_114.0.5735.90-1_amd64.deb --allow-downgrades
6357
6458
- name: Checkout Repo
6559
uses: actions/checkout@master
6660
with:
6761
# This makes Actions fetch all Git history so run-changed script can diff properly.
6862
fetch-depth: 0
69-
- name: Set up Node (14)
63+
- name: Set up Node (16)
7064
uses: actions/setup-node@v3
7165
with:
72-
node-version: 14.x
66+
node-version: 16.x
7367
- name: Bump Node memory limit
7468
run: echo "NODE_OPTIONS=--max_old_space_size=4096" >> $GITHUB_ENV
7569
- name: Test setup and yarn install

.github/workflows/test-changed-fcm-integration.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,10 @@ jobs:
2222
with:
2323
# This makes Actions fetch all Git history so run-changed script can diff properly.
2424
fetch-depth: 0
25-
- name: Set up Node (14)
25+
- name: Set up Node (16)
2626
uses: actions/setup-node@v3
2727
with:
28-
node-version: 14.x
28+
node-version: 16.x
2929
- name: Bump Node memory limit
3030
run: echo "NODE_OPTIONS=--max_old_space_size=4096" >> $GITHUB_ENV
3131
- name: Test setup and yarn install

.github/workflows/test-changed-firestore-integration.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,10 +13,10 @@ jobs:
1313
with:
1414
# This makes Actions fetch all Git history so run-changed script can diff properly.
1515
fetch-depth: 0
16-
- name: Set up Node (14)
16+
- name: Set up Node (16)
1717
uses: actions/setup-node@v3
1818
with:
19-
node-version: 14.x
19+
node-version: 16.x
2020
- name: install Chrome stable
2121
run: |
2222
sudo apt-get update

.github/workflows/test-changed-firestore.yml

Lines changed: 13 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -19,10 +19,10 @@ jobs:
1919
with:
2020
# This makes Actions fetch all Git history so run-changed script can diff properly.
2121
fetch-depth: 0
22-
- name: Set up Node (14)
22+
- name: Set up Node (16)
2323
uses: actions/setup-node@v3
2424
with:
25-
node-version: 14.x
25+
node-version: 16.x
2626
- name: install Chrome stable
2727
run: |
2828
sudo apt-get update
@@ -68,10 +68,10 @@ jobs:
6868
needs: build
6969
if: ${{ needs.build.outputs.changed == 'true'}}
7070
steps:
71-
- name: Set up Node (14)
71+
- name: Set up Node (16)
7272
uses: actions/setup-node@v3
7373
with:
74-
node-version: 14.x
74+
node-version: 16.x
7575
- name: install Chrome stable
7676
run: |
7777
sudo apt-get update
@@ -98,10 +98,10 @@ jobs:
9898
needs: build
9999
if: ${{ needs.build.outputs.changed == 'true'}}
100100
steps:
101-
- name: Set up Node (14)
101+
- name: Set up Node (16)
102102
uses: actions/setup-node@v3
103103
with:
104-
node-version: 14.x
104+
node-version: 16.x
105105
- name: install Chrome stable
106106
run: |
107107
sudo apt-get update
@@ -118,6 +118,8 @@ jobs:
118118
run: cp config/ci.config.json config/project.json
119119
- name: Run tests
120120
run: cd packages/firestore && yarn run ${{ matrix.test-name }}
121+
env:
122+
EXPERIMENTAL_MODE: true
121123

122124
compat-test-firefox:
123125
name: Test Firestore Compatible on Firefox
@@ -132,10 +134,10 @@ jobs:
132134
run: |
133135
sudo apt-get update
134136
sudo apt-get install firefox
135-
- name: Set up Node (14)
137+
- name: Set up Node (16)
136138
uses: actions/setup-node@v3
137139
with:
138-
node-version: 14.x
140+
node-version: 16.x
139141
- name: Download build archive
140142
uses: actions/download-artifact@v3
141143
with:
@@ -173,10 +175,10 @@ jobs:
173175
name: build.tar.gz
174176
- name: Unzip build artifact
175177
run: tar xf build.tar.gz
176-
- name: Set up Node (14)
178+
- name: Set up Node (16)
177179
uses: actions/setup-node@v3
178180
with:
179-
node-version: 14.x
181+
node-version: 16.x
180182
- name: Bump Node memory limit
181183
run: echo "NODE_OPTIONS=--max_old_space_size=4096" >> $GITHUB_ENV
182184
- name: Test setup and yarn install
@@ -185,6 +187,7 @@ jobs:
185187
run: cd packages/firestore && xvfb-run yarn run ${{ matrix.test-name }}
186188
env:
187189
BROWSERS: 'Firefox'
190+
EXPERIMENTAL_MODE: true
188191

189192
# A job that fails if any required job in the test matrix fails,
190193
# to be used as a required check for merging.

.github/workflows/test-changed.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,10 +13,10 @@ jobs:
1313
with:
1414
# This makes Actions fetch all Git history so run-changed script can diff properly.
1515
fetch-depth: 0
16-
- name: Set up Node (14)
16+
- name: Set up Node (16)
1717
uses: actions/setup-node@v3
1818
with:
19-
node-version: 14.x
19+
node-version: 16.x
2020
- name: install Chrome stable
2121
run: |
2222
sudo apt-get update
@@ -44,10 +44,10 @@ jobs:
4444
uses: actions/checkout@master
4545
with:
4646
fetch-depth: 0
47-
- name: Set up Node (14)
47+
- name: Set up Node (16)
4848
uses: actions/setup-node@v3
4949
with:
50-
node-version: 14.x
50+
node-version: 16.x
5151
- name: install Firefox stable
5252
run: |
5353
sudo apt-get update

.github/workflows/test-firebase-integration.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,10 +13,10 @@ jobs:
1313
with:
1414
# This makes Actions fetch all Git history so run-changed script can diff properly.
1515
fetch-depth: 0
16-
- name: Set up Node (14)
16+
- name: Set up Node (16)
1717
uses: actions/setup-node@v3
1818
with:
19-
node-version: 14.x
19+
node-version: 16.x
2020
- name: install Chrome stable
2121
run: |
2222
sudo apt-get update

integration/messaging/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
"devDependencies": {
1212
"firebase": "10.4.0",
1313
"chai": "4.3.7",
14-
"chromedriver": "98.0.1",
14+
"chromedriver": "114.0.2",
1515
"express": "4.18.2",
1616
"geckodriver": "2.0.4",
1717
"mocha": "9.2.2",

packages/analytics-compat/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@
4545
"build:deps": "lerna run --scope @firebase/analytics-compat --include-dependencies build",
4646
"build:release": "yarn build && yarn add-compat-overloads",
4747
"dev": "rollup -c -w",
48-
"test": "run-p lint test:browser",
48+
"test": "run-p --npm-path npm lint test:browser",
4949
"test:ci": "node ../../scripts/run_tests_in_ci.js -s test:browser",
5050
"test:browser": "karma start --single-run",
5151
"test:browser:debug": "karma start --browsers=Chrome --auto-watch",

packages/analytics/package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,8 @@
2626
"build:release": "yarn build && yarn typings:public",
2727
"build:deps": "lerna run --scope @firebase/analytics --include-dependencies build",
2828
"dev": "rollup -c -w",
29-
"test": "run-p lint test:all",
30-
"test:all": "run-p test:browser test:integration",
29+
"test": "run-p --npm-path npm lint test:all",
30+
"test:all": "run-p --npm-path npm test:browser test:integration",
3131
"test:ci": "node ../../scripts/run_tests_in_ci.js -s test:all",
3232
"test:browser": "karma start --single-run --nocache",
3333
"test:integration": "karma start ./karma.integration.conf.js --single-run --nocache",

packages/app-check-compat/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
"build:release": "yarn build && yarn add-compat-overloads",
2727
"build:deps": "lerna run --scope @firebase/app-check-compat --include-dependencies build",
2828
"dev": "rollup -c -w",
29-
"test": "run-p lint test:browser",
29+
"test": "run-p --npm-path npm lint test:browser",
3030
"test:ci": "node ../../scripts/run_tests_in_ci.js -s test:browser",
3131
"test:browser": "karma start --single-run --nocache",
3232
"add-compat-overloads": "ts-node-script ../../scripts/build/create-overloads.ts -i ../app-check/dist/app-check-public.d.ts -o dist/src/index.d.ts -a -r AppCheck:FirebaseAppCheck -r FirebaseApp:FirebaseAppCompat --moduleToEnhance @firebase/app-check"

packages/app-check/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
"build:release": "yarn build && yarn api-report && yarn typings:public",
2727
"build:deps": "lerna run --scope @firebase/app-check --include-dependencies build",
2828
"dev": "rollup -c -w",
29-
"test": "run-p lint test:browser",
29+
"test": "run-p --npm-path npm lint test:browser",
3030
"test:ci": "node ../../scripts/run_tests_in_ci.js -s test:browser",
3131
"test:browser": "karma start --single-run --nocache",
3232
"api-report": "api-extractor run --local --verbose",

packages/app-compat/package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,8 @@
2929
"build": "rollup -c && yarn api-report",
3030
"build:deps": "lerna run --scope @firebase/app-compat --include-dependencies build",
3131
"dev": "rollup -c -w",
32-
"test": "run-p lint test:all",
33-
"test:all": "run-p test:browser test:node",
32+
"test": "run-p --npm-path npm lint test:all",
33+
"test:all": "run-p --npm-path npm test:browser test:node",
3434
"test:ci": "node ../../scripts/run_tests_in_ci.js -s test:all",
3535
"test:browser": "karma start --single-run",
3636
"test:browser:debug": "karma start --browsers Chrome --auto-watch",

packages/app/package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,9 @@
2626
"build:release": "rollup -c rollup.config.release.js && yarn api-report && yarn typings:public",
2727
"build:deps": "lerna run --scope @firebase/app --include-dependencies build",
2828
"dev": "rollup -c -w",
29-
"test": "run-p lint test:all",
29+
"test": "run-p --npm-path npm lint test:all",
3030
"test:ci": "node ../../scripts/run_tests_in_ci.js -s test:all",
31-
"test:all": "run-p test:browser test:node",
31+
"test:all": "run-p --npm-path npm test:browser test:node",
3232
"test:browser": "karma start --single-run",
3333
"test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha src/**/*.test.ts --config ../../config/mocharc.node.js",
3434
"api-report": "api-extractor run --local --verbose",

packages/auth-compat/package.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,16 +34,16 @@
3434
"build:deps": "lerna run --scope @firebase/auth-compat --include-dependencies build",
3535
"build:release": "yarn build && yarn add-compat-overloads",
3636
"dev": "rollup -c -w",
37-
"test": "run-p lint test:all",
38-
"test:all": "run-p test:browser test:node test:integration",
37+
"test": "run-p --npm-path npm lint test:all",
38+
"test:all": "run-p --npm-path npm test:browser test:node test:integration",
3939
"test:ci": "node ../../scripts/run_tests_in_ci.js -s test:all",
4040
"test:browser": "karma start --single-run",
4141
"test:browser:unit": "karma start --single-run --unit",
4242
"test:browser:integration": "karma start --single-run --integration",
4343
"test:node": "ts-node -O '{\"module\": \"commonjs\", \"target\": \"es6\"}' scripts/run_node_tests.ts",
4444
"test:node:integration": "ts-node -O '{\"module\": \"commonjs\", \"target\": \"es6\"}' scripts/run_node_tests.ts --integration",
4545
"test:webdriver": "rollup -c test/integration/webdriver/static/rollup.config.js && ts-node -O '{\"module\": \"commonjs\", \"target\": \"es6\"}' scripts/run_node_tests.ts --webdriver",
46-
"test:integration": "firebase emulators:exec --project demo-emulatedproject --only auth \"run-s test:browser:integration test:node:integration test:webdriver\"",
46+
"test:integration": "firebase emulators:exec --project demo-emulatedproject --only auth \"run-s --npm-path npm test:browser:integration test:node:integration test:webdriver\"",
4747
"add-compat-overloads": "ts-node-script ../../scripts/build/create-overloads.ts -i ../auth/dist/auth-public.d.ts -o dist/auth-compat/index.d.ts -a -r Auth:types.FirebaseAuth -r User:types.User -r FirebaseApp:FirebaseAppCompat --moduleToEnhance @firebase/auth"
4848
},
4949
"peerDependencies": {

packages/auth/package.json

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -78,11 +78,11 @@
7878
"build:release": "yarn build && yarn typings:public",
7979
"build:scripts": "tsc -moduleResolution node --module commonjs scripts/*.ts && ls scripts/*.js | xargs -I % sh -c 'terser % -o %'",
8080
"dev": "rollup -c -w",
81-
"test": "run-p lint test:all",
82-
"test:all": "run-p test:browser:unit test:node:unit test:integration test:browser:integration:prodbackend",
83-
"test:integration": "firebase emulators:exec --project emulatedproject --only auth \"run-s test:browser:integration:local test:node:integration:local test:webdriver\"",
81+
"test": "run-p --npm-path npm lint test:all",
82+
"test:all": "run-p --npm-path npm test:browser:unit test:node:unit test:integration test:browser:integration:prodbackend",
83+
"test:integration": "firebase emulators:exec --project emulatedproject --only auth \"run-s --npm-path npm test:browser:integration:local test:node:integration:local test:webdriver\"",
8484
"test:ci": "node ../../scripts/run_tests_in_ci.js -s test:all",
85-
"test:integration:local": "run-s test:node:integration:local test:browser:integration:local test:webdriver",
85+
"test:integration:local": "run-s --npm-path npm test:node:integration:local test:browser:integration:local test:webdriver",
8686
"test:browser": "karma start --single-run --local",
8787
"test:browser:unit": "karma start --single-run --unit",
8888
"test:browser:integration": "karma start --single-run --integration",
@@ -92,7 +92,7 @@
9292
"test:browser:unit:debug": "karma start --auto-watch --unit",
9393
"test:cordova": "karma start --single-run --cordova",
9494
"test:cordova:debug": "karma start --auto-watch --cordova",
95-
"test:node": "run-s test:node:unit test:node:integration:local",
95+
"test:node": "run-s --npm-path npm test:node:unit test:node:integration:local",
9696
"test:node:unit": "ts-node -O '{\"module\": \"commonjs\", \"target\": \"es6\"}' scripts/run_node_tests.ts",
9797
"test:node:integration": "ts-node -O '{\"module\": \"commonjs\", \"target\": \"es6\"}' scripts/run_node_tests.ts --integration",
9898
"test:node:integration:local": "ts-node -O '{\"module\": \"commonjs\", \"target\": \"es6\"}' scripts/run_node_tests.ts --integration --local",
@@ -124,7 +124,7 @@
124124
"@rollup/plugin-json": "4.1.0",
125125
"@rollup/plugin-strip": "2.1.0",
126126
"@types/express": "4.17.17",
127-
"chromedriver": "98.0.1",
127+
"chromedriver": "114.0.2",
128128
"rollup": "2.79.1",
129129
"rollup-plugin-sourcemaps": "0.6.3",
130130
"rollup-plugin-typescript2": "0.31.2",

0 commit comments

Comments
 (0)