Decided not to include zero fields, for consistency.

ef4 · ef4 · commit 795ef164fb7f · 2011-04-29T10:52:55.000-04:00
diff --git a/lib/types.js b/lib/types.js
@@ -111,6 +111,10 @@ var parseInterval = function(val) {
       if (i.minutes) i.minutes *= -1;
       if (i.seconds) i.seconds *= -1;
   }
+  for (field in i){
+      if (i[field] == 0)
+	  delete i[field];
+  }
   return i;
 };
 
diff --git a/test/unit/client/typed-query-results-tests.js b/test/unit/client/typed-query-results-tests.js
@@ -117,7 +117,7 @@ test('typed results', function() {
     dataTypeID: 1186,
     actual: '1 day -00:00:03',
     expected: function(val) {
-      assert.deepEqual(val, {'days':1, 'hours': 0, 'minutes': 0, 'seconds':-3})
+      assert.deepEqual(val, {'days':1, 'seconds':-3})
     }
   }];
 

Original file line number	Diff line number	Diff line change
`@@ -117,7 +117,7 @@ test('typed results', function() {`
`117`	`117`	`dataTypeID: 1186,`
`118`	`118`	`actual: '1 day -00:00:03',`
`119`	`119`	`expected: function(val) {`
`120`		`- assert.deepEqual(val, {'days':1, 'hours': 0, 'minutes': 0, 'seconds':-3})`
	`120`	`+ assert.deepEqual(val, {'days':1, 'seconds':-3})`
`121`	`121`	`}`
`122`	`122`	`}];`
`123`	`123`