Fix #1987 by only doing remote <script> type ajax with GET requests. All other types...

[jquery.git] / test / unit / ajax.js

diff --git a/test/unit/ajax.js b/test/unit/ajax.js
index 5274b68..40ec964 100644 (file)
--- a/test/unit/ajax.js
+++ b/test/unit/ajax.js
@@ -218,16 +218,26 @@ test("synchronous request with callbacks", function() {
 });
 
 test("pass-through request object", function() {
-       expect(6);
+       expect(8);
        stop(true);
        
        var target = "data/name.html";
-       var count = 0;
+       var successCount = 0;
+       var errorCount = 0;
+  var errorEx = "";
        var success = function() {
-               // Re-enabled because a bug was found in the unit test that probably caused the problem
-               if(++count == 5)
-                       start();
+               successCount++;
        };
+       $("#foo").ajaxError(function (e, xml, s, ex) {
+               errorCount++;
+    errorEx += ": " + xml.status;
+       });
+       $("#foo").one('ajaxStop', function () {
+               equals(successCount, 5, "Check all ajax calls successful");
+               equals(errorCount, 0, "Check no ajax errors (status" + errorEx + ")");
+               $("#foo").unbind('ajaxError');
+               start();
+       });
        
        ok( $.get(url(target), success), "get" );
        ok( $.post(url(target), success), "post" );
@@ -388,6 +398,10 @@ test("$.ajax() - JSONP, Local", function() {
                success: function(data){
                        ok( data.data, "JSON results returned (GET, no callback)" );
                        plus();
+               },
+               error: function(data){
+                       ok( false, "Ajax error JSON (GET, no callback)" );
+                       plus();
                }
        });
 
@@ -397,6 +411,10 @@ test("$.ajax() - JSONP, Local", function() {
                success: function(data){
                        ok( data.data, "JSON results returned (GET, url callback)" );
                        plus();
+               },
+               error: function(data){
+                       ok( false, "Ajax error JSON (GET, url callback)" );
+                       plus();
                }
        });
 
@@ -407,6 +425,10 @@ test("$.ajax() - JSONP, Local", function() {
                success: function(data){
                        ok( data.data, "JSON results returned (GET, data callback)" );
                        plus();
+               },
+               error: function(data){
+                       ok( false, "Ajax error JSON (GET, data callback)" );
+                       plus();
                }
        });
 
@@ -417,6 +439,10 @@ test("$.ajax() - JSONP, Local", function() {
                success: function(data){
                        ok( data.data, "JSON results returned (GET, data obj callback)" );
                        plus();
+               },
+               error: function(data){
+                       ok( false, "Ajax error JSON (GET, data obj callback)" );
+                       plus();
                }
        });
 
@@ -427,6 +453,10 @@ test("$.ajax() - JSONP, Local", function() {
                success: function(data){
                        ok( data.data, "JSON results returned (POST, no callback)" );
                        plus();
+               },
+               error: function(data){
+                       ok( false, "Ajax error JSON (GET, data obj callback)" );
+                       plus();
                }
        });
 
@@ -438,6 +468,10 @@ test("$.ajax() - JSONP, Local", function() {
                success: function(data){
                        ok( data.data, "JSON results returned (POST, data callback)" );
                        plus();
+               },
+               error: function(data){
+                       ok( false, "Ajax error JSON (POST, data callback)" );
+                       plus();
                }
        });
 
@@ -449,6 +483,10 @@ test("$.ajax() - JSONP, Local", function() {
                success: function(data){
                        ok( data.data, "JSON results returned (POST, data obj callback)" );
                        plus();
+               },
+               error: function(data){
+                       ok( false, "Ajax error JSON (POST, data obj callback)" );
+                       plus();
                }
        });
 });
@@ -469,6 +507,10 @@ test("$.ajax() - JSONP, Remote", function() {
                success: function(data){
                        ok( data.data, "JSON results returned (GET, no callback)" );
                        plus();
+               },
+               error: function(data){
+                       ok( false, "Ajax error JSON (GET, no callback)" );
+                       plus();
                }
        });
 
@@ -478,6 +520,10 @@ test("$.ajax() - JSONP, Remote", function() {
                success: function(data){
                        ok( data.data, "JSON results returned (GET, url callback)" );
                        plus();
+               },
+               error: function(data){
+                       ok( false, "Ajax error JSON (GET, url callback)" );
+                       plus();
                }
        });
 
@@ -488,6 +534,10 @@ test("$.ajax() - JSONP, Remote", function() {
                success: function(data){
                        ok( data.data, "JSON results returned (GET, data callback)" );
                        plus();
+               },
+               error: function(data){
+                       ok( false, "Ajax error JSON (GET, data callback)" );
+                       plus();
                }
        });
 
@@ -498,6 +548,10 @@ test("$.ajax() - JSONP, Remote", function() {
                success: function(data){
                        ok( data.data, "JSON results returned (GET, data obj callback)" );
                        plus();
+               },
+               error: function(data){
+                       ok( false, "Ajax error JSON (GET, data obj callback)" );
+                       plus();
                }
        });
 });
@@ -519,6 +573,25 @@ test("$.ajax() - script, Remote", function() {
        });
 });
 
+test("$.ajax() - script, Remote with POST", function() {
+       expect(3);
+
+       var base = window.location.href.replace(/\?.*$/, "");
+
+       stop();
+
+       $.ajax({
+               url: base + "data/test.js",
+               type: "POST",
+               dataType: "script",
+               success: function(data, status){
+                       ok( foobar, "Script results returned (GET, no callback)" );
+                       equals( status, "success", "Script results returned (GET, no callback)" );
+                       start();
+               }
+       });
+});
+
 test("$.getJSON(String, Hash, Function) - JSON array", function() {
        expect(4);
        stop();