Skip to content

Fix chrome browser name to lower case#10906

Merged
christian-bromann merged 4 commits intowebdriverio:mainfrom
HananArgov:fix-chrome-browser-name-to-lower-case
Aug 13, 2023
Merged

Fix chrome browser name to lower case#10906
christian-bromann merged 4 commits intowebdriverio:mainfrom
HananArgov:fix-chrome-browser-name-to-lower-case

Conversation

@HananArgov
Copy link
Contributor

Proposed changes

//: # fix when chrome is capitalized in browserName in capabilities

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Checklist

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • I have added proper type definitions for new commands (if appropriate)

Further comments

Reviewers: @webdriverio/project-committers

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Aug 10, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

Copy link
Member

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 comments, also mind updating the branch? The build is passing again.

)
})

it('should start chrome driver with capitalized Chrome browserName', async () => {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of a new unit test can we incorporate this into an existing test?

HananArgov and others added 2 commits August 13, 2023 12:03
fix according to comment

Co-authored-by: Christian Bromann <git@bromann.dev>
Copy link
Member

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@christian-bromann christian-bromann added the PR: Polish 💅 PRs that contain improvements on existing features label Aug 13, 2023
@christian-bromann christian-bromann merged commit 7c73a30 into webdriverio:main Aug 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: Polish 💅 PRs that contain improvements on existing features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants