datastore: default ordering to ascending. fixes #134.

Author: stephenplusplus

lib/datastore/query.js

@@ -119,32 +119,32 @@ Query.prototype.hasAncestor = function(key) {
 };
 
 /**
- * Sort the results by a property name ascendingly or descendingly.
+ * Sort the results by a property name ascendingly or descendingly. By default,
+ * an ascending sort order will be used.
  *
  * *Reference: {@link http://goo.gl/mfegFR}*
  *
- * @param {string} order - Operator (+, -) + property to order by.
+ * @param {string} property - Optional operator (+, -) and property to order by.
  * @return {module:datastore/query}
  *
  * @example
  * ```js
  * // Sort by size ascendingly.
- * var companiesAscending = companyQuery.order('+size');
+ * var companiesAscending = companyQuery.order('size');
  *
  * // Sort by size descendingly.
  * var companiesDescending = companyQuery.order('-size');
  * ```
  */
-Query.prototype.order = function(order) {
+Query.prototype.order = function(property) {
   var q = util.extend(this, new Query());
-  var sign = order[0];
-  var fieldName = order.substr(1);
-  if (sign !== '-' && sign !== '+' ) {
-    throw new Error(
-        'Invalid order pattern. Expected "+fieldName" or "-fieldName".');
+  var sign = '+';
+  if (property[0] === '-' || property[0] === '+') {
+    sign = property[0];
+    property = property.substr(1);
   }
   q.orders = q.orders || [];
-  q.orders.push({ name: fieldName, sign: sign });
+  q.orders.push({ name: property, sign: sign });
   return q;
 };
 

test/datastore/query.js

@@ -101,10 +101,10 @@ describe('Query', function() {
     assert.equal(query.orders[1].sign, '-');
   });
 
-  it('should throw error is invalid sort sign is provided', function() {
-    assert.throws(function() {
-      new Query(['kind1']).order('*name');
-    }, /Invalid order pattern/);
+  it('should default ordering to ascending', function() {
+    var query = new Query(['kind1']).order('name');
+    assert.equal(query.orders[0].name, 'name');
+    assert.equal(query.orders[0].sign, '+');
   });
 
   it('should provide pagination with offset and limit', function() {