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

fix The XhrSandbox.openNativeXhr function calls non native the xhr.setRequestHeader function (close #1252) #1254

Merged
merged 3 commits into from
Aug 3, 2017

Conversation

LavrovArtem
Copy link
Contributor

@testcafe-build-bot
Copy link
Collaborator

❌ Tests for the commit ff3128e have failed. See details:

@testcafe-build-bot
Copy link
Collaborator

❌ Tests for the commit f4bcdb9 have failed. See details:

@LavrovArtem
Copy link
Contributor Author

@testcafe-build-bot \retest

@testcafe-build-bot
Copy link
Collaborator

✅ Tests for the commit f4bcdb9 have passed. See details:

@@ -17,6 +17,13 @@ QUnit.testDone(function () {
iframeSandbox.off(iframeSandbox.RUN_TASK_SCRIPT_EVENT, initIframeTestHandler);
});

function findPrototypeWithProp (obj, prop) {
Copy link
Contributor

Choose a reason for hiding this comment

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

getPrototypeFromChainContainsProp (obj, prop)

@@ -43,6 +50,23 @@ test('createNativeXHR', function () {
ok(xhr instanceof nativeMethods.XMLHttpRequest);

window.XMLHttpRequest = nativeMethods.XMLHttpRequest;

var isWrappedFunctionRE = /^return 'function wrapped'$/;
Copy link
Contributor

Choose a reason for hiding this comment

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

var isWrappedFunctionRE = /^return 'function is wrapped'$/;

@testcafe-build-bot
Copy link
Collaborator

✅ Tests for the commit 150eabb have passed. See details:

@AlexanderMoskovkin AlexanderMoskovkin merged commit d09f0a8 into DevExpress:master Aug 3, 2017
@LavrovArtem LavrovArtem deleted the i1252 branch August 3, 2017 11:37
AndreyBelym pushed a commit to AndreyBelym/testcafe-hammerhead that referenced this pull request Feb 28, 2019
…etRequestHeader function` (close DevExpress#1252) (DevExpress#1254)

* fix `The XhrSandbox.openNativeXhr function calls non native the xhr.setRequestHeader function` (close DevExpress#1252)

* fix test

* test refactoring
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.

4 participants