From edfc8631d1449ee30c0c760aec6ca6ec23d40ecf Mon Sep 17 00:00:00 2001 From: =?utf8?q?Scott=20Gonz=C3=A1lez?= Date: Thu, 26 Aug 2010 10:59:51 -0400 Subject: [PATCH] Widget Tests: Coding standards. --- tests/unit/widget/widget_core.js | 162 ++++++++++++++++--------------- 1 file changed, 83 insertions(+), 79 deletions(-) diff --git a/tests/unit/widget/widget_core.js b/tests/unit/widget/widget_core.js index 63ea31576..cc0019825 100644 --- a/tests/unit/widget/widget_core.js +++ b/tests/unit/widget/widget_core.js @@ -1,92 +1,95 @@ /* * widget unit tests */ -(function($) { +(function( $ ) { -module('widget factory', { +module( "widget factory", { teardown: function() { delete $.ui.testWidget; } }); -test('widget creation', function() { +test( "widget creation", function() { var myPrototype = { _create: function() {}, creationTest: function() {} }; - - $.widget('ui.testWidget', myPrototype); - ok($.isFunction($.ui.testWidget), 'constructor was created'); - equals('object', typeof $.ui.testWidget.prototype, 'prototype was created'); - equals($.ui.testWidget.prototype._create, myPrototype._create, 'create function is copied over'); - equals($.ui.testWidget.prototype.creationTest, myPrototype.creationTest, 'random function is copied over'); - equals($.ui.testWidget.prototype.option, $.Widget.prototype.option, 'option method copied over from base widget'); + + $.widget( "ui.testWidget", myPrototype ); + ok( $.isFunction( $.ui.testWidget ), "constructor was created" ); + equals( "object", typeof $.ui.testWidget.prototype, "prototype was created" ); + equals( $.ui.testWidget.prototype._create, myPrototype._create, + "create function is copied over" ); + equals( $.ui.testWidget.prototype.creationTest, myPrototype.creationTest, + "random function is copied over" ); + equals( $.ui.testWidget.prototype.option, $.Widget.prototype.option, + "option method copied over from base widget" ); }); -test('jQuery usage', function() { - expect(10); - +test( "jQuery usage", function() { + expect( 10 ); + var shouldInit = false; - - $.widget('ui.testWidget', { + + $.widget( "ui.testWidget", { getterSetterVal: 5, _create: function() { - ok(shouldInit, 'init called on instantiation'); + ok( shouldInit, "init called on instantiation" ); }, - methodWithParams: function(param1, param2) { - ok(true, 'method called via .pluginName(methodName)'); - equals(param1, 'value1', - 'parameter passed via .pluginName(methodName, param)'); - equals(param2, 'value2', - 'multiple parameters passed via .pluginName(methodName, param, param)'); + methodWithParams: function( param1, param2 ) { + ok( true, "method called via .pluginName(methodName)" ); + equals( param1, "value1", + "parameter passed via .pluginName(methodName, param)" ); + equals( param2, "value2", + "multiple parameters passed via .pluginName(methodName, param, param)" ); return this; }, - getterSetterMethod: function(val) { - if (val) { + getterSetterMethod: function( val ) { + if ( val ) { this.getterSetterVal = val; } else { return this.getterSetterVal; } } }); - + shouldInit = true; - var elem = $('
').testWidget(); + var elem = $( "
" ).testWidget(); shouldInit = false; - - var instance = elem.data('testWidget'); - equals(typeof instance, 'object', 'instance stored in .data(pluginName)'); - equals(instance.element[0], elem[0], 'element stored on widget'); - var ret = elem.testWidget('methodWithParams', 'value1', 'value2'); - equals(ret, elem, 'jQuery object returned from method call'); - - ret = elem.testWidget('getterSetterMethod'); - equals(ret, 5, 'getter/setter can act as getter'); - ret = elem.testWidget('getterSetterMethod', 30); - equals(ret, elem, 'getter/setter method can be chainable'); - equals(instance.getterSetterVal, 30, 'getter/setter can act as setter'); + + var instance = elem.data( "testWidget" ); + equals( typeof instance, "object", "instance stored in .data(pluginName)" ); + equals( instance.element[0], elem[0], "element stored on widget" ); + var ret = elem.testWidget( "methodWithParams", "value1", "value2" ); + equals( ret, elem, "jQuery object returned from method call" ); + + ret = elem.testWidget( "getterSetterMethod" ); + equals( ret, 5, "getter/setter can act as getter" ); + ret = elem.testWidget( "getterSetterMethod", 30 ); + equals( ret, elem, "getter/setter method can be chainable" ); + equals( instance.getterSetterVal, 30, "getter/setter can act as setter" ); }); -test('direct usage', function() { - expect(9); - +test( "direct usage", function() { + expect( 9 ); + var shouldInit = false; - - $.widget('ui.testWidget', { + + $.widget( "ui.testWidget", { getterSetterVal: 5, _create: function() { - ok(shouldInit, 'init called on instantiation'); + ok( shouldInit, "init called on instantiation" ); }, - methodWithParams: function(param1, param2) { - ok(true, 'method called dirctly'); - equals(param1, 'value1', 'parameter passed via direct call'); - equals(param2, 'value2', 'multiple parameters passed via direct call'); - + methodWithParams: function( param1, param2 ) { + ok( true, "method called dirctly" ); + equals( param1, "value1", "parameter passed via direct call" ); + equals( param2, "value2", "multiple parameters passed via direct call" ); + return this; }, - getterSetterMethod: function(val) { - if (val) { + getterSetterMethod: function( val ) { + if ( val ) { this.getterSetterVal = val; } else { return this.getterSetterVal; @@ -94,29 +97,30 @@ test('direct usage', function() { } }); - var elem = $('
')[0]; + var elem = $( "
" )[ 0 ]; shouldInit = true; - var instance = new $.ui.testWidget({}, elem); + var instance = new $.ui.testWidget( {}, elem ); shouldInit = false; + + equals( $( elem ).data( "testWidget" ), instance, + "instance stored in .data(pluginName)" ); + equals( instance.element[ 0 ], elem, "element stored on widget" ); - equals($(elem).data('testWidget'), instance, 'instance stored in .data(pluginName)'); - equals(instance.element[0], elem, 'element stored on widget'); - - var ret = instance.methodWithParams('value1', 'value2'); - equals(ret, instance, 'plugin returned from method call'); - + var ret = instance.methodWithParams( "value1", "value2" ); + equals( ret, instance, "plugin returned from method call" ); + ret = instance.getterSetterMethod(); - equals(ret, 5, 'getter/setter can act as getter'); - instance.getterSetterMethod(30); - equals(instance.getterSetterVal, 30, 'getter/setter can act as setter'); + equals( ret, 5, "getter/setter can act as getter" ); + instance.getterSetterMethod( 30 ); + equals( instance.getterSetterVal, 30, "getter/setter can act as setter" ); }); -test('merge multiple option arguments', function() { - expect(1); - $.widget("ui.testWidget", { +test("merge multiple option arguments", function() { + expect( 1 ); + $.widget( "ui.testWidget", { _create: function() { - same(this.options, { + same( this.options, { disabled: false, option1: "value1", option2: "value2", @@ -128,7 +132,7 @@ test('merge multiple option arguments', function() { }); } }); - $("
").testWidget({ + $( "
" ).testWidget({ option1: "valuex", option2: "valuex", option3: "value3", @@ -148,7 +152,7 @@ test('merge multiple option arguments', function() { }); }); -test("re-init", function() { +test( "re-init", function() { var div = $( "
" ), actions = []; @@ -236,7 +240,7 @@ test( ".option() - setter", function() { test( ".enable()", function() { expect( 2 ); - $.widget("ui.testWidget", { + $.widget( "ui.testWidget", { _create: function() {}, _setOption: function( key, val ) { same( key, "disabled", "_setOption called with disabled option" ); @@ -248,7 +252,7 @@ test( ".enable()", function() { test( ".disable()", function() { expect( 2 ); - $.widget("ui.testWidget", { + $.widget( "ui.testWidget", { _create: function() {}, _setOption: function( key, val ) { same( key, "disabled", "_setOption called with disabled option" ); @@ -258,23 +262,23 @@ test( ".disable()", function() { $( "
" ).testWidget().testWidget( "disable" ); }); -test(".widget() - base", function() { - $.widget("ui.testWidget", { +test( ".widget() - base", function() { + $.widget( "ui.testWidget", { _create: function() {} }); - var div = $("
").testWidget(); - same(div[0], div.testWidget("widget")[0]); + var div = $( "
" ).testWidget(); + same( div[0], div.testWidget( "widget" )[0]); }); -test(".widget() - overriden", function() { - var wrapper = $("
"); - $.widget("ui.testWidget", { +test( ".widget() - overriden", function() { + 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] ); }); -})(jQuery); +})( jQuery ); -- 2.39.5