Skip to content

Commit 54aabff

Browse files
authored
fix(browser_type): ignoreDefaultArgs in launch() (#190)
1 parent 992cd77 commit 54aabff

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

playwright/browser_type.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -132,10 +132,9 @@ def normalize_launch_params(params: Dict) -> None:
132132
if "env" in params:
133133
params["env"] = {name: str(value) for [name, value] in params["env"].items()}
134134
if "ignoreDefaultArgs" in params:
135-
if isinstance(params["ignoreDefaultArgs"], bool):
135+
if params["ignoreDefaultArgs"] is True:
136136
params["ignoreAllDefaultArgs"] = True
137137
del params["ignoreDefaultArgs"]
138-
params["env"] = {name: str(value) for [name, value] in params["env"].items()}
139138
if "executablePath" in params:
140139
params["executablePath"] = str(Path(params["executablePath"]))
141140
if "downloadsPath" in params:

tests/async/test_issues.py

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
import pytest
2+
3+
4+
@pytest.mark.only_browser("chromium")
5+
async def test_issue_189(browser_type):
6+
browser = await browser_type.launch(ignoreDefaultArgs=["--mute-audio"])
7+
page = await browser.newPage()
8+
assert await page.evaluate("1 + 1") == 2
9+
await browser.close()

0 commit comments

Comments
 (0)