Merge pull request #96 from bagby/aria-checked-string

set aria-checked attribute value to string

Author: raycohen

addon/components/radio-button-input.js

@@ -33,13 +33,23 @@ export default Component.extend({
     'value',
     'ariaLabelledby:aria-labelledby',
     'ariaDescribedby:aria-describedby',
-    'checked:aria-checked'
+    'checkedStr:aria-checked'
   ],
 
   checked: computed('groupValue', 'value', function() {
     return isEqual(this.get('groupValue'), this.get('value'));
   }).readOnly(),
 
+  checkedStr: computed('checked', function() {
+    let checked = this.get('checked');
+
+    if (typeof checked === 'boolean') {
+      return checked.toString();
+    }
+
+    return null;
+  }),
+
   invokeChangedAction() {
     let value = this.get('value');
     let changedAction = this.get('changed');

tests/unit/components/radio-button-test.js

@@ -384,7 +384,7 @@ test('it binds `aria-describedby` when specified', function(assert) {
 });
 
 test('it updates when clicked, and supports legacy-style string `changed` actions', function(assert) {
-  assert.expect(5);
+  assert.expect(7);
 
   let changedActionCallCount = 0;
   this.on('changed', function() {
@@ -403,12 +403,14 @@ test('it updates when clicked, and supports legacy-style string `changed` action
 
   assert.equal(changedActionCallCount, 0);
   assert.equal(this.$('input').prop('checked'), false);
+  assert.equal(this.$('input').attr('aria-checked'), 'false');
 
   run(() => {
     this.$('input').trigger('click');
   });
 
   assert.equal(this.$('input').prop('checked'), true, 'updates element property');
+  assert.equal(this.$('input').attr('aria-checked'), 'true', 'updates element property');
   assert.equal(this.get('groupValue'), 'component-value', 'updates groupValue');
 
   assert.equal(changedActionCallCount, 1);