unit tests: minor updates

jquery · Feb 2, 2009 · 7ed45e6 · 7ed45e6
1 parent 4ad5310
commit 7ed45e6
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 5 deletions.
diff --git a/tests/unit/draggable/draggable_options.js b/tests/unit/draggable/draggable_options.js
@@ -28,11 +28,15 @@ test("{ addClasses: true }, default", function() {
 
 	el = $("<div></div>").draggable({ addClasses: true });
 	ok(el.is(".ui-draggable"), "'ui-draggable' class added");
+
+	el.draggable("destroy");
 });
 
 test("{ addClasses: false }", function() {
 	el = $("<div></div>").draggable({ addClasses: false });
 	ok(!el.is(".ui-draggable"), "'ui-draggable' class not added");
+
+	el.draggable("destroy");
 });
 
 test("{ appendTo: 'parent' }, default", function() {

diff --git a/tests/unit/droppable/droppable_defaults.js b/tests/unit/droppable/droppable_defaults.js
@@ -3,9 +3,9 @@
  */
 
 var droppable_defaults = {
-	addClasses: true,
 	accept: '*',
 	activeClass: false,
+	addClasses: true,
 	disabled: false,
 	greedy: false,
 	hoverClass: false,

diff --git a/tests/unit/droppable/droppable_options.js b/tests/unit/droppable/droppable_options.js
@@ -1,15 +1,21 @@
 /*
- * droppable_optinos.js
+ * droppable_options.js
  */
 (function($) {
 
 module("droppable: options");
 
-test("accept, selector", function() {
+test("{ accept '*' }, default ", function() {
+	equals(droppable_defaults.accept, '*');
+
+	ok(false, 'missing test - untested code is broken code');
+});
+
+test("{ accept: Selector }", function() {
 	ok(false, 'missing test - untested code is broken code');
 });
 
-test("accept, fn", function() {
+test("{ accept: function(draggable) }", function() {
 	ok(false, 'missing test - untested code is broken code');
 });
 

diff --git a/tests/unit/testsuite.js b/tests/unit/testsuite.js
@@ -88,6 +88,6 @@ function commonWidgetTests(widget, settings) {
 	module(widget + ": common widget");
 
     testWidgetDefaults(widget, settings.defaults);
-//    testSettingOptions(widget, options);
+    testSettingOptions(widget, options);
     testWidgetOverrides(widget);
 }