Skip to content

Commit

Permalink
Merge pull request #1859 from emanlove/mutable-default-value-1817
Browse files Browse the repository at this point in the history
Mutable default value 1817
  • Loading branch information
emanlove authored Nov 18, 2023
2 parents 7dd3766 + f539edb commit 68edcfe
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/CI.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ jobs:
strategy:
matrix:
python-version: [3.8, 3.11] # 3.12, pypy-3.9
rf-version: [4.1.3, 5.0.1, 6.0.1]
rf-version: [4.1.3, 5.0.1, 6.0.1, 6.1.1]

steps:
- uses: actions/checkout@v3
Expand Down
4 changes: 3 additions & 1 deletion src/SeleniumLibrary/keywords/browsermanagement.py
Original file line number Diff line number Diff line change
Expand Up @@ -334,7 +334,7 @@ def _make_new_browser(

@keyword
def create_webdriver(
self, driver_name: str, alias: Optional[str] = None, kwargs={}, **init_kwargs
self, driver_name: str, alias: Optional[str] = None, kwargs: Optional[dict] = None, **init_kwargs
) -> str:
"""Creates an instance of Selenium WebDriver.
Expand Down Expand Up @@ -362,6 +362,8 @@ def create_webdriver(
`Close All Browsers` keyword is used. See `Switch Browser` for an
example.
"""
if kwargs is None:
kwargs = {}
if not isinstance(kwargs, dict):
raise RuntimeError("kwargs must be a dictionary.")
for arg_name in kwargs:
Expand Down

0 comments on commit 68edcfe

Please sign in to comment.