Skip to content

Commit d79bb57

Browse files
authored
chore(bidi): rename bidi-firefox-stable to moz-firefox channel (#35611)
1 parent 6262a12 commit d79bb57

File tree

4 files changed

+9
-9
lines changed

4 files changed

+9
-9
lines changed

.github/workflows/tests_bidi.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ jobs:
3232
strategy:
3333
fail-fast: false
3434
matrix:
35-
channel: [bidi-chromium, bidi-firefox-nightly]
35+
channel: [bidi-chromium, moz-firefox]
3636
steps:
3737
- uses: actions/checkout@v4
3838
if: github.event_name != 'workflow_dispatch'
@@ -47,7 +47,7 @@ jobs:
4747
- run: npm run build
4848
- run: npx playwright install --with-deps chromium
4949
if: matrix.channel == 'bidi-chromium'
50-
- if: matrix.channel == 'bidi-firefox-nightly'
50+
- if: matrix.channel == 'moz-firefox'
5151
id: install_firefox
5252
run: |
5353
npx -y @puppeteer/browsers install firefox@nightly |\

packages/playwright-core/src/server/registry/index.ts

+4-4
Original file line numberDiff line numberDiff line change
@@ -475,7 +475,7 @@ function readDescriptors(browsersJSON: BrowsersJSON): BrowsersJSONDescriptor[] {
475475

476476
export type BrowserName = 'chromium' | 'firefox' | 'webkit' | 'bidi';
477477
type InternalTool = 'ffmpeg' | 'winldd' | 'firefox-beta' | 'chromium-tip-of-tree' | 'chromium-headless-shell' | 'chromium-tip-of-tree-headless-shell' | 'android';
478-
type BidiChannel = 'bidi-firefox-stable' | 'bidi-firefox-beta' | 'bidi-firefox-nightly' | 'bidi-chrome-canary' | 'bidi-chrome-stable' | 'bidi-chromium';
478+
type BidiChannel = 'moz-firefox' | 'moz-firefox-beta' | 'moz-firefox-nightly' | 'bidi-chrome-canary' | 'bidi-chrome-stable' | 'bidi-chromium';
479479
type ChromiumChannel = 'chrome' | 'chrome-beta' | 'chrome-dev' | 'chrome-canary' | 'msedge' | 'msedge-beta' | 'msedge-dev' | 'msedge-canary';
480480
const allDownloadable = ['android', 'chromium', 'firefox', 'webkit', 'ffmpeg', 'firefox-beta', 'chromium-tip-of-tree', 'chromium-headless-shell', 'chromium-tip-of-tree-headless-shell'];
481481

@@ -682,17 +682,17 @@ export class Registry {
682682
'win32': `\\Microsoft\\Edge SxS\\Application\\msedge.exe`,
683683
}));
684684

685-
this._executables.push(this._createBidiFirefoxChannel('bidi-firefox-stable', {
685+
this._executables.push(this._createBidiFirefoxChannel('moz-firefox', {
686686
'linux': '/snap/bin/firefox',
687687
'darwin': '/Applications/Firefox.app/Contents/MacOS/firefox',
688688
'win32': '\\Mozilla Firefox\\firefox.exe',
689689
}));
690-
this._executables.push(this._createBidiFirefoxChannel('bidi-firefox-beta', {
690+
this._executables.push(this._createBidiFirefoxChannel('moz-firefox-beta', {
691691
'linux': '/opt/firefox-beta/firefox',
692692
'darwin': '/Applications/Firefox.app/Contents/MacOS/firefox',
693693
'win32': '\\Mozilla Firefox\\firefox.exe',
694694
}));
695-
this._executables.push(this._createBidiFirefoxChannel('bidi-firefox-nightly', {
695+
this._executables.push(this._createBidiFirefoxChannel('moz-firefox-nightly', {
696696
'linux': '/opt/firefox-nightly/firefox',
697697
'darwin': '/Applications/Firefox Nightly.app/Contents/MacOS/firefox',
698698
'win32': '\\Mozilla Firefox\\firefox.exe',

tests/bidi/README.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ git clone https://github.com/microsoft/playwright.git
77
cd playwright
88
npm run build # call `npm run watch` for watch mode
99
npx playwright install chromium
10-
npm run biditest -- --project='bidi-firefox-beta-*'
10+
npm run biditest -- --project='moz-firefox-*'
1111
```
1212

1313
To install beta channel of Firefox, run the following command in the project root:
@@ -16,7 +16,7 @@ npx -y @puppeteer/browsers install firefox@beta
1616
```
1717
After that you need to pass custom firefox binary path to the test runner via `BIDI_FFPATH`:
1818
```sh
19-
BIDI_FFPATH='/Users/myself/playwright/firefox/mac_arm-beta_138.0b5/Firefox.app/Contents/MacOS/firefox' npm run biditest -- --project='bidi-firefox-beta-*'
19+
BIDI_FFPATH='/Users/myself/playwright/firefox/mac_arm-beta_138.0b5/Firefox.app/Contents/MacOS/firefox' npm run biditest -- --project='moz-firefox-*'
2020
```
2121

2222
For custom Chromium path use `BIDI_CRPATH`.

tests/bidi/playwright.config.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ const getExecutablePath = (browserName: BrowserName) => {
7676

7777
const browserToChannels = {
7878
'_bidiChromium': ['bidi-chromium', 'bidi-chrome-canary', 'bidi-chrome-stable'],
79-
'_bidiFirefox': ['bidi-firefox-nightly', 'bidi-firefox-beta', 'bidi-firefox-stable'],
79+
'_bidiFirefox': ['moz-firefox'],
8080
};
8181

8282
for (const [key, channels] of Object.entries(browserToChannels)) {

0 commit comments

Comments
 (0)