fix($urlMatcherFactory): detect injected functions

Correctly detects injectable functions annotated as arrays when registering new type definitions.

Author: nateabele

src/urlMatcherFactory.js

@@ -675,12 +675,16 @@ function $UrlMatcherFactory() {
     return this;
   }];
 
+  // To ensure proper order of operations in object configuration, and to allow internal
+  // types to be overridden, `flushTypeQueue()` waits until `$urlMatcherFactory` is injected
+  // before actually wiring up and assigning type definitions
   function flushTypeQueue() {
     forEach(typeQueue, function(type) {
       if (UrlMatcher.prototype.$types[type.name]) {
         throw new Error("A type named '" + type.name + "' has already been defined.");
       }
-      var def = new Type(isFunction(type.def) ? injector.invoke(type.def) : type.def);
+      var isAnnotated = isFunction(type.def) || isArray(type.def);
+      var def = new Type(isAnnotated ? injector.invoke(type.def) : type.def);
       UrlMatcher.prototype.$types[type.name] = def;
     });
   }

test/urlMatcherFactorySpec.js

@@ -227,6 +227,17 @@ describe("urlMatcherFactory", function () {
       expect($umf.type("myType").decode()).toBe($stateParams);
     }));
 
+    it("should accept annotated function definitions", inject(function ($stateParams) {
+      $umf.type("myAnnotatedType", ['$stateParams', function(s) {
+        return {
+          decode: function() {
+            return s;
+          }
+        };
+      }]);
+      expect($umf.type("myAnnotatedType").decode()).toBe($stateParams);
+    }));
+
     it("should match built-in types", function () {
       var m = new UrlMatcher("/{foo:int}/{flag:bool}");
       expect(m.exec("/1138/1")).toEqual({ foo: 1138, flag: true });