From 23a7a113ac61bb12ad6a3775553d5ac1426f82f6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Scott=20Gonz=C3=A1lez?= Date: Mon, 6 Apr 2015 15:07:04 -0400 Subject: [PATCH] Widget: Convert tests to new infrastructure Ref #10119 Ref gh-1528 --- tests/unit/widget/widget.html | 28 ++++++++------------------- tests/unit/widget/widget_animation.js | 6 ++++++ tests/unit/widget/widget_classes.js | 11 +++++++---- tests/unit/widget/widget_core.js | 10 +++++++--- tests/unit/widget/widget_extend.js | 7 +++++-- 5 files changed, 33 insertions(+), 29 deletions(-) diff --git a/tests/unit/widget/widget.html b/tests/unit/widget/widget.html index ac106ea32..8ccce8525 100644 --- a/tests/unit/widget/widget.html +++ b/tests/unit/widget/widget.html @@ -4,27 +4,15 @@ jQuery UI Widget Test Suite - - - - - - - + - - - - - - - diff --git a/tests/unit/widget/widget_animation.js b/tests/unit/widget/widget_animation.js index 8ef55aa11..4e9d93ccf 100644 --- a/tests/unit/widget/widget_animation.js +++ b/tests/unit/widget/widget_animation.js @@ -1,3 +1,7 @@ +define( [ + "jquery", + "ui/widget" +], function( $ ) { module( "widget animation", (function() { var show = $.fn.show, @@ -255,3 +259,5 @@ asyncTest( "show: object(effect)", function() { next(); }); }); + +} ); diff --git a/tests/unit/widget/widget_classes.js b/tests/unit/widget/widget_classes.js index 0202d2683..1b4a45120 100644 --- a/tests/unit/widget/widget_classes.js +++ b/tests/unit/widget/widget_classes.js @@ -1,4 +1,7 @@ -(function( $ ) { +define( [ + "jquery", + "ui/widget" +], function( $ ) { module( "widget factory classes", { setup: function() { @@ -115,10 +118,10 @@ test( ".option() - classes setter", function( assert ) { "Appending a class to the current value works as expected" ); }); -test( ".destroy() - class removal", function() { +test( ".destroy() - class removal", function( assert ) { expect( 1 ); - domEqual( "#widget", function() { + assert.domEqual( "#widget", function() { $( "#widget" ).classesWidget().classesWidget( "destroy" ); }); }); @@ -140,4 +143,4 @@ test( "._add/_remove/_toggleClass()", function( assert ) { elementLacksClasses( widget, "remove", assert ); }); -}( jQuery ) ); +} ); diff --git a/tests/unit/widget/widget_core.js b/tests/unit/widget/widget_core.js index 74d6af9da..22832765b 100644 --- a/tests/unit/widget/widget_core.js +++ b/tests/unit/widget/widget_core.js @@ -1,4 +1,8 @@ -(function( $ ) { +define( [ + "jquery", + "lib/common", + "ui/widget" +], function( $, common ) { module( "widget factory", { teardown: function() { @@ -9,7 +13,7 @@ module( "widget factory", { } }); -TestHelpers.testJshint( "widget" ); +common.testJshint( "widget" ); test( "widget creation", function() { expect( 5 ); @@ -1507,4 +1511,4 @@ test( "$.widget.bridge() - widgetFullName", function() { equal( typeof instance, "object", "instance stored in .data(widgetFullName)" ); }); -}( jQuery ) ); +} ); diff --git a/tests/unit/widget/widget_extend.js b/tests/unit/widget/widget_extend.js index f5d81bbdc..a5a8cccaa 100644 --- a/tests/unit/widget/widget_extend.js +++ b/tests/unit/widget/widget_extend.js @@ -1,4 +1,7 @@ -(function( $ ) { +define( [ + "jquery", + "ui/widget" +], function( $ ) { test( "$.widget.extend()", function() { expect( 27 ); @@ -106,4 +109,4 @@ test( "$.widget.extend()", function() { deepEqual( input, output, "don't clone arrays" ); }); -}( jQuery )); +} );