From 9e4c041c528740f6c5bfe121c3deea7b6832f9e8 Mon Sep 17 00:00:00 2001
From: Jimmy Jia <tesrin@gmail.com>
Date: Fri, 16 Oct 2015 19:26:30 -0400
Subject: [PATCH] [fixed] Incorrect 'aria-selected' on NavItem

---
 src/NavItem.js      |  4 +++-
 test/NavItemSpec.js | 16 +++++++++++++---
 2 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/src/NavItem.js b/src/NavItem.js
index 0ef949fb08..f30243cad4 100644
--- a/src/NavItem.js
+++ b/src/NavItem.js
@@ -55,11 +55,13 @@ const NavItem = React.createClass({
 
     if (!role && href === '#') {
       linkProps.role = 'button';
+    } else if (role === 'tab') {
+      linkProps['aria-selected'] = active;
     }
 
     return (
       <li {...props} role="presentation" className={classNames(props.className, classes)}>
-        <SafeAnchor {...linkProps} aria-selected={active} aria-controls={ariaControls}>
+        <SafeAnchor {...linkProps} aria-controls={ariaControls}>
           { children }
         </SafeAnchor>
       </li>
diff --git a/test/NavItemSpec.js b/test/NavItemSpec.js
index 62fca77ae0..c7b5eb84fa 100644
--- a/test/NavItemSpec.js
+++ b/test/NavItemSpec.js
@@ -135,14 +135,24 @@ describe('NavItem', () => {
       assert.ok(linkElement.hasAttribute('aria-controls'));
     });
 
-    it('Should add aria-selected to the link', () => {
+    it('Should add aria-selected to the link when role is "tab"', () => {
       let instance = ReactTestUtils.renderIntoDocument(
-          <NavItem active>Item content</NavItem>
+          <NavItem role="tab" active>Item content</NavItem>
         );
 
       let linkElement = ReactTestUtils.findRenderedDOMComponentWithTag(instance, 'a');
 
-      assert.equal(linkElement.getAttribute('aria-selected'), 'true');
+      expect(linkElement.getAttribute('aria-selected')).to.equal('true');
+    });
+
+    it('Should not add aria-selected to the link when role is not "tab"', () => {
+      let instance = ReactTestUtils.renderIntoDocument(
+          <NavItem role="button" active>Item content</NavItem>
+        );
+
+      let linkElement = ReactTestUtils.findRenderedDOMComponentWithTag(instance, 'a');
+
+      expect(linkElement.getAttribute('aria-selected')).to.not.exist;
     });
 
     it('Should pass role down', () => {