Allow className to be passed to Spinner (#523)

* add classname to destructured props and test to add custom classname

* remove linting change

* find spinner by id in test

* incrementing package.json version for backwards compatible patch

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@launchpadlab/lp-components",
-  "version": "5.4.6",
+  "version": "5.4.7",
   "engines": {
     "node": "^8.0.0 || ^10.13.0 || ^12.0.0"
   },

src/indicators/spinner.js

@@ -30,8 +30,8 @@ const propTypes = {}
 
 const defaultProps = {}
 
-function Spinner (props) {
-  return <div id="spinner" { ...filterInvalidDOMProps(props) } />
+function Spinner({ className, ...rest }) {
+  return <div id="spinner" className={className} {...filterInvalidDOMProps(rest)} />
 }
 
 Spinner.propTypes = propTypes

test/indicators/spinner.test.js

@@ -5,7 +5,7 @@ import { Spinner } from '../../src/'
 
 test('Spinner creates div with id "spinner"', () => {
   const wrapper = mount(
-    <Spinner/>
+    <Spinner />
   )
   expect(wrapper.find('div#spinner').exists()).toBe(true)
 })
@@ -22,4 +22,11 @@ test('Spinner filters invalid props', () => {
     <Spinner name="Bob" someInvalidProp="I am not a valid DOM prop" />
   )
   expect(wrapper.find('div#spinner').props().someInvalidProp).not.toBeDefined()
+})
+
+test('Spinner can have custom className', () => {
+  const wrapper = mount(
+    <Spinner className="custom" />
+  )
+  expect(wrapper.find('div#spinner.custom').exists()).toBe(true)
 })