Apply lint rules

Author: gilbarbara

packages/tree-changes-hook/src/index.ts

@@ -1,6 +1,5 @@
 import { useEffect, useRef } from 'react';
 import * as equal from 'fast-deep-equal';
-
 import treeChanges, { Data, KeyType, TreeChanges } from 'tree-changes';
 
 export default function useTreeChanges<T extends Data>(value: T) {

packages/tree-changes-hook/test/__snapshots__/index.spec.tsx.snap

@@ -8,7 +8,9 @@ exports[`useTreeChanges with state 1`] = `
     <h1>
       tree-changes
     </h1>
-    <button>
+    <button
+      type="button"
+    >
       Clicked 
       0
        times
@@ -28,7 +30,9 @@ exports[`useTreeChanges with state 2`] = `
     <h1>
       tree-changes
     </h1>
-    <button>
+    <button
+      type="button"
+    >
       Clicked 
       0
        times

packages/tree-changes-hook/test/index.spec.tsx

@@ -1,6 +1,6 @@
 import * as React from 'react';
-import { fireEvent, render, screen } from '@testing-library/react';
 import { act } from 'react-dom/test-utils';
+import { fireEvent, render, screen } from '@testing-library/react';
 
 import useTreeChanges from '../src';
 
@@ -24,7 +24,7 @@ function WithState() {
     isReady: false,
     milestones: [],
   });
-  const { added, changed, filled, removed, emptied } = useTreeChanges(state);
+  const { added, changed, emptied, filled, removed } = useTreeChanges(state);
 
   React.useEffect(() => {
     setTimeout(() => {
@@ -77,7 +77,7 @@ function WithState() {
     if (emptied('milestones')) {
       mockState('emptied:milestones');
     }
-  }, [changed, state]);
+  }, [added, changed, emptied, filled, removed, state]);
 
   const handleClickCount = () => {
     setState(s => ({ ...s, count: s.count + 1 }));
@@ -87,7 +87,9 @@ function WithState() {
     <div data-testid="app">
       <header>
         <h1>tree-changes</h1>
-        <button onClick={handleClickCount}>Clicked {state.count} times</button>
+        <button onClick={handleClickCount} type="button">
+          Clicked {state.count} times
+        </button>
         {state.isReady ? <p>side-effect is complete</p> : <p>loading...</p>}
       </header>
     </div>

packages/tree-changes/src/helpers.ts

@@ -1,5 +1,6 @@
 import * as equal from 'fast-deep-equal';
 import is from 'is-lite';
+
 import { CompareValuesOptions, Data, Key, Options, ValidTypes, Value } from './types';
 
 export function canHaveLength(...args: any): boolean {

packages/tree-changes/src/index.ts

@@ -1,7 +1,7 @@
 import * as equal from 'fast-deep-equal';
 import is from 'is-lite';
-import { compareNumbers, compareValues, getIterables, includesOrEqualsTo, nested } from './helpers';
 
+import { compareNumbers, compareValues, getIterables, includesOrEqualsTo, nested } from './helpers';
 import { Data, KeyType, TreeChanges, Value } from './types';
 
 export default function treeChanges<P extends Data, D extends Data, K = KeyType<P, D>>(

packages/tree-changes/test/helpers.spec.ts

@@ -4,10 +4,10 @@ import {
   compareNumbers,
   compareValues,
   getIterables,
-  includesOrEqualsTo,
   hasEntry,
   hasExtraKeys,
   hasValue,
+  includesOrEqualsTo,
   isSameType,
   nested,
 } from '../src/helpers';
@@ -249,7 +249,7 @@ describe('hasValue', () => {
 });
 
 describe('includesOrEqualsTo', () => {
-  it('should ', () => {
+  it('should match', () => {
     expect(
       includesOrEqualsTo(
         {

packages/tree-changes/test/index.spec.ts

@@ -61,20 +61,16 @@ describe('tree-changes', () => {
   describe('basic', () => {
     it('should throw without all parameters', () => {
       // @ts-ignore
-      expect(() => treeChanges()).toThrowError('Missing required parameters');
+      expect(() => treeChanges()).toThrow('Missing required parameters');
       // @ts-ignore
-      expect(() => treeChanges(A)).toThrowError('Missing required parameters');
+      expect(() => treeChanges(A)).toThrow('Missing required parameters');
     });
 
     it('should throw with invalid parameters', () => {
       // @ts-ignore
-      expect(() => treeChanges(A.missing, B.missing)).toThrowError(
-        'Expected plain objects or array',
-      );
+      expect(() => treeChanges(A.missing, B.missing)).toThrow('Expected plain objects or array');
       // @ts-ignore
-      expect(() => treeChanges(A.hasData, B.hasData)).toThrowError(
-        'Expected plain objects or array',
-      );
+      expect(() => treeChanges(A.hasData, B.hasData)).toThrow('Expected plain objects or array');
     });
   });
 
@@ -366,6 +362,7 @@ describe('tree-changes', () => {
 
     it('with objects', () => {
       const { emptied } = treeChanges(A, B);
+
       expect(emptied('messages')).toBe(true);
 
       expect(emptied('notifications')).toBe(false);