Simpler boolean being returned

nearform · Feb 12, 2019 · 50bcbfe · 50bcbfe
1 parent 4894613
commit 50bcbfe
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 10 deletions.
diff --git a/src/hooks/online.js b/src/hooks/online.js
@@ -23,7 +23,5 @@ export function useOnline() {
     }
   }, [])
 
-  return {
-    online
-  }
+  return online
 }
diff --git a/storybook/stories/online/README.md b/storybook/stories/online/README.md
@@ -11,10 +11,10 @@ import { useOnline } from '@nearform/react-browser-hooks'
 Example of usage:
 
 ```javascript
-const { online } = useOnline()
+const online = useOnline()
 <p>Status: {online ? 'online' : 'offline'}</p>
 ```
 
-Returns an object containing:
+Returns:
 
-- status (boolean): whether online or not
+- online (boolean): whether online or not
diff --git a/storybook/stories/online/online.stories.js b/storybook/stories/online/online.stories.js
@@ -5,7 +5,7 @@ import { useOnline } from '../../../src'
 import readme from './README.md'
 
 function Online() {
-  const { online } = useOnline()
+  const online = useOnline()
   return (
     <div>
       <h2>Online Demo</h2>

diff --git a/test/unit/hooks/online.unit.test.js b/test/unit/hooks/online.unit.test.js
@@ -17,7 +17,7 @@ describe('useOnline', () => {
 
     onLineGetter.mockReturnValue(true)
 
-    testHook(() => ({ online } = useOnline()))
+    testHook(() => (online = useOnline()))
 
     expect(online).toBe(true)
   })
@@ -28,7 +28,7 @@ describe('useOnline', () => {
     onLineGetter.mockReturnValue(false)
 
     act(() => {
-      testHook(() => ({ online } = useOnline()))
+      testHook(() => (online = useOnline()))
     })
 
     onLineGetter.mockReturnValue(true)
@@ -53,7 +53,7 @@ it('updates state on "offline"', () => {
   onLineGetter.mockReturnValue(true)
 
   act(() => {
-    testHook(() => ({ online } = useOnline()))
+    testHook(() => (online = useOnline()))
   })
 
   onLineGetter.mockReturnValue(false)