Skip to content
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

feat: added the possibility to disable a cross-domain emulation #2806

Merged
merged 3 commits into from
Oct 31, 2022

Conversation

Aleksey28
Copy link
Collaborator

@Aleksey28 Aleksey28 commented Oct 27, 2022

Purpose

Add --disable-cross-domain option

Approach

Describe how your changes address the issue or implement the desired functionality in as much detail as possible.

References

DevExpress/testcafe#7350
DevExpress/testcafe#7022
testcafe-hammerhead-28.0.1.zip

Pre-Merge TODO

  • Write tests for your proposed changes
  • Make sure that existing tests do not fail

@Aleksey28 Aleksey28 temporarily deployed to CI October 27, 2022 14:19 Inactive
@Aleksey28 Aleksey28 changed the title Gh7022 feat: added the possibility to disable a cross-domain emulation Oct 27, 2022
@Aleksey28 Aleksey28 self-assigned this Oct 28, 2022
@miherlosev miherlosev merged commit 2171e71 into DevExpress:master Oct 31, 2022
AndreyBelym added a commit to DevExpress/testcafe that referenced this pull request Nov 2, 2022
<!--
Thank you for your contribution.

Before making a PR, please read our contributing guidelines at

https://github.com/DevExpress/testcafe/blob/master/CONTRIBUTING.md#code-contribution

We recommend creating a *draft* PR, so that you can mark it as 'ready
for review' when you are done.
-->
[closes #7022]

## Purpose
Add `--disable-cross-domain` option

## Approach
1. Add handling CLI option `--disable-cross-domain`
2. Add switching cross-domain

## References
#7022
DevExpress/testcafe-hammerhead#2806

## Pre-Merge TODO
- [ ] Write tests for your proposed changes
- [ ] Make sure that existing tests do not fail

Co-authored-by: Andrey Belym <belym.a.2105@gmail.com>
Co-authored-by: Andrey Belym <andrey.belym@devexpress.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants