Merge pull request #171 from helpscout/Card/fix-autoWordWrap-prop

Card: Fix autoWordWrap prop use for non-Links
helpscout · Jan 3, 2018 · 83df19b · 83df19b
2 parents 74051d6 + fa85cb4
commit 83df19b
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 0 deletions.
diff --git a/src/components/Card/index.js b/src/components/Card/index.js
@@ -36,6 +36,7 @@ const defaultProps = {
 
 const Card = props => {
   const {
+    autoWordWrap,
     borderless,
     className,
     children,
@@ -68,6 +69,7 @@ const Card = props => {
 
   const element = href || to ? (
     <Link
+      autoWordWrap={autoWordWrap}
       block
       className={componentClassName}
       onClick={onClick}

diff --git a/src/components/Card/tests/Card.test.js b/src/components/Card/tests/Card.test.js
@@ -164,6 +164,20 @@ describe('Link', () => {
 
     expect(spy).toHaveBeenCalled()
   })
+
+  test('Does not pass autoWordWrap prop to div', () => {
+    const wrapper = shallow(<Card autoWordWrap />)
+    const o = wrapper.find('.c-Card')
+
+    expect(o.props().autoWordWrap).toBeFalsy()
+  })
+
+  test('Can pass autoWordWrap to Link', () => {
+    const wrapper = shallow(<Card href={link} autoWordWrap />)
+    const o = wrapper.find('.c-Card')
+
+    expect(o.props().autoWordWrap).toBeTruthy()
+  })
 })
 
 describe('Click', () => {