fix(DropDownItem): when href use a tag (#377)

TheSharpieOne · eddywashere · commit 96616af78813 · 2017-03-30T17:19:29.000-07:00
fixes #367
diff --git a/src/DropdownItem.js b/src/DropdownItem.js
@@ -76,6 +76,8 @@ class DropdownItem extends React.Component {
         Tag = 'h6';
       } else if (divider) {
         Tag = 'div';
+      } else if (props.href) {
+        Tag = 'a';
       }
     }
 
diff --git a/src/__tests__/DropdownItem.spec.js b/src/__tests__/DropdownItem.spec.js
@@ -46,6 +46,16 @@ describe('DropdownItem', () => {
     });
   });
 
+  describe('link (with href)', () => {
+    it('should render an anchor tag', () => {
+      const wrapper = mount(<DropdownItem href="#">GO!</DropdownItem>);
+
+      expect(wrapper.find('a').length).toBe(1);
+      expect(wrapper.find('a').hasClass('dropdown-item')).toBe(true);
+      expect(wrapper.text()).toBe('GO!');
+    });
+  });
+
   describe('onClick', () => {
     it('should not be called when disabled', () => {
       const e = { preventDefault: jasmine.createSpy('preventDefault') };

Original file line number	Diff line number	Diff line change
`@@ -76,6 +76,8 @@ class DropdownItem extends React.Component {`
`76`	`76`	`Tag = 'h6';`
`77`	`77`	`} else if (divider) {`
`78`	`78`	`Tag = 'div';`
	`79`	`+ } else if (props.href) {`
	`80`	`+ Tag = 'a';`
`79`	`81`	`}`
`80`	`82`	`}`
`81`	`83`