fix(ngAria): handle non-string model values

Fix for #10212. Compare the string value of the model
so that aria-checked will be properly applied to
type=radio in the case where the model is a number

Closes #10212

Author: Subra

src/ngAria/aria.js

@@ -228,13 +228,13 @@ ngAriaModule.directive('ngShow', ['$aria', function($aria) {
         if (needsTabIndex) {
           needsTabIndex = false;
           return function ngAriaRadioReaction(newVal) {
-            var boolVal = newVal === attr.value;
+            var boolVal = (newVal ? newVal.toString() : newVal) === attr.value;
             elem.attr('aria-checked', boolVal);
             elem.attr('tabindex', 0 - !boolVal);
           };
         } else {
           return function ngAriaRadioReaction(newVal) {
-            elem.attr('aria-checked', newVal === attr.value);
+            elem.attr('aria-checked', (newVal ? newVal.toString() : newVal) === attr.value);
           };
         }
       }

test/ngAria/ariaSpec.js

@@ -98,6 +98,19 @@ describe('$aria', function() {
       expect(element.eq(1).attr('aria-checked')).toBe('true');
     });
 
+    it('should handle non-string model values', function() {
+      var element = $compile('<input type="radio" ng-model="val" value="1">' +
+          '<input type="radio" ng-model="val" value="2">')(scope);
+
+      scope.$apply("val=1");
+      expect(element.eq(0).attr('aria-checked')).toBe('true');
+      expect(element.eq(1).attr('aria-checked')).toBe('false');
+
+      scope.$apply("val=2");
+      expect(element.eq(0).attr('aria-checked')).toBe('false');
+      expect(element.eq(1).attr('aria-checked')).toBe('true');
+    });
+
     it('should attach itself to role="radio"', function() {
       scope.$apply("val = 'one'");
       compileInput('<div role="radio" ng-model="val" value="{{val}}"></div>');