feat($compile): put custom annotations on DDO

Author: petebacondarwin

src/ng/compile.js

@@ -1106,7 +1106,7 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
       }
 
       var template = (!options.template && !options.templateUrl ? '' : options.template);
-      return {
+      var ddo = {
         controller: controller,
         controllerAs: identifierForController(options.controller) || options.controllerAs || '$ctrl',
         template: makeInjectable(template),
@@ -1117,8 +1117,18 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
         restrict: 'E',
         require: options.require
       };
+
+      // Copy annotations (starting with $) over to the DDO
+      forEach(options, function(val, key) {
+        if (key.charAt(0) === '$') ddo[key] = val;
+      });
+
+      return ddo;
     }
 
+    // TODO(pete) remove the following `forEach` before we release 1.6.0
+    // when the router no longer looks for the annotations on the controller
+
     // Copy any annotation properties (starting with $) over to the factory function
     // These could be used by libraries such as the new component router
     forEach(options, function(val, key) {

test/ng/compileSpec.js

@@ -10493,15 +10493,15 @@ describe('$compile', function() {
       }));
     });
 
-    it('should add additional annotations to the controller constructor', function() {
+    it('should expose additional annotations on the directive definition object', function() {
       var myModule = angular.module('my', []).component('myComponent', {
         $canActivate: 'canActivate',
         $routeConfig: 'routeConfig',
         $customAnnotation: 'XXX'
       });
       module('my');
       inject(function(myComponentDirective) {
-        expect(myComponentDirective[0].controller).toEqual(jasmine.objectContaining({
+        expect(myComponentDirective[0]).toEqual(jasmine.objectContaining({
           $canActivate: 'canActivate',
           $routeConfig: 'routeConfig',
           $customAnnotation: 'XXX'
@@ -10515,9 +10515,11 @@ describe('$compile', function() {
         controller: 'SomeNamedController'
       });
       module('my');
-      expect(function() {
-        inject(function(myComponentDirective) {});
-      }).not.toThrow();
+      inject(function(myComponentDirective) {
+        expect(myComponentDirective[0]).toEqual(jasmine.objectContaining({
+          $customAnnotation: 'XXX'
+        }));
+      });
     });
 
     it('should return ddo with reasonable defaults', function() {