fix($resolve): resolve factories from string names

$resolve now correctly executes factory functions when provided the name of the factory as a string.

Closes #449

Author: nateabele

Diff for: src/resolve.js

@@ -42,7 +42,7 @@ function $Resolve(  $q,    $injector) {
       visited[key] = VISIT_IN_PROGRESS;
 
       if (isString(value)) {
-        plan.push(key, [ function() { return $injector.get(key); }], NO_DEPENDENCIES);
+        plan.push(key, [ function() { return $injector.get(value); }], NO_DEPENDENCIES);
       } else {
         var params = $injector.annotate(value);
         forEach(params, function (param) {

Diff for: test/resolveSpec.js

@@ -3,6 +3,12 @@ describe("resolve", function () {
   var $r, tick;
 
   beforeEach(module('ui.router.util'));
+  beforeEach(module(function($provide) {
+    $provide.factory('Foo', function() {
+      return "Working";
+    });
+  }));
+
   beforeEach(inject(function($resolve, $q) {
     $r = $resolve;
     tick = $q.flush;
@@ -292,6 +298,15 @@ describe("resolve", function () {
       r({ what: 'hi' });
       expect(trace).toEqual([ 'a: 1', 'a: hi' ]);
     });
+
+    it("resolves values from string factory names", function () {
+      var result, r = $r.study({ foo: "Foo" })().then(function(values) {
+        result = values['foo'];
+      });
+      tick();
+
+      expect(result).toBe("Working");
+    });
   });
 });