feat(toolbar): forwardRef in toolbar, add tests

Author: arturbien

src/components/Toolbar/Toolbar.js

@@ -6,30 +6,25 @@ const StyledToolbar = styled.div`
   position: relative;
   display: flex;
   align-items: center;
-  padding: ${props => (props.noPadding ? '0px' : '4px')};
+  padding: ${props => (props.noPadding ? '0' : '4px')};
 `;
 
-const Toolbar = ({ children, className, style, noPadding, ...otherProps }) => (
-  <StyledToolbar
-    noPadding={noPadding}
-    className={className}
-    style={style}
-    {...otherProps}
-  >
-    {children}
-  </StyledToolbar>
-);
+const Toolbar = React.forwardRef(function Toolbar(props, ref) {
+  const { children, noPadding, ...otherProps } = props;
+  return (
+    <StyledToolbar noPadding={noPadding} ref={ref} {...otherProps}>
+      {children}
+    </StyledToolbar>
+  );
+});
 
 Toolbar.defaultProps = {
-  noPadding: false,
-  style: {},
-  className: ''
+  children: null,
+  noPadding: false
 };
 
 Toolbar.propTypes = {
-  style: propTypes.shape([propTypes.string, propTypes.number]),
-  className: propTypes.string,
-  children: propTypes.node.isRequired,
+  children: propTypes.node,
   noPadding: propTypes.bool
 };
 

src/components/Toolbar/Toolbar.spec.js

@@ -0,0 +1,29 @@
+import React from 'react';
+import { render } from '@testing-library/react';
+
+import Toolbar from './Toolbar';
+
+describe('<Toolbar />', () => {
+  it('should render', () => {
+    const { container } = render(<Toolbar />);
+    const toolbar = container.firstChild;
+
+    expect(toolbar).toBeInTheDocument();
+  });
+
+  it('should render children', () => {
+    const { container } = render(<Toolbar>A nice app bar</Toolbar>);
+    const toolbar = container.firstChild;
+
+    expect(toolbar).toHaveTextContent('A nice app bar');
+  });
+
+  describe('prop: noPadding', () => {
+    it('should apply 0 padding', () => {
+      const { container } = render(<Toolbar noPadding />);
+      const toolbar = container.firstChild;
+
+      expect(toolbar).toHaveStyleRule('padding', '0');
+    });
+  });
+});