-
Notifications
You must be signed in to change notification settings - Fork 12
Update CMS micro-app locator after Shadow DOM removal #89
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
Open
IvanRadnevQA
wants to merge
27
commits into
AutomateThePlanet:main
Choose a base branch
from
Sportal365:smpInternal
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
7220112
Added error handling in browser shutdown method
IvanRadnevQA cd15743
Rearrange final and static keywords
IvanRadnevQA 5ab9053
Revert "Added error handling in browser shutdown method"
IvanRadnevQA 239a4a9
update selenium version to latest
i5kov 2c6196b
change usage of URI object
i5kov 9954e94
Fix create instance for no args constructor in InstanceFactory
i5kov f8fc544
replace --headless=old with --headless=new for executing tests in hea…
i5kov 9ed7926
Add method for mocking response for specific url
i5kov b11a0e6
Revert "Add method for mocking response for specific url"
i5kov 17abb53
Refactor inline and fix warning
i5kov 89e42ca
Add method for mock specific request by URL
i5kov fd60b67
Adding new device name with specific screen resolution
borisb83 aba847b
commit
hhristovQASportal fa6f06e
add --proxy-bypass-list=* to Chrome and headless options
hhristovQASportal 68582e1
remove 'proxy-bypass-list' argument from browser configuration
hhristovQASportal c223e2a
add bypass proxy in browser configuration to avoid WebSocket handshak…
hhristovQASportal 818afd6
add bypass proxy in browser configuration to avoid WebSocket handshak…
hhristovQASportal 6cb61e3
visual test maintenance - breakpoints updated, removing default style…
in6411s 5321725
add shadowRoot for CMS tests
hhristovQASportal 8fd89c7
add shadow root method for CMS
hhristovQASportal 87eda23
update cmsShadowHost()
hhristovQASportal 6cee063
update cmsShadowHost()
hhristovQASportal a043a63
add shadowHosType and use getShadowRoot from bellatrix
hhristovQASportal 614226f
add scrollToShellWindowBottom in bellatrix framework
hhristovQASportal 70843f5
change the location from WebComponent to BrowserService class
hhristovQASportal a038779
add cms tng to shadow host
hhristovQASportal f1910a2
Update CMS micro-app locator after Shadow DOM removal
IvanRadnevQA File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -252,6 +252,7 @@ private static WebDriver initializeDriverRegularMode() { | |
| var chromeOptions = new ChromeOptions(); | ||
| addDriverOptions(chromeOptions); | ||
| addDriverCapabilities(chromeOptions); | ||
| chromeOptions.addArguments("--proxy-bypass-list=news-tracker-sockets.api.staging.sportal365.com"); | ||
| chromeOptions.addArguments("--log-level=3","--remote-allow-origins=*", "--disable-search-engine-choice-screen"); | ||
| chromeOptions.setAcceptInsecureCerts(true); | ||
| chromeOptions.setCapability(CapabilityType.UNHANDLED_PROMPT_BEHAVIOUR, UnexpectedAlertBehaviour.ACCEPT); | ||
|
|
@@ -266,9 +267,10 @@ private static WebDriver initializeDriverRegularMode() { | |
| var chromeHeadlessOptions = new ChromeOptions(); | ||
| addDriverOptions(chromeHeadlessOptions); | ||
| chromeHeadlessOptions.setAcceptInsecureCerts(true); | ||
| chromeHeadlessOptions.addArguments("--proxy-bypass-list=news-tracker-sockets.api.staging.sportal365.com"); | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same here |
||
| chromeHeadlessOptions.addArguments("--log-level=3","--remote-allow-origins=*", "--disable-search-engine-choice-screen"); | ||
| chromeHeadlessOptions.setCapability(CapabilityType.UNHANDLED_PROMPT_BEHAVIOUR, UnexpectedAlertBehaviour.ACCEPT); | ||
| chromeHeadlessOptions.addArguments("--headless=old"); | ||
| chromeHeadlessOptions.addArguments("--headless=new"); | ||
| System.setProperty("webdriver.chrome.silentOutput", "true"); | ||
| if (shouldCaptureHttpTraffic) chromeHeadlessOptions.setProxy(proxyConfig); | ||
|
|
||
|
|
@@ -278,6 +280,7 @@ private static WebDriver initializeDriverRegularMode() { | |
| var chromeHeadlessOptions = new ChromeOptions(); | ||
| addDriverOptions(chromeHeadlessOptions); | ||
| chromeHeadlessOptions.setAcceptInsecureCerts(true); | ||
| chromeHeadlessOptions.addArguments("--proxy-bypass-list=news-tracker-sockets.api.staging.sportal365.com"); | ||
| chromeHeadlessOptions.addArguments("--log-level=3","--remote-allow-origins=*", "--disable-search-engine-choice-screen"); | ||
| chromeHeadlessOptions.setCapability(CapabilityType.UNHANDLED_PROMPT_BEHAVIOUR, UnexpectedAlertBehaviour.ACCEPT); | ||
|
|
||
|
|
@@ -300,25 +303,29 @@ private static WebDriver initializeDriverRegularMode() { | |
| var firefoxOptions = new FirefoxOptions(); | ||
| addDriverOptions(firefoxOptions); | ||
| firefoxOptions.setAcceptInsecureCerts(true); | ||
| firefoxOptions.addPreference("network.proxy.no_proxies_on", "news-tracker-sockets.api.staging.sportal365.com"); | ||
| if (shouldCaptureHttpTraffic) firefoxOptions.setProxy(proxyConfig); | ||
| driver = new FirefoxDriver(firefoxOptions); | ||
| } | ||
| case FIREFOX_HEADLESS -> { | ||
| var firefoxHeadlessOptions = new FirefoxOptions(); | ||
| addDriverOptions(firefoxHeadlessOptions); | ||
| firefoxHeadlessOptions.setAcceptInsecureCerts(true); | ||
| firefoxHeadlessOptions.addPreference("network.proxy.no_proxies_on", "news-tracker-sockets.api.staging.sportal365.com"); | ||
| firefoxHeadlessOptions.addArguments("--headless"); | ||
| if (shouldCaptureHttpTraffic) firefoxHeadlessOptions.setProxy(proxyConfig); | ||
| driver = new FirefoxDriver(firefoxHeadlessOptions); | ||
| } | ||
| case EDGE -> { | ||
| var edgeOptions = new EdgeOptions(); | ||
| edgeOptions.addArguments("--proxy-bypass-list=news-tracker-sockets.api.staging.sportal365.com"); | ||
| addDriverOptions(edgeOptions); | ||
| if (shouldCaptureHttpTraffic) edgeOptions.setProxy(proxyConfig); | ||
| driver = new EdgeDriver(edgeOptions); | ||
| } | ||
| case EDGE_HEADLESS -> { | ||
| var edgeOptions = new EdgeOptions(); | ||
| edgeOptions.addArguments("--proxy-bypass-list=news-tracker-sockets.api.staging.sportal365.com"); | ||
| edgeOptions.addArguments("--headless"); | ||
| edgeOptions.addArguments("--disable-gpu"); | ||
| addDriverOptions(edgeOptions); | ||
|
|
||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Project-specific urls cannot be merged into the framework - maybe you should keep them in the fork.