feat($urlMatcherFactory): fail on bad parameters

Author: nateabele

src/urlMatcherFactory.js

@@ -248,8 +248,8 @@ UrlMatcher.prototype.format = function (values) {
     param = params[i];
     value = values[param];
     type  = this.params[param];
-    // TODO: Maybe we should throw on null here? It's not really good style
-    // to use '' and null interchangeabley
+
+    if (!type.is(value)) return null;
     if (value != null) result += encodeURIComponent(type.encode(value));
     result += segments[i + 1];
   }
@@ -270,11 +270,11 @@ function Type(options) {
 }
 
 Type.prototype.is = function(val, key) {
-  return angular.toJson(this.decode(this.encode(val))) === angular.toJson(val);
+  return true;
 };
 
 Type.prototype.encode = function(val, key) {
-  return String(val);
+  return val;
 };
 
 Type.prototype.decode = function(val, key) {

test/urlMatcherFactorySpec.js

@@ -196,5 +196,15 @@ describe("urlMatcherFactory", function () {
       expect(result.date.toUTCString()).toEqual('Wed, 26 Mar 2014 00:00:00 GMT');
       expect(m.format({ date: new Date(2014, 2, 26) })).toBe("/calendar/2014-03-26");
     });
+
+    it("should not match invalid typed parameter values", function () {
+      var m = new UrlMatcher('/users/{id:int}');
+
+      expect(m.exec('/users/1138').id).toBe(1138);
+      expect(m.exec('/users/alpha')).toBeNull();
+
+      expect(m.format({ id: 1138 })).toBe("/users/1138");
+      expect(m.format({ id: "alpha" })).toBeNull();
+    });
   });
 });