fix(ngController): allow dots in a controller name

vojtajina · vojtajina · commit de2cdb0658b8 · 2013-04-30T14:38:14.000-07:00
The issue was introduced in cd38cbf
diff --git a/src/ng/controller.js b/src/ng/controller.js
@@ -12,7 +12,7 @@
  */
 function $ControllerProvider() {
   var controllers = {},
-      CNTRL_REG = /^(\w+)(\s+as\s+(\w+))?$/;
+      CNTRL_REG = /^(\S+)(\s+as\s+(\w+))?$/;
 
 
   /**
diff --git a/test/ng/controllerSpec.js b/test/ng/controllerSpec.js
@@ -90,13 +90,27 @@ describe('$controller', function() {
   });
 
 
-  it('should publish controller instance into scope', function() {
-    var scope = {};
+  describe('ctrl as syntax', function() {
 
-    $controllerProvider.register('FooCtrl', function() { this.mark = 'foo'; });
+    it('should publish controller instance into scope', function() {
+      var scope = {};
 
-    var foo = $controller('FooCtrl as foo', {$scope: scope});
-    expect(scope.foo).toBe(foo);
-    expect(scope.foo.mark).toBe('foo');
+      $controllerProvider.register('FooCtrl', function() { this.mark = 'foo'; });
+
+      var foo = $controller('FooCtrl as foo', {$scope: scope});
+      expect(scope.foo).toBe(foo);
+      expect(scope.foo.mark).toBe('foo');
+    });
+
+
+    it('should allow controllers with dots', function() {
+      var scope = {};
+
+      $controllerProvider.register('a.b.FooCtrl', function() { this.mark = 'foo'; });
+
+      var foo = $controller('a.b.FooCtrl as foo', {$scope: scope});
+      expect(scope.foo).toBe(foo);
+      expect(scope.foo.mark).toBe('foo');
+    });
   });
 });
