enable spreading props to buttongroup

SAP · Jan 16, 2019 · 7393b97 · 7393b97
1 parent 00d5fd7
commit 7393b97
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/src/Button/Button.js b/src/Button/Button.js
@@ -34,11 +34,11 @@ Button.propTypes = {
     onClick: PropTypes.func
 };
 
-export const ButtonGroup = props => {
-    const { children } = props;
+export const ButtonGroup = ({children, ...props}) => {
     return (
         <div aria-label='Group label' className='fd-button-group'
-            role='group'>
+            role='group'
+            {...props}>
             {children}
         </div>
     );

diff --git a/src/Button/Button.test.js b/src/Button/Button.test.js
@@ -93,8 +93,12 @@ describe('<Button />', () => {
             ).toBe('Sample');
         });
 
-        xtest('should allow props to be spread to the ButtonGroup component', () => {
-            // TODO: placeholder for this test description once that functionality is built
+        test('should allow props to be spread to the ButtonGroup component', () => {
+            const element = mount(<ButtonGroup data-sample='Sample' />);
+
+            expect(
+                element.getDOMNode().attributes['data-sample'].value
+            ).toBe('Sample');
         });
     });
 });