fix lint and test errors

Author: solkimicreb

__mocks__/react-dom.js

@@ -0,0 +1,3 @@
+// this is here to avoid duplicate react entries in the examples
+// (one from the example's node_modules and one from the root's node_modules)
+module.exports = require('react-dom')

__mocks__/react-native.js

@@ -0,0 +1,3 @@
+// this is here to avoid duplicate react entries in the examples
+// (one from the example's node_modules and one from the root's node_modules)
+module.exports = require('react-native')

__tests__/Clock.test.js

@@ -1,4 +1,5 @@
 import React, { StrictMode } from 'react'
+import { act } from 'react-dom/test-utils'
 import { render, cleanup, flushEffects } from 'react-testing-library'
 import sinon from 'sinon'
 import App from '../examples/clock/src/App'
@@ -24,10 +25,14 @@ describe('Clock App', () => {
   test('should update to display the current time every second', () => {
     expect(container).toHaveTextContent('12:00:00 AM')
 
-    clock.tick(2000)
+    act(() => {
+      clock.tick(2000)
+    })
     expect(container).toHaveTextContent('12:00:02 AM')
 
-    clock.tick(8500)
+    act(() => {
+      clock.tick(8500)
+    })
     expect(container).toHaveTextContent('12:00:10 AM')
   })
 

__tests__/router.test.js

@@ -1,4 +1,5 @@
 import React, { Component } from 'react'
+import { act } from 'react-dom/test-utils'
 import { render, cleanup, fireEvent } from 'react-testing-library'
 import { view, store } from 'react-easy-state'
 import {
@@ -25,7 +26,9 @@ describe('withRouter interaction', () => {
         </Router>
       )
       expect(container).toHaveTextContent('0')
-      counter.num++
+      act(() => {
+        counter.num++
+      })
       expect(container).toHaveTextContent('1')
     })
 
@@ -70,7 +73,9 @@ describe('withRouter interaction', () => {
         </Router>
       )
       expect(container).toHaveTextContent('0')
-      counter.num++
+      act(() => {
+        counter.num++
+      })
       expect(container).toHaveTextContent('1')
     })
 

__tests__/styled.test.js

@@ -1,4 +1,5 @@
 import React, { Component } from 'react'
+import { act } from 'react-dom/test-utils'
 import { render, cleanup } from 'react-testing-library'
 import { view, store } from 'react-easy-state'
 import { withTheme, ThemeProvider } from 'styled-components'
@@ -24,7 +25,9 @@ describe('withRouter interaction', () => {
         </Theme>
       )
       expect(container).toHaveTextContent('0')
-      counter.num++
+      act(() => {
+        counter.num++
+      })
       expect(container).toHaveTextContent('1')
     })
 
@@ -61,7 +64,9 @@ describe('withRouter interaction', () => {
         </Theme>
       )
       expect(container).toHaveTextContent('0')
-      counter.num++
+      act(() => {
+        counter.num++
+      })
       expect(container).toHaveTextContent('1')
     })
 

src/view.js

@@ -5,7 +5,7 @@ import { hasHooks } from './utils'
 export let isInsideFunctionComponent = false
 const COMPONENT = Symbol('owner component')
 
-export default function view(Comp) {
+export default function view (Comp) {
   const isStatelessComp = !(Comp.prototype && Comp.prototype.isReactComponent)
 
   let ReactiveComp
@@ -47,7 +47,7 @@ export default function view(Comp) {
     // a HOC which overwrites render, shouldComponentUpdate and componentWillUnmount
     // it decides when to run the new reactive methods and when to proxy to the original methods
     class ReactiveClassComp extends BaseComp {
-      constructor(props, context) {
+      constructor (props, context) {
         super(props, context)
 
         this.state = this.state || {}
@@ -60,12 +60,14 @@ export default function view(Comp) {
         })
       }
 
-      render() {
-        return isStatelessComp ? Comp(this.props, this.context) : super.render()
+      render () {
+        return isStatelessComp
+          ? Comp(this.props, this.context)
+          : super.render()
       }
 
       // react should trigger updates on prop changes, while easyState handles store changes
-      shouldComponentUpdate(nextProps, nextState) {
+      shouldComponentUpdate (nextProps, nextState) {
         const { props, state } = this
 
         // respect the case when the user defines a shouldComponentUpdate
@@ -88,7 +90,7 @@ export default function view(Comp) {
       }
 
       // add a custom deriveStoresFromProps lifecyle method
-      static getDerivedStateFromProps(props, state) {
+      static getDerivedStateFromProps (props, state) {
         if (super.deriveStoresFromProps) {
           // inject all local stores and let the user mutate them directly
           const stores = mapStateToStores(state)
@@ -101,7 +103,7 @@ export default function view(Comp) {
         return null
       }
 
-      componentWillUnmount() {
+      componentWillUnmount () {
         // call user defined componentWillUnmount
         if (super.componentWillUnmount) {
           super.componentWillUnmount()
@@ -126,7 +128,7 @@ export default function view(Comp) {
   return ReactiveComp
 }
 
-function mapStateToStores(state) {
+function mapStateToStores (state) {
   // find store properties and map them to their none observable raw value
   // to do not trigger none static this.setState calls
   // from the static getDerivedStateFromProps lifecycle method