From abbf6a79a91cdc64005fcbb28d98ad71a27b61c0 Mon Sep 17 00:00:00 2001 From: Scott González Date: Mon, 25 Jun 2012 14:51:51 -0400 Subject: Tests: Update support comments. --- tests/unit/core/core.js | 2 +- tests/unit/tooltip/tooltip_core.js | 4 +++- tests/unit/tooltip/tooltip_methods.js | 4 +++- 3 files changed, 7 insertions(+), 3 deletions(-) (limited to 'tests') diff --git a/tests/unit/core/core.js b/tests/unit/core/core.js index b3020179d..41bc2ab5b 100644 --- a/tests/unit/core/core.js +++ b/tests/unit/core/core.js @@ -158,7 +158,7 @@ test( "uniqueId / removeUniqueId", function() { // support: jQuery <1.6.2 // support: IE <8 - // We should use equal( id, undefined ) when dropping jQuery 1.6.1 support (or IE6/7) + // We should use strictEqual( id, undefined ) when dropping jQuery 1.6.1 support (or IE6/7) ok( !el.attr( "id" ), "element has no initial id" ); el.uniqueId(); ok( /ui-id-\d+$/.test( el.attr( "id" ) ), "element has generated id" ); diff --git a/tests/unit/tooltip/tooltip_core.js b/tests/unit/tooltip/tooltip_core.js index 52941784f..2b39253a2 100644 --- a/tests/unit/tooltip/tooltip_core.js +++ b/tests/unit/tooltip/tooltip_core.js @@ -34,7 +34,9 @@ test( "accessibility", function() { equal( element.attr( "aria-describedby" ), "fixture-span " + tooltipId, "multiple describedby when open" ); // strictEqual to distinguish between .removeAttr( "title" ) and .attr( "title", "" ) - // TODO actually use strictEqual(..., undefined, msg) when dropping jQuery 1.6 support (or IE6/7) + // support: jQuery <1.6.2 + // support: IE <8 + // We should use strictEqual( ..., undefined ) when dropping jQuery 1.6.1 support (or IE6/7) ok( !element.attr( "title" ), "no title when open" ); element.tooltip( "close" ); equal( element.attr( "aria-describedby" ), "fixture-span", diff --git a/tests/unit/tooltip/tooltip_methods.js b/tests/unit/tooltip/tooltip_methods.js index 33ce44764..ef783cbd3 100644 --- a/tests/unit/tooltip/tooltip_methods.js +++ b/tests/unit/tooltip/tooltip_methods.js @@ -42,7 +42,9 @@ test( "enable/disable", function() { element.tooltip( "disable" ); equal( $( ".ui-tooltip" ).length, 0, "no tooltip when disabled" ); - // TODO use equal(..., undefined, msg) when dropping jQuery 1.6 support (or IE6/7) + // support: jQuery <1.6.2 + // support: IE <8 + // We should use strictEqual( ..., undefined ) when dropping jQuery 1.6.1 support (or IE6/7) ok( !tooltip.attr( "title" ), "title removed on disable" ); element.tooltip( "open" ); -- cgit v1.2.3