Merge pull request #107 from trovit/remove-jest-profiler

Remove dependency with react-jest-profiler

packages/reffects-store/package.json

@@ -36,7 +36,6 @@
     "eslint-plugin-jest": "^22.4.1",
     "eslint-plugin-prettier": "^3.0.1",
     "jest": "^26.6.3",
-    "jest-react-profiler": "^0.1.3",
     "jest-when": "^3.4.1",
     "prettier": "^1.16.4",
     "reffects": "^1.6.1-alpha.0",

packages/reffects-store/src/subscription/index.test.js

@@ -1,7 +1,7 @@
-import { withProfiler } from 'jest-react-profiler';
 import React from 'react';
 import { act } from 'react-dom/test-utils';
 import { render } from '@testing-library/react';
+import withProfiler from './rendererProfiler';
 import subscribe from '.';
 import * as storeModule from '../store';
 import '@testing-library/jest-dom';

packages/reffects-store/src/subscription/rendererProfiler.js

@@ -0,0 +1,23 @@
+import { createElement, Profiler } from 'react';
+
+export default function withProfiler(Component) {
+  const onRender = () => {
+    SnapshotProfiler.__numCommits++;
+  };
+  const SnapshotProfiler = props =>
+    createElement(
+      Profiler,
+      { id: 'withProfiler', onRender },
+      createElement(Component, props)
+    );
+  SnapshotProfiler.__numCommits = 0;
+  return SnapshotProfiler;
+}
+
+function toHaveCommittedTimes(SnapshotProfiler, expectedNumCommits) {
+  expect(SnapshotProfiler.__numCommits).toBe(expectedNumCommits);
+  SnapshotProfiler.__numCommits = 0;
+  return { pass: true };
+}
+
+expect.extend({ toHaveCommittedTimes });

packages/reffects-store/src/subscription/useSelector.test.js

@@ -1,7 +1,7 @@
-import { withProfiler } from 'jest-react-profiler';
 import React from 'react';
 import { render } from '@testing-library/react';
 import { act } from 'react-dom/test-utils';
+import withProfiler from './rendererProfiler';
 import * as storeModule from '../store';
 import useSelector from './useSelector';
 import '@testing-library/jest-dom';
@@ -73,12 +73,12 @@ describe('useSelector hook', () => {
   });
 
   it.each([
-    ['primitive', 1, '1'],
-    ['array', [1, 2], '[1,2]'],
-    ['object', { b: 2 }, '{"b":2}'],
+    ['primitive', 1, 1, '1'],
+    ['array', [1, 2], [1, 2], '[1,2]'],
+    ['object', { b: 2 }, { b: 2 }, '{"b":2}'],
   ])(
     "shouldn't update the component using it when the state is the same using %s",
-    (_, value, expected) => {
+    (_, value, newValue, expected) => {
       const initialProps = { a: value };
       const store = storeModule;
       store.initialize(initialProps);
@@ -91,7 +91,7 @@ describe('useSelector hook', () => {
       expect(getByText(expected)).toBeInTheDocument();
 
       act(() => {
-        store.setState({ path: ['a'], newValue: value });
+        store.setState({ path: ['a'], newValue });
         store.setState({ path: ['koko'], newValue: 'loko' });
       });