]> source.dussan.org Git - jquery-ui.git/commitdiff
Tests: Handle jQuery git returning `null` for empty attributes
authorScott González <scott.gonzalez@gmail.com>
Mon, 23 Mar 2015 22:42:23 +0000 (18:42 -0400)
committerScott González <scott.gonzalez@gmail.com>
Wed, 25 Mar 2015 19:55:53 +0000 (15:55 -0400)
jQuery now returns `null` for empty attributes instead of `undefined`.

Closes gh-1516

tests/unit/core/core.js
tests/unit/dialog/dialog_core.js
tests/unit/menu/menu_options.js
tests/unit/progressbar/progressbar_core.js
tests/unit/tooltip/tooltip_core.js
tests/unit/tooltip/tooltip_methods.js

index 2f3657605121e0e266e33c961a217d149f6d0055..a0d0a2de12f87a216d959ad5218de9a750e11641 100644 (file)
@@ -127,11 +127,11 @@ test( "outerHeight(true) - setter", function() {
 test( "uniqueId / removeUniqueId", function() {
        expect( 3 );
        var el = $( "img" ).eq( 0 );
-       strictEqual( el.attr( "id" ), undefined, "element has no initial id" );
+       equal( el.attr( "id" ), null, "element has no initial id" );
        el.uniqueId();
        ok( /ui-id-\d+$/.test( el.attr( "id" ) ), "element has generated id" );
        el.removeUniqueId();
-       strictEqual( el.attr( "id" ), undefined, "unique id has been removed from element" );
+       equal( el.attr( "id" ), null, "unique id has been removed from element" );
 });
 
 })( jQuery );
index f600722fc2f352be9d9b9630762c71326a4fa81d..73d24fed0af76300785afb3df63999cd0fbdd69b 100644 (file)
@@ -78,7 +78,7 @@ test( "ARIA", function() {
        element.remove();
 
        element = $("<div><div aria-describedby='section2'><p id='section2'>descriotion</p></div></div>").dialog();
-       strictEqual( element.dialog( "widget" ).attr( "aria-describedby" ), undefined, "no aria-describedby added, as already present in markup" );
+       equal( element.dialog( "widget" ).attr( "aria-describedby" ), null, "no aria-describedby added, as already present in markup" );
        element.remove();
 });
 
index 39ba62e3c2c3bd662aeb04b000cc3c448ae77119..3df5473348d4f3bd72e59d04a1fc54a094e00197 100644 (file)
@@ -107,11 +107,11 @@ test( "{ role: null }", function( assert ) {
                }),
                items = element.find( "li" );
        expect( 2 + 3 * items.length );
-       strictEqual( element.attr( "role" ), undefined );
+       equal( element.attr( "role" ), null );
        ok( items.length > 0, "number of menu items" );
        items.each(function( item ) {
                assert.hasClasses( $( this ), "ui-menu-item" );
-               equal( $( this ).find( ".ui-menu-item-wrapper" ).attr( "role" ), undefined,
+               equal( $( this ).find( ".ui-menu-item-wrapper" ).attr( "role" ), null,
                        "menu item ("+ item + ") role" );
                equal( $( this ).find( ".ui-menu-item-wrapper" ).attr( "tabindex" ), "-1",
                        "tabindex for menu item ("+ item + ")" );
index 686e93b432b03ad945990f7554e20805e82b5238..60ff3141128328852cf96764256cbdd11598b052 100644 (file)
@@ -55,7 +55,7 @@ test( "accessibility", function() {
        element.progressbar( "option", "value", false );
        equal( element.attr( "aria-valuemin" ), 0, "aria-valuemin" );
        equal( element.attr( "aria-valuemax" ), 150, "aria-valuemax" );
-       strictEqual( element.attr( "aria-valuenow" ), undefined, "aria-valuenow" );
+       equal( element.attr( "aria-valuenow" ), null, "aria-valuenow" );
 });
 
 }( jQuery ) );
index 10ebb2829bb4839d5969d58f385f9348b9843e17..d4520a4ea682eb3f7c0b7f3f658d4a27ab5e4459 100644 (file)
@@ -38,8 +38,7 @@ test( "accessibility", function() {
        equal( element.attr( "aria-describedby" ), "fixture-span " + tooltipId,
                "multiple describedby when open" );
 
-       // strictEqual to distinguish between .removeAttr( "title" ) and .attr( "title", "" )
-       strictEqual( element.attr( "title" ), undefined, "no title when open" );
+       equal( element.attr( "title" ), null, "no title when open" );
        equal( liveRegion.children().length, 1 );
        equal( liveRegion.children().last().html(), "..." );
        element.tooltip( "close" );
index ea69216d097e834d84b4eeae5ff55cceda73c9f2..f027d5226204027207a00ceee16ff60db5d539f7 100644 (file)
@@ -70,7 +70,7 @@ test( "enable/disable", function( assert ) {
        assert.lacksClasses( element.tooltip( "widget" ), "ui-state-disabled" );
        ok( !element.tooltip( "widget" ).attr( "aria-disabled" ), "element doesn't get aria-disabled" );
        assert.lacksClasses( element.tooltip( "widget" ), "ui-tooltip-disabled" );
-       strictEqual( tooltip.attr( "title" ), undefined, "title removed on disable" );
+       equal( tooltip.attr( "title" ), null, "title removed on disable" );
 
        element.tooltip( "open" );
        equal( $( ".ui-tooltip" ).length, 0, "open does nothing when disabled" );