From af70dee88e374d27dd8c0c8ec18af5c89541d4c9 Mon Sep 17 00:00:00 2001 From: Dan Heberden Date: Tue, 1 Feb 2011 00:18:16 -0800 Subject: [PATCH] Improve $.ajax method signature unit test and placement --- test/unit/ajax.js | 34 +++++++++++++++------------------- 1 file changed, 15 insertions(+), 19 deletions(-) diff --git a/test/unit/ajax.js b/test/unit/ajax.js index 25e0218..ad14d0a 100644 --- a/test/unit/ajax.js +++ b/test/unit/ajax.js @@ -38,6 +38,21 @@ test("jQuery.ajax() - success callbacks", function() { }); }); +test( "jQuery.ajax - multiple method signatures introduced in 1.5 ( #8107)", function() { + + expect( 4 ); + + stop(); + + jQuery.when( + jQuery.ajax().success(function() { ok( true, 'With no arguments' ); }), + jQuery.ajax('data/name.html').success(function() { ok( true, 'With only string URL argument' ); }), + jQuery.ajax('data/name.html', {} ).success(function() { ok( true, 'With string URL param and map' ); }), + jQuery.ajax({ url: 'data/name.html'} ).success(function() { ok( true, 'With only map' ); }) + ).then( start, start ); + +}); + test("jQuery.ajax() - success callbacks - (url, options) syntax", function() { expect( 8 ); @@ -2161,25 +2176,6 @@ test("jQuery.ajax - active counter", function() { ok( jQuery.active == 0, "ajax active counter should be zero: " + jQuery.active ); }); -test( "jQuery.ajax - multiple method signatures introduced in 1.5 ( #8107)", 4, function() { - - expect( 3 ); - - var i = 3; - - jQuery.ajaxSetup({success: function() { - ok( true, "Success"); - if ( ! --i ) start(); - }}); - - stop(); - jQuery.ajax('data/name.html'); - jQuery.ajax('data/name.html', {} ); - jQuery.ajax({ url: 'data/name.html'} ); - - jQuery.ajaxSetup({ success: null }); -}); - } //} \ No newline at end of file -- 1.7.10.4