chore(ARCH-345): update all redux related packages

.changeset/modern-cars-hope.md

@@ -0,0 +1,14 @@
+---
+'@talend/react-cmf': major
+'@talend/react-cmf-cqrs': major
+'@talend/react-cmf-router': major
+'@talend/react-forms': major
+'@talend/react-containers': major
+'@talend/react-datagrid': major
+'@talend/react-sagas': major
+'@talend/react-stepper': major
+'@talend/react-storybook-cmf': major
+'react-flow-designer': major
+---
+
+Redux major upgrade with saga

.github/workflows/pr-lint.yml

@@ -11,6 +11,9 @@ jobs:
     steps:
       - name: Checkout sources
         uses: actions/checkout@v2
+        with:
+          persist-credentials: false # otherwise, the token used is the GITHUB_TOKEN, instead of your personal token
+          fetch-depth: 0 # otherwise, you will failed to push refs to dest repo
 
       - name: Use Node.js 14
         uses: actions/setup-node@v2
@@ -60,3 +63,4 @@ jobs:
         with:
           github_token: ${{ secrets.GITHUB_TOKEN }}
           message: "chore: apply prettier on modified files"
+          branch: ${{ github.head_ref }}

.github/workflows/yarn-deduplicate.yml

@@ -3,7 +3,7 @@ name: yarn-deduplicate
 on:
   pull_request:
     paths:
-      - 'yarn.lock'
+      - "yarn.lock"
 
 jobs:
   dedupe:
@@ -22,9 +22,9 @@ jobs:
         uses: actions/setup-node@v2
         with:
           node-version: 14
-          registry-url: 'https://registry.npmjs.org/'
-          scope: '@talend'
-          cache: 'yarn'
+          registry-url: "https://registry.npmjs.org/"
+          scope: "@talend"
+          cache: "yarn"
 
       - name: yarn-deduplicate
         id: deduplicate
@@ -38,5 +38,5 @@ jobs:
         uses: actions-js/push@master
         with:
           github_token: ${{ secrets.GITHUB_TOKEN }}
-          message: 'chore: yarn-deduplicate'
+          message: "chore: yarn-deduplicate"
           branch: ${{ github.head_ref }}

packages/cmf-cqrs/package.json

@@ -33,7 +33,7 @@
   "dependencies": {
     "@talend/react-cmf": "^6.38.5",
     "immutable": "^3.8.2",
-    "redux-saga": "^0.15.6"
+    "redux-saga": "^1.1.3"
   },
   "devDependencies": {
     "@talend/scripts-core": "^11.3.0",

packages/cmf-cqrs/src/components/ACKDispatcher/ACKDispatcher.test.js

@@ -42,7 +42,7 @@ describe('Container ACKDispatcher', () => {
 		}
 
 		registry['actionCreator:myActionCreator'] = myActionCreator;
-		mount(<Container acks={Map()} />, { context: { registry } });
+		mount(<Container acks={Map()} />, mock.Provider.getEnzymeOption({ registry }));
 		expect(mockProcessACK).toHaveBeenCalled();
 	});
 	it('should processACK call dispatch', () => {

packages/cmf-router/package.json

@@ -22,16 +22,17 @@
     "lodash": "^4.17.21",
     "path-to-regexp": "^2.4.0",
     "prop-types": "^15.8.1",
-    "react-redux": "^5.1.2",
+    "react-redux": "^7.2.2",
     "react-router": "^3.2.6",
     "react-router-redux": "^4.0.8",
-    "redux-saga": "^0.15.6"
+    "redux-saga": "^1.1.3"
   },
   "peerDependencies": {
     "react": "^16.8.6",
     "react-dom": "^16.8.6"
   },
   "devDependencies": {
+    "@redux-saga/testing-utils": "^1.1.3",
     "@talend/scripts-core": "^11.3.0",
     "@talend/scripts-preset-react-lib": "^9.9.3",
     "enzyme": "^3.11.0",

packages/cmf-router/src/index.test.js

@@ -33,7 +33,7 @@ describe('getModule', () => {
 		const generator = mod.cmfModule.saga();
 		generator.next();
 		const result = generator.next();
-		expect(result.value.FORK.args[1]).toEqual({
+		expect(result.value.payload.args[1]).toEqual({
 			'/foo': config.sagaRouterConfig['/foo'],
 			'/foo/bar': configBis.sagaRouterConfig['/foo/bar'],
 		});

packages/cmf-router/src/sagaRouter.test.js

@@ -1,5 +1,5 @@
 import { spawn, take, cancel } from 'redux-saga/effects';
-import { createMockTask } from 'redux-saga/utils';
+import { createMockTask } from '@redux-saga/testing-utils';
 import sagaRouter from './sagaRouter';
 
 describe('sagaRouter import', () => {

packages/cmf/__tests__/Dispatcher.test.js

@@ -1,16 +1,21 @@
 import React from 'react';
-import PropTypes from 'prop-types';
 import { shallow, mount } from 'enzyme';
 import { Dispatcher } from '../src/Dispatcher';
 import CONST from '../src/constant';
+import { RegistryProvider } from '../src/RegistryProvider';
 
-const mockContext = {
-	registry: {
-		[`${CONST.REGISTRY_ACTION_CREATOR_PREFIX}:actionCreator:id`]: jest.fn(),
-		[`${CONST.REGISTRY_ACTION_CREATOR_PREFIX}:another:actionCreator:id`]: jest.fn(),
-	},
+const REGISTRY = {
+	[`${CONST.REGISTRY_ACTION_CREATOR_PREFIX}:actionCreator:id`]: jest.fn(),
+	[`${CONST.REGISTRY_ACTION_CREATOR_PREFIX}:another:actionCreator:id`]: jest.fn(),
 };
 
+function getEnzymeOption() {
+	return {
+		wrappingComponent: RegistryProvider,
+		wrappingComponentProps: { value: REGISTRY },
+	};
+}
+
 jest.mock('../src/action', () => ({
 	getOnProps() {
 		return ['onClick', 'onDoubleClick'];
@@ -49,9 +54,7 @@ describe('Testing <Dispatcher />', () => {
 			>
 				<button />
 			</Dispatcher>,
-			{
-				context: mockContext,
-			},
+			getEnzymeOption(),
 		);
 		expect(
 			JSON.stringify(wrapper.find('button').props(), replacer).replace(/(\\t|\\n)/g, ''),
@@ -70,9 +73,7 @@ describe('Testing <Dispatcher />', () => {
 				>
 					<button />
 				</Dispatcher>,
-				{
-					context: mockContext,
-				},
+				getEnzymeOption(),
 			);
 		}).toThrow('action not found id: unknnown:actionCreator:id');
 	});
@@ -82,9 +83,7 @@ describe('Testing <Dispatcher />', () => {
 			<Dispatcher onClick="noOp" onDoubleClick="noOp">
 				<button />
 			</Dispatcher>,
-			{
-				context: mockContext,
-			},
+			getEnzymeOption(),
 		);
 		const buttonWrapper = wrapper.find('button').at(0);
 		const instance = wrapper.instance();
@@ -103,9 +102,7 @@ describe('Testing <Dispatcher />', () => {
 					<Dispatcher onClick="error:actionCreator:id" onDoubleClick="another:actionCreator:id">
 						<button />
 					</Dispatcher>,
-					{
-						context: mockContext,
-					},
+					getEnzymeOption(),
 				);
 			}).toThrowError('action not found id: error:actionCreator:id');
 		},
@@ -119,9 +116,7 @@ describe('Testing <Dispatcher />', () => {
 				<Dispatcher onClick="existingActionCreator:id" onDoubleClick="existingActionCreator:id">
 					<button />
 				</Dispatcher>,
-				{
-					context: mockContext,
-				},
+				getEnzymeOption(),
 			);
 			expect(() => {
 				wrapper.setProps({ onClick: 'error:another:actionCreator:id' });
@@ -142,12 +137,7 @@ describe('Testing <Dispatcher />', () => {
 					<a />
 				</Dispatcher>
 			</div>,
-			{
-				context: mockContext,
-				childContextTypes: {
-					registry: PropTypes.object.isRequired,
-				},
-			},
+			getEnzymeOption(),
 		);
 		wrapper.find('a').simulate('click');
 		expect(onClick).toHaveBeenCalled();
@@ -167,12 +157,7 @@ describe('Testing <Dispatcher />', () => {
 					<a />
 				</Dispatcher>
 			</div>,
-			{
-				context: mockContext,
-				childContextTypes: {
-					registry: PropTypes.object.isRequired,
-				},
-			},
+			getEnzymeOption(),
 		);
 		wrapper.find('a').simulate('click');
 		expect(onClick).not.toHaveBeenCalled();
@@ -188,9 +173,7 @@ describe('Testing <Dispatcher />', () => {
 			<Dispatcher dispatchActionCreator={dispatchActionCreator} preventDefault onClick="noOp">
 				<a />
 			</Dispatcher>,
-			{
-				context: mockContext,
-			},
+			getEnzymeOption(),
 		);
 		wrapper.find('a').simulate('click', event);
 		expect(event.preventDefault).toHaveBeenCalled();
@@ -210,9 +193,10 @@ describe('Testing <Dispatcher />', () => {
 			extra: 'foo',
 			children: <a />,
 		};
-		const wrapper = shallow(<Dispatcher {...props} />, {
-			context: mockContext,
-		});
+		const wrapper = shallow(
+			<Dispatcher {...props} />,
+			getEnzymeOption(),
+		);
 		wrapper.find('a').simulate('click', event);
 		expect(dispatchActionCreator).toHaveBeenCalledWith('noOp', event, props);
 	});

packages/cmf/__tests__/Inject.test.js

@@ -1,40 +1,48 @@
 import React from 'react';
-import { shallow } from 'enzyme';
+import { mount } from 'enzyme';
 
 import Inject from '../src/Inject.component';
+import { mock } from '../src';
 
 describe('Inject', () => {
 	it('should render', () => {
 		// given
-		const MyComponent = jest.fn();
+		const MyComponent = jest.fn(() => <span>Hello</span>);
 		MyComponent.displayName = 'MyComponent';
 		const context = {
 			registry: {
 				'_.route.component:MyComponent': MyComponent,
 			},
 		};
+		const Provider = mock.Provider;
 
 		// when
-		const wrapper = shallow(<Inject component="MyComponent" extra-props />, { context });
+		const wrapper = mount(
+			<Inject component="MyComponent" extra-props />,
+			Provider.getEnzymeOption(context),
+		);
 
 		// then
-		expect(wrapper.equals(<MyComponent extra-props />)).toBe(true);
+		expect(wrapper.find(MyComponent).equals(<MyComponent extra-props />)).toBe(true);
 	});
 
 	it('should render error if component not found', () => {
 		// given
 		const MyComponent = jest.fn();
 		MyComponent.displayName = 'MyComponent';
-		const context = { registry: {} };
+		// const context = { registry: {} };
+		const Provider = mock.Provider;
 
 		// when
-		const wrapper = shallow(<Inject component="MyComponent" />, { context });
+		const wrapper = mount(<Inject component="MyComponent" />, Provider.getEnzymeOption());
 
 		// then
-		expect(wrapper.equals(
-			<Inject.NotFoundComponent
-				error="component not found in the registry: MyComponent"
-			/>
-		)).toBe(true);
+		expect(
+			wrapper
+				.find(Inject.NotFoundComponent)
+				.equals(
+					<Inject.NotFoundComponent error="component not found in the registry: MyComponent" />,
+				),
+		).toBe(true);
 	});
 });

packages/cmf/__tests__/action.test.js

@@ -1,5 +1,5 @@
 import actionAPI from '../src/action';
-import mock from '../src/mock';
+import {mock} from '../src';
 
 describe('CMF action', () => {
 	let state;
@@ -8,11 +8,11 @@ describe('CMF action', () => {
 	let settings;
 
 	beforeEach(() => {
-		settings = mock.settings();
-		state = mock.state();
+		settings = mock.store.settings();
+		state = mock.store.state();
 		state.cmf.settings = settings;
-		context = mock.context(state);
-		emptyContext = mock.emptyContext();
+		context = mock.store.context(state);
+		emptyContext = mock.store.emptyContext();
 	});
 	it('getActionsById should return action from settings', () => {
 		const actions = actionAPI.getActionsById(context);

packages/cmf/__tests__/actionCreator.test.js

@@ -1,12 +1,11 @@
-import mock from '../src/mock';
+import {mock} from '../src';
 import actionCreatorAPI from '../src/actionCreator';
-import { create } from 'domain';
 
 describe('CMF action', () => {
 	let context;
 
 	beforeEach(() => {
-		context = mock.context();
+		context = mock.store.context();
 	});
 
 	it('get should return a function', () => {

packages/cmf/__tests__/bootstrap.test.js

@@ -34,6 +34,7 @@ jest.mock('../src/onError', () => ({
 }));
 jest.mock('../src/registry', () => ({
 	registerMany: jest.fn(),
+	getRegistry: jest.fn(),
 }));
 jest.mock('../src/actionCreator', () => ({
 	registerMany: jest.fn(),