fix(ngAria): add config option for role=button

Closes angular#11580

Author: marcysutton

Diff for: src/ngAria/aria.js

@@ -83,7 +83,8 @@ function $AriaProvider() {
     ariaMultiline: true,
     ariaValue: true,
     tabindex: true,
-    bindKeypress: true
+    bindKeypress: true,
+    clickRole: true
   };
 
   /**
@@ -102,6 +103,7 @@ function $AriaProvider() {
    *  - **tabindex** – `{boolean}` – Enables/disables tabindex tags
    *  - **bindKeypress** – `{boolean}` – Enables/disables keypress event binding on `<div>` and
    *    `<li>` elements with ng-click
+   *  - **clickRole** – `{boolean}` – Adds role=button to non-interactive elements with ng-click
    *
    * @description
    * Enables/disables various ARIA attributes
@@ -348,7 +350,7 @@ ngAriaModule.directive('ngShow', ['$aria', function($aria) {
 
         if (!$aria.isNodeOneOf(elem, $aria.nodeBlackList)) {
 
-          if (!elem.attr('role')) {
+          if ($aria.config('clickRole') && !elem.attr('role')) {
             elem.attr('role', 'button');
           }
 

Diff for: test/ngAria/ariaSpec.js

@@ -256,6 +256,18 @@ describe('$aria', function() {
     });
   });
 
+  describe('custom ng-clicks', function() {
+    beforeEach(configAriaProvider({
+      clickRole: false
+    }));
+    beforeEach(injectScopeAndCompiler);
+
+    it('should not add a role', function() {
+      compileElement('<radio-group ng-click="something"></radio-group>');
+      expect(element.attr('role')).toBeUndefined();
+    })
+  })
+
   describe('aria-checked when disabled', function() {
     beforeEach(configAriaProvider({
       ariaChecked: false