aboutsummaryrefslogtreecommitdiffstats
path: root/test
diff options
context:
space:
mode:
authorMichał Gołębiowski-Owczarek <m.goleb@gmail.com>2021-01-11 18:20:36 +0100
committerMichał Gołębiowski-Owczarek <m.goleb@gmail.com>2021-01-11 18:41:36 +0100
commit7298e04faee8b01b335148cf5b91f84af9cf5826 (patch)
treeec0d398f082b35a8c2033895e160ace13cea6836 /test
parentbcd40aa7a34900cca3ca646258903f3069e9bd59 (diff)
downloadjquery-7298e04faee8b01b335148cf5b91f84af9cf5826.tar.gz
jquery-7298e04faee8b01b335148cf5b91f84af9cf5826.zip
Tests: Fix tests for not auto-executing scripts without dataType
Two issues are fixed in testing for responses with a script Content-Type not getting auto-executed unless an explicit `dataType: "script"` is provided: * the test is now using a correct "text/javascript" Content-Type; it was using "text/html" until now which doesn't really check if the fix works * the Node.js based version of the tests didn't account for an empty `header` query string parameter Closes gh-4824 Ref gh-2432 Ref gh-2588 Ref 39cdb8c9aa0fde68f733553ba050a2ba9d86474c (cherry picked from commit d38528b17a846b7ca4513b41150a05436546292d)
Diffstat (limited to 'test')
-rw-r--r--test/middleware-mockserver.js2
-rw-r--r--test/unit/ajax.js2
2 files changed, 2 insertions, 2 deletions
diff --git a/test/middleware-mockserver.js b/test/middleware-mockserver.js
index bb31eabfa..36216ecc1 100644
--- a/test/middleware-mockserver.js
+++ b/test/middleware-mockserver.js
@@ -62,7 +62,7 @@ var mocks = {
script: function( req, resp ) {
if ( req.query.header === "ecma" ) {
resp.writeHead( 200, { "content-type": "application/ecmascript" } );
- } else if ( req.query.header ) {
+ } else if ( "header" in req.query ) {
resp.writeHead( 200, { "content-type": "text/javascript" } );
} else {
resp.writeHead( 200, { "content-type": "text/html" } );
diff --git a/test/unit/ajax.js b/test/unit/ajax.js
index 3f301f18f..bcb234640 100644
--- a/test/unit/ajax.js
+++ b/test/unit/ajax.js
@@ -114,7 +114,7 @@ QUnit.module( "ajax", {
return {
create: function( options ) {
options.crossDomain = true;
- return jQuery.ajax( url( "mock.php?action=script" ), options );
+ return jQuery.ajax( url( "mock.php?action=script&header" ), options );
},
success: function() {
assert.ok( true, "success" );