-
Notifications
You must be signed in to change notification settings - Fork 24
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
wd_demo.au3 - SetupOpera() suplemented + fixes #218
Conversation
as I said here: it is needed to merge and observe the issue on this tree which is for us only a DRAFT for #213 There is some issue with operadriver.exe behavior or in my understanding the code. |
There is some issue with creating capabilities for |
This issue was disscused before here: Using the following Func SetupOpera($bHeadless)
#forceref $bHeadless
_WD_Option('Driver', 'operadriver.exe')
_WD_Option('Port', 9515)
_WD_Option('DriverParams', '--verbose --log-path="' & @ScriptDir & '\opera.log"')
Local $sCapabilities = '{"capabilities": {"alwaysMatch": {"operaOptions": {"excludeSwitches": [ "enable-automation"]}}}}'
#Region - for testing stage only
;~ $sCapabilities = "{}"
;~ $sCapabilities = '{"capabilities":{}}'
;~ $sCapabilities = '{"capabilities":{"operaOptions": {}}}'
$sCapabilities = '{"capabilities":{"operaOptions": {"w3c":true}}}'
;~ $sCapabilities = '{"capabilities":{"operaOptions": {"w3c":true, "binary": "C:\\Users\\agatk\\AppData\\Local\\Programs\\Opera\\opera.exe"}}}'
#EndRegion - for testing stage only
Return $sCapabilities
EndFunc ;==>SetupOpera I always get
|
btw. |
Thanks to @Danp2 findings from here: #220 (comment) I'm going back to work on I also posted issue here: |
* Array of supported browsers * _WD_GetBrowserVersion: Eliminate hardcoding * _WD_UpdateDriver: Partial elimination of hardcoding * wd_demo.au3 Opera first commit (#214) * Expand and move Support Browsers array * Implement $_WD_BROWSER* enum * Utilize $_WD_BROWSER_Name * __WD_CapabilitiesInitialize + 'operaOptions' (#217) * __WD_CapabilitiesInitialize + 'operaOptions' * using $_WD_SupportedBrowsers * 'edge' >> 'msedge' ScriptBreakingChange * Readme update for opera (#216) * wd_core.au3 - operaOptions > goog:chromeOptions (#221) related to @Danp2 findings: #220 (comment) * added _WD_GetBrowserPath() with Opera path support (#223) * _WD_GetBrowserPath: Eliminate browser specific code * _WD_UpdateDriver: Support alternate browser location (#226) * wd_demo.au3 - SetupOpera() suplemented + fixes (#218) * Update change log (#230) * Fix [Unreleased] header * Update CHANGELOG.md * _WD_UpdateDriver: Eliminate remaining hardcoding (#231) * Expand $_WD_SupportedBrowsers * Add __WD_GetLatestWebdriverInfo * _WD_UpdateDriver: Use __WD_GetLatestWebdriverInfo * OPERA: _WD_GetBrowserVersion/Path refactoring (#235) * $_WD_ERROR_NotSupported * using $iIndex = @Extended * OPERA: _WD_UpdateDriver + __WD_UpdateExtractor (#236) * Opera final proposals (#237) Co-authored-by: Michał Lipok <[email protected]>
Pull request
Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
Please ensure you have read and noticed the checklist below.
Checklist
Put an
x
in the boxes that apply. If you're unsure about any of them, don't hesitate to ask. We are here to help!This is simply a reminder of what we are going to look for before merging your code.
Types of changes
Please check
x
the type of change your PR introduces:What is the current behavior?
$aBrowsers
have issues.SetupOpera()
was only a draft.What is the new behavior?
fixed
$aBrowsers
Suplemented
SetupOpera()
Additional context
Add any other context about the problem here.
#187
System under test
Opera