fix: remove observer check from hook

Author: ardaogulcan

src/useObserver.js

@@ -6,15 +6,6 @@ import createObserverPublisher from './helpers/createObserverPublisher'
 let publishers
 
 export default function useObserver(Observer, { observerOptions, subscribeOptions } = {}) {
-  if (process.env.NODE_ENV !== 'production') {
-    if (!Observer) {
-      // eslint-disable-next-line no-console
-      console.error(
-        'useObserver requires a valid WebAPI Observer as a first parameter',
-      )
-    }
-  }
-
   const [entry, setEntry] = useState({})
   const [element, setElement] = useState()
 

src/useObserver.test.js

@@ -23,34 +23,6 @@ beforeEach(() => {
   })
 })
 
-it('should throw an error if no Observer given', () => {
-  jest.spyOn(console, 'error').mockImplementation(() => { })
-
-  renderHook(() => useObserver())
-
-  // eslint-disable-next-line no-console
-  expect(console.error).toBeCalledWith('useObserver requires a valid WebAPI Observer as a first parameter')
-
-  // eslint-disable-next-line no-console
-  console.error.mockRestore()
-})
-
-it('should not throw an error on production mode', () => {
-  jest.spyOn(console, 'error').mockImplementation(() => {})
-  const env = process.env.NODE_ENV
-  process.env.NODE_ENV = 'production'
-
-  const { result } = renderHook(() => useObserver())
-  expect(result.error).toBeUndefined()
-
-  // eslint-disable-next-line no-console
-  expect(console.error).not.toBeCalled()
-  // eslint-disable-next-line no-console
-  console.error.mockRestore()
-
-  process.env.NODE_ENV = env
-})
-
 it('should create only one observer instance for same Observer and Options for all uses and rerenders', () => {
   const loopArr = [...Array(10)]