-
Notifications
You must be signed in to change notification settings - Fork 31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement WebdriverClassicDriver factory #26
Implement WebdriverClassicDriver factory #26
Conversation
spec/Behat/MinkExtension/ServiceContainer/Driver/WebdriverClassicFactorySpec.php
Outdated
Show resolved
Hide resolved
src/Behat/MinkExtension/ServiceContainer/Driver/WebdriverClassicFactory.php
Show resolved
Hide resolved
src/Behat/MinkExtension/ServiceContainer/Driver/EnvironmentCapabilities.php
Show resolved
Hide resolved
5023d3b
to
18a3f77
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving, because all points from my previous review were addressed.
If I use
But this is reporting that I do what I have done. It looks like somewhere the hyphen is being replaced by an underscore. Solution is to change the hyphen to an underscore here |
@benr77 , I'm new to Behat. Please describe the steps needed to create a test project with Behat + MinkExtension + this PR code so that it's easier to diagnose the problem. Yaml itself doesn't prohibit hyphen usage inside the key names. |
This underscore fix is now working as expected. Thank you. Hopefully this PR can get merged soon. |
FYI: The
|
Landed here trying to use minkphp/webdriver-classic-driver with my Behat setup. @benr77, could you share a |
@claudiu-cristea All I have is this behat.yaml, alongside using this branch instead of main - the important bit is using default:
suites:
default:
contexts:
- FeatureContext
extensions:
Behat\MinkExtension:
base_url: http://php
browser_name: chrome
sessions:
default:
webdriver_classic:
browser: chrome
wd_host: http://chrome:4444/wd/hub
capabilities:
browserName: chrome
platform: any
chrome:
switches:
- '--headless'
- '--window-size=1200,1100'
- '--disable-gpu'
- '--no-sandbox' |
Implements the driver at https://github.com/minkphp/webdriver-classic-driver
Fixes minkphp/webdriver-classic-driver#15