From 8034cc38038ff1246cd98caecd517f9685d50f8b Mon Sep 17 00:00:00 2001 From: =?utf8?q?Scott=20Gonz=C3=A1lez?= Date: Mon, 24 Jan 2011 21:22:40 -0500 Subject: [PATCH] Widget tests: Cleanup. --- tests/unit/widget/widget.html | 24 ++++++++++++------------ tests/unit/widget/widget_core.js | 29 +++++++++++++---------------- tests/unit/widget/widget_tickets.js | 5 +---- 3 files changed, 26 insertions(+), 32 deletions(-) diff --git a/tests/unit/widget/widget.html b/tests/unit/widget/widget.html index c6fa54e41..d8fa2ecaf 100644 --- a/tests/unit/widget/widget.html +++ b/tests/unit/widget/widget.html @@ -1,22 +1,22 @@ - + jQuery UI Widget Test Suite - - - + + + - - - - + + + + - - - - + + + + diff --git a/tests/unit/widget/widget_core.js b/tests/unit/widget/widget_core.js index 269b897c5..26059b96b 100644 --- a/tests/unit/widget/widget_core.js +++ b/tests/unit/widget/widget_core.js @@ -1,6 +1,3 @@ -/* - * widget unit tests - */ (function( $ ) { module( "widget factory", { @@ -104,7 +101,7 @@ test( "jQuery usage", function() { }); shouldCreate = true; - var elem = $( "
" ) + var elem = $( "
" ) .bind( "testwidgetcreate", function() { ok( shouldCreate, "create event triggered on instantiation" ); }) @@ -150,7 +147,7 @@ test( "direct usage", function() { } }); - var elem = $( "
" )[ 0 ]; + var elem = $( "
" )[ 0 ]; shouldCreate = true; var instance = new $.ui.testWidget( {}, elem ); @@ -202,7 +199,7 @@ test("merge multiple option arguments", function() { }); } }); - $( "
" ).testWidget({ + $( "
" ).testWidget({ option1: "valuex", option2: "valuex", option3: "value3", @@ -249,7 +246,7 @@ test( "_getCreateOptions()", function() { }); test( "re-init", function() { - var div = $( "
" ), + var div = $( "
" ), actions = []; $.widget( "ui.testWidget", { @@ -335,7 +332,7 @@ test( ".option() - getter", function() { _create: function() {} }); - var div = $( "
" ).testWidget({ + var div = $( "
" ).testWidget({ foo: "bar", baz: 5, qux: [ "quux", "quuux" ] @@ -366,7 +363,7 @@ test( ".option() - delegate to ._setOptions()", function() { calls.push( options ); } }); - var div = $( "
" ).testWidget(); + var div = $( "
" ).testWidget(); calls = []; div.testWidget( "option", "foo", "bar" ); @@ -392,7 +389,7 @@ test( ".option() - delegate to ._setOption()", function() { }); } }); - var div = $( "
" ).testWidget(); + var div = $( "
" ).testWidget(); calls = []; div.testWidget( "option", "foo", "bar" ); @@ -419,7 +416,7 @@ test( ".enable()", function() { same( val, false, "disabled set to false" ); } }); - $( "
" ).testWidget().testWidget( "enable" ); + $( "
" ).testWidget().testWidget( "enable" ); }); test( ".disable()", function() { @@ -431,26 +428,26 @@ test( ".disable()", function() { same( val, true, "disabled set to true" ); } }); - $( "
" ).testWidget().testWidget( "disable" ); + $( "
" ).testWidget().testWidget( "disable" ); }); test( ".widget() - base", function() { $.widget( "ui.testWidget", { _create: function() {} }); - var div = $( "
" ).testWidget(); + var div = $( "
" ).testWidget(); same( div[0], div.testWidget( "widget" )[0]); }); test( ".widget() - overriden", function() { - var wrapper = $( "
" ); + var wrapper = $( "
" ); $.widget( "ui.testWidget", { _create: function() {}, widget: function() { return wrapper; } }); - same( wrapper[0], $( "
" ).testWidget().testWidget( "widget" )[0] ); + same( wrapper[0], $( "
" ).testWidget().testWidget( "widget" )[0] ); }); test( "._trigger() - no event, no ui", function() { @@ -670,4 +667,4 @@ test( "auto-destroy - .detach()", function() { $( "#widget" ).testWidget().detach(); }); -})( jQuery ); +}( jQuery ) ); diff --git a/tests/unit/widget/widget_tickets.js b/tests/unit/widget/widget_tickets.js index 21f44b54c..0267c8ff4 100644 --- a/tests/unit/widget/widget_tickets.js +++ b/tests/unit/widget/widget_tickets.js @@ -1,6 +1,3 @@ -/* - * widget unit tests - */ (function( $ ) { module( "widget: tickets" ); @@ -43,4 +40,4 @@ test( "#5830 - Widget: Using inheritance overwrites the base classes options", f delete $.ui.testWidgetExtension; }); -})( jQuery ); +}( jQuery ) ); -- 2.39.5