Merge pull request #87 from Shyp/throw-on-unknown-operator

Throw an error if the operator is unknown

Makefile

@@ -1,5 +1,10 @@
+.PHONY: test test-unit install test-integration
+
 ROOT=$(shell pwd)
 
+install:
+	npm install
+
 test: test-unit
 
 test-unit:
@@ -12,4 +17,4 @@ test-integration:
 	ln -s $(ROOT) node_modules/sails-postgresql/node_modules/waterline-sequel
 	rm -rf node_modules/sails-mysql/node_modules/waterline-sequel
 	ln -s $(ROOT) node_modules/sails-mysql/node_modules/waterline-sequel
-	@NODE_ENV=test node test/integration/runnerDispatcher.js
+	@NODE_ENV=test node test/integration/runnerDispatcher.js

sequel/lib/criteriaProcessor.js

@@ -830,6 +830,11 @@ CriteriaProcessor.prototype.prepareCriterion = function prepareCriterion(key, va
       }
 
       break;
+
+    default:
+      var err = new Error('Unknown filtering operator: "' + key + "\". Should be 'startsWith', '>', 'contains' or similar");
+      err.operator = key;
+      throw err;
   }
 
   // Bump paramCount

test/unit/index.test.js

@@ -82,6 +82,14 @@ describe('Sequel', function () {
     });
   });
 
+  describe('queries with an unknown operator', function () {
+    it('throws an error when the operator is unknown', function() {
+      var sequel = new Sequel(schema);
+      assert.throws(sequel.find.bind(sequel, 'bar', { id: { 'in': [ 1, 2 ] } }),
+        Error, "Unknown filtering operator: \"in\". Should be 'startsWith', '>', 'contains' or similar");
+    });
+  });
+
   describe('.find() with schema name', function () {
     var _options = _.extend({}, options, {schemaName: {'foo':'myschema','oddity':'anotherschema','bat':'public'}});
     // Loop through the query objects and test them against the `.find()` method.
@@ -99,5 +107,5 @@ describe('Sequel', function () {
         done();
       });
     });
-  });  
+  });
 });