From: Tim Wood Date: Thu, 14 Jun 2012 16:54:36 +0000 (-0700) Subject: Whitespace OCD and adding hasownproperty to tests X-Git-Tag: 1.7.0~16 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=0a2665c77174a81eaf4020238f8fa82a1227db2c;p=thirdparty%2Fmoment.git Whitespace OCD and adding hasownproperty to tests #399 --- diff --git a/test/moment/mutable.js b/test/moment/mutable.js index e3c2c0298..4d45c94b1 100644 --- a/test/moment/mutable.js +++ b/test/moment/mutable.js @@ -1,48 +1,53 @@ var moment = require("../../moment"); exports.mutable = { - "manipulation methods" : function(test) { + "manipulation methods" : function (test) { var mutableMethods = { - 'year': function (m){ return m.year(2011); }, - 'month': function (m){ return m.month(1); }, - 'date': function (m){ return m.date(9); }, - 'hours': function (m){ return m.hours(7); }, - 'minutes': function (m){ return m.minutes(33); }, - 'seconds': function (m){ return m.seconds(44); }, - 'milliseconds': function (m){ return m.milliseconds(55); }, - 'day': function (m){ return m.day(2); }, - 'startOf': function (m){ return m.startOf('week') }, - 'endOf': function (m){ return m.endOf('week') }, - 'add': function (m){ return m.add('days', 1) }, - 'subtract': function (m){ return m.subtract('years', 2) }, - 'local': function (m){ return m.local() }, - 'utc': function (m){ return m.utc() } + 'year': function (m){ return m.year(2011); }, + 'month': function (m){ return m.month(1); }, + 'date': function (m){ return m.date(9); }, + 'hours': function (m){ return m.hours(7); }, + 'minutes': function (m){ return m.minutes(33); }, + 'seconds': function (m){ return m.seconds(44); }, + 'milliseconds': function (m){ return m.milliseconds(55); }, + 'day': function (m){ return m.day(2); }, + 'startOf': function (m){ return m.startOf('week') }, + 'endOf': function (m){ return m.endOf('week') }, + 'add': function (m){ return m.add('days', 1) }, + 'subtract': function (m){ return m.subtract('years', 2) }, + 'local': function (m){ return m.local() }, + 'utc': function (m){ return m.utc() } }; test.expect(14); - for (method in mutableMethods){ - var d = new Date(); - var d2 = mutableMethods[method](moment(d)).toDate(); - test.equal(d, d2, method + "() should be mutable"); + for (method in mutableMethods) { + if (mutableMethods.hasOwnProperty(method)) { + var d = new Date(); + var d2 = mutableMethods[method](moment(d)).toDate(); + test.equal(d, d2, method + "() should be mutable"); + } } test.done(); }, - "non mutable methods" : function(test) { + + "non mutable methods" : function (test) { var nonMutableMethods = { - 'sod': function (m){ return m.sod() }, - 'eod': function (m){ return m.eod() } + 'sod': function (m){ return m.sod() }, + 'eod': function (m){ return m.eod() } }; test.expect(2); for (method in nonMutableMethods){ - var d = new Date(); - var d2 = nonMutableMethods[method](moment(d)).toDate(); - test.notEqual(d, d2, method + "() should not be mutable"); + if (nonMutableMethods.hasOwnProperty(method)) { + var d = new Date(); + var d2 = nonMutableMethods[method](moment(d)).toDate(); + test.notEqual(d, d2, method + "() should not be mutable"); + } } test.done();