diff options
author | Michał Gołębiowski-Owczarek <m.goleb@gmail.com> | 2020-03-02 22:15:06 +0100 |
---|---|---|
committer | Michał Gołębiowski-Owczarek <m.goleb@gmail.com> | 2020-03-02 22:18:10 +0100 |
commit | 5ea844f65a820ebba7ae7c82af7bfab58f873e5a (patch) | |
tree | 3b18d0c8755d4848713097979f64ef9880e48482 /test/unit | |
parent | 362075aeaa269f27ba2c2832026f211c2767d010 (diff) | |
download | jquery-5ea844f65a820ebba7ae7c82af7bfab58f873e5a.tar.gz jquery-5ea844f65a820ebba7ae7c82af7bfab58f873e5a.zip |
Tests: Pass a number of necessary done() calls to assert.async()
It is no longer needed to create `done` wrappers in tests that require
multiple async operations to complete.
Closes gh-4633
(cherry picked from commit 364476c3dc1231603ba61fc08068fa89fb095e1a)
Diffstat (limited to 'test/unit')
-rw-r--r-- | test/unit/basic.js | 8 | ||||
-rw-r--r-- | test/unit/ready.js | 12 |
2 files changed, 10 insertions, 10 deletions
diff --git a/test/unit/basic.js b/test/unit/basic.js index c42ee5450..9cb7d25be 100644 --- a/test/unit/basic.js +++ b/test/unit/basic.js @@ -4,14 +4,14 @@ if ( jQuery.ajax ) { QUnit.test( "ajax", function( assert ) { assert.expect( 4 ); - var done = jQuery.map( new Array( 3 ), function() { return assert.async(); } ); + var done = assert.async( 3 ); jQuery.ajax( { type: "GET", url: url( "mock.php?action=name&name=foo" ), success: function( msg ) { assert.strictEqual( msg, "bar", "Check for GET" ); - done.pop()(); + done(); } } ); @@ -21,14 +21,14 @@ QUnit.test( "ajax", function( assert ) { data: "name=peter", success: function( msg ) { assert.strictEqual( msg, "pan", "Check for POST" ); - done.pop()(); + done(); } } ); jQuery( "#first" ).load( url( "name.html" ), function() { assert.ok( /^ERROR/.test( jQuery( "#first" ).text() ), "Check if content was injected into the DOM" ); - done.pop()(); + done(); } ); } ); } diff --git a/test/unit/ready.js b/test/unit/ready.js index fd33c0a18..d1a728b47 100644 --- a/test/unit/ready.js +++ b/test/unit/ready.js @@ -107,31 +107,31 @@ QUnit.module( "ready" ); QUnit[ jQuery.when ? "test" : "skip" ]( "jQuery.when(jQuery.ready)", function( assert ) { assert.expect( 2 ); - var done = jQuery.map( new Array( 2 ), function() { return assert.async(); } ); + var done = assert.async( 2 ); whenified.then( function() { assert.ok( jQuery.isReady, "jQuery.when Deferred resolved" ); - done.pop()(); + done(); } ); jQuery.when( jQuery.ready ).then( function() { assert.ok( jQuery.isReady, "jQuery.when Deferred resolved" ); - done.pop()(); + done(); } ); } ); QUnit.test( "Promise.resolve(jQuery.ready)", function( assert ) { assert.expect( 2 ); - var done = jQuery.map( new Array( 2 ), function() { return assert.async(); } ); + var done = assert.async( 2 ); promisified.then( function() { assert.ok( jQuery.isReady, "Native promised resolved" ); - done.pop()(); + done(); } ); Promise.resolve( jQuery.ready ).then( function() { assert.ok( jQuery.isReady, "Native promised resolved" ); - done.pop()(); + done(); } ); } ); |