Fixed munge week expression and added test cases. fixes #979

http://source.rd.rcg.local/trac/eagle6/changeset/1276/Eagle6_SVN

lib/pipeline/expressions/WeekExpression.js

@@ -17,10 +17,15 @@ var WeekExpression = module.exports = (function(){
 
 	proto.addOperand = function addOperand(expr) {
 		this.checkArgLimit(1);
-		base.addOperand(expr);
+		base.prototype.addOperand.call(this, expr);
 	};
 
-	/** Takes a date and returns the week of the year as a number between 0 and 53. **/
+	/** 
+	* Takes a date and returns the week of the year as a number between 0 and 53. 
+	* Weeks begin on Sundays, and week 1 begins with the first Sunday of the year. 
+	* Days preceding the first Sunday of the year are in week 0. 
+	* This behavior is the same as the “%U” operator to the strftime standard library function.
+	**/
 	proto.evaluate = function evaluate(doc) {
 		this.checkArgCount(1);
 		var date = this.operands[0].evaluate(doc),

test/lib/pipeline/expressions/WeekExpression.js

@@ -0,0 +1,47 @@
+var assert = require("assert"),
+	WeekExpression = require("../../../../lib/pipeline/expressions/WeekExpression"),
+	Expression = require("../../../../lib/pipeline/expressions/Expression");
+
+module.exports = {
+
+	"WeekExpression": {
+
+		"constructor()": {
+
+			"should not throw Error when constructing without args": function testConstructor(){
+				assert.doesNotThrow(function(){
+					new WeekExpression();
+				});
+			}
+
+		},
+
+		"#getOpName()": {
+
+			"should return the correct op name; $week": function testOpName(){
+				assert.equal(new WeekExpression().getOpName(), "$week");
+			}
+
+		},
+
+		"#getFactory()": {
+
+			"should return the constructor for this class": function factoryIsConstructor(){
+				assert.strictEqual(new WeekExpression().getFactory(), undefined);
+			}
+
+		},
+
+		"#evaluate()": {
+
+			"should return week; 8 for 2013-02-18": function testStuff(){
+				assert.strictEqual(Expression.parseOperand({$week:"$someDate"}).evaluate({someDate:new Date("Mon Feb 18 2013 00:00:00 GMT-0500 (EST)")}), 7);
+			}
+
+		}
+
+	}
+
+};
+
+if (!module.parent)(new(require("mocha"))()).ui("exports").reporter("spec").addFile(__filename).run(process.exit);