MINOR Fixed jquery.changetracker JSpec tests, added support for calli…

…ng public methods as string arguments on plugin (which was already assumed to be the case in the JSpec tests)
silverstripe · Dec 15, 2011 · ddad7fb · ddad7fb
1 parent 482324b
commit ddad7fb
Show file tree

Hide file tree

Showing 30 changed files with 12,008 additions and 253 deletions.
diff --git a/admin/javascript/jquery-changetracker/lib/jquery.changetracker.js b/admin/javascript/jquery-changetracker/lib/jquery.changetracker.js
@@ -90,7 +90,7 @@
 		 * @param DOMElement field
 		 */
 		this.resetField = function(field) {
-			return $(field).removeData('changetracker.origVal');
+			return $(field).removeData('changetracker.origVal').removeClass('changed');
 		};
 
 		/**
@@ -99,7 +99,16 @@
 		this.getFields = function() {
 			return this.find(options.fieldSelector).not(options.ignoreFieldSelector);
 		};
-
-		return this.initialize();
+
+		// Support invoking "public" methods as string arguments
+		if (typeof arguments[0] === 'string') {  
+      var property = arguments[1];  
+      var args = Array.prototype.slice.call(arguments);  
+      args.splice(0, 1);  
+      return this[arguments[0]].apply(this, args);  
+  	} else {
+    	return this.initialize();
+    }
+
 	};
 }(jQuery));
diff --git a/admin/javascript/jquery-changetracker/spec/.tmp_spec.html.49538~ b/admin/javascript/jquery-changetracker/spec/.tmp_spec.html.49538~
diff --git a/admin/javascript/jquery-changetracker/spec/.tmp_spec.html.8678~ b/admin/javascript/jquery-changetracker/spec/.tmp_spec.html.8678~
diff --git a/admin/javascript/jquery-changetracker/spec/.tmp_spec.html.970~ b/admin/javascript/jquery-changetracker/spec/.tmp_spec.html.970~
diff --git a/admin/javascript/jquery-changetracker/spec/.tmp_spec_selectors.html.13329~ b/admin/javascript/jquery-changetracker/spec/.tmp_spec_selectors.html.13329~
diff --git a/admin/javascript/jquery-changetracker/spec/.tmp_speed.html.70283~ b/admin/javascript/jquery-changetracker/spec/.tmp_speed.html.70283~
diff --git a/...t/jquery-changetracker/spec/spec.dom.html → ...script/jquery-changetracker/spec/dom.html b/...t/jquery-changetracker/spec/spec.dom.html → ...script/jquery-changetracker/spec/dom.html
@@ -1,17 +1,16 @@
 <html>
 	<head>
-		<link type="text/css" rel="stylesheet" href="../vendor/jspec/lib/jspec.css" />
-
-		<script src="../vendor/jquery-1.3.2.js"></script>
-		<script src="../vendor/jspec/lib/jspec.js"></script>
-		<script src="../vendor/jspec/lib/jspec.jquery.js"></script>
-
+		<link type="text/css" rel="stylesheet" href="./lib/jspec.css" />
+		<script src="http://code.jquery.com/jquery-1.7.1.js"></script>
+		<script src="./lib/jspec.js"></script>
+		<script src="./lib/jspec.xhr.js"></script>
 		<script src="../lib/jquery.changetracker.js"></script>
+		<script src="unit/spec.helper.js"></script>
 		<script>
 			function runSuites() {
 				JSpec
 				.exec('spec.changetracker.basics.js')
-				.run()
+				.run({ fixturePath: 'fixtures' })
 				.report()
 			}
 		</script>

diff --git a/admin/javascript/jquery-changetracker/spec/lib/images/bg.png b/admin/javascript/jquery-changetracker/spec/lib/images/bg.png
diff --git a/admin/javascript/jquery-changetracker/spec/lib/images/hr.png b/admin/javascript/jquery-changetracker/spec/lib/images/hr.png
diff --git a/admin/javascript/jquery-changetracker/spec/lib/images/loading.gif b/admin/javascript/jquery-changetracker/spec/lib/images/loading.gif
diff --git a/admin/javascript/jquery-changetracker/spec/lib/images/sprites.bg.png b/admin/javascript/jquery-changetracker/spec/lib/images/sprites.bg.png
diff --git a/admin/javascript/jquery-changetracker/spec/lib/images/sprites.png b/admin/javascript/jquery-changetracker/spec/lib/images/sprites.png
diff --git a/admin/javascript/jquery-changetracker/spec/lib/images/vr.png b/admin/javascript/jquery-changetracker/spec/lib/images/vr.png