fix($state): registering ignores Object properties

Closes #511

Author: nateabele

src/state.js

@@ -161,7 +161,7 @@ function $StateProvider(   $urlRouterProvider,   $urlMatcherFactory,           $
 
     var name = state.name;
     if (!isString(name) || name.indexOf('@') >= 0) throw new Error("State must have a valid name");
-    if (states[name]) throw new Error("State '" + name + "'' is already defined");
+    if (states.hasOwnProperty(name)) throw new Error("State '" + name + "'' is already defined");
 
     // Get parent name
     var parentName =

test/stateSpec.js

@@ -87,6 +87,17 @@ describe('state', function () {
     expect($state.current).toBe(state);
   }
 
+  describe('provider', function () {
+    it ('should ignore Object properties when registering states', function () {
+      expect(function() {
+        stateProvider.state('toString', { url: "/to-string" });
+      }).not.toThrow();
+      expect(function() {
+        stateProvider.state('watch', { url: "/watch" });
+      }).not.toThrow();
+    });
+  });
+
   describe('.transitionTo()', function () {
     it('returns a promise for the target state', inject(function ($state, $q) {
       var trans = $state.transitionTo(A, {});