Refactor reducers to use RTK

.eslintrc

@@ -14,6 +14,7 @@
   "settings": { "react": { "version": "detect" } },
   "rules": {
     "eqeqeq": "error",
+    "no-param-reassign": ["error", { "props": true }],
     "@typescript-eslint/consistent-type-assertions": "error",
     "@typescript-eslint/no-unused-vars": [
       "error",

__tests__/lib/weechat/action_transformer.ts

@@ -2,6 +2,10 @@ import { AnyAction, configureStore } from '@reduxjs/toolkit';
 import { StoreState, reducer } from '../../../src/store';
 import { transformToReduxAction } from '../../../src/lib/weechat/action_transformer';
 import { ThunkAction } from 'redux-thunk';
+import {
+  fetchBuffersAction,
+  fetchBuffersRemovedAction
+} from '../../../src/store/actions';
 
 describe('transformToReduxAction', () => {
   describe('on buffers', () => {
@@ -68,12 +72,9 @@ describe('transformToReduxAction', () => {
       );
       expect(dispatch).toHaveBeenNthCalledWith(
         1,
-        expect.objectContaining({ type: 'FETCH_BUFFERS_REMOVED' })
-      );
-      expect(dispatch).toHaveBeenNthCalledWith(
-        2,
-        expect.objectContaining({ type: 'FETCH_BUFFERS' })
+        fetchBuffersRemovedAction([])
       );
+      expect(dispatch).toHaveBeenNthCalledWith(2, fetchBuffersAction({}));
     });
 
     it('preserves currentBufferId if the buffer is still open', () => {
@@ -212,4 +213,51 @@ describe('transformToReduxAction', () => {
       });
     });
   });
+
+  describe('on _nicklist_diff', () => {
+    it('synchronizes the nicklist state with the provided diff', () => {
+      const preloadedState = {
+        nicklists: {
+          '84ed37600': [
+            {
+              name: 'oldnick',
+              pointers: ['84ed37600', '85139f000']
+            } as WeechatNicklist
+          ]
+        }
+      };
+      const store = configureStore({ reducer, preloadedState });
+
+      const action = transformToReduxAction({
+        id: '_nicklist_diff',
+        header: { compression: 0, length: 0 },
+        objects: [
+          {
+            type: 'hda',
+            content: [
+              {
+                _diff: 45,
+                group: 0,
+                name: 'oldnick',
+                pointers: ['84ed37600', '85139f000']
+              },
+              {
+                _diff: 43,
+                group: 0,
+                name: 'newnick',
+                pointers: ['84ed37600', '85139f000']
+              }
+            ]
+          }
+        ]
+      });
+      expect(action).toBeDefined();
+
+      store.dispatch(action!);
+
+      expect(store.getState().nicklists).toMatchObject({
+        '84ed37600': [{ name: 'newnick', pointers: ['84ed37600', '85139f000'] }]
+      });
+    });
+  });
 });

__tests__/lib/weechat/connection.ts

@@ -1,6 +1,10 @@
 import WeechatConnection, {
   ConnectionError
 } from '../../../src/lib/weechat/connection';
+import {
+  disconnectAction,
+  fetchVersionAction
+} from '../../../src/store/actions';
 
 const mockWebSocket = jest.fn(function () {
   this.send = jest.fn();
@@ -86,10 +90,7 @@ describe(WeechatConnection, () => {
 
     expect(onSuccess).toHaveBeenCalled();
     expect(onError).not.toHaveBeenCalled();
-    expect(dispatch).toHaveBeenCalledWith({
-      type: 'FETCH_VERSION',
-      payload: '4.1.2'
-    });
+    expect(dispatch).toHaveBeenCalledWith(fetchVersionAction('4.1.2'));
   });
 
   describe('close', () => {
@@ -120,9 +121,7 @@ describe(WeechatConnection, () => {
         'quit\n'
       );
       expect(mockWebSocket.mock.instances[0].close).toHaveBeenCalled();
-      expect(dispatch).toHaveBeenNthCalledWith(2, {
-        type: 'DISCONNECT'
-      });
+      expect(dispatch).toHaveBeenNthCalledWith(2, disconnectAction());
       expect(mockWebSocket.mock.instances).toHaveLength(1);
     });
 
@@ -151,9 +150,7 @@ describe(WeechatConnection, () => {
       mockWebSocket.mock.instances[0].close();
 
       expect(onError).toHaveBeenCalledWith(true, ConnectionError.Socket);
-      expect(dispatch).toHaveBeenNthCalledWith(2, {
-        type: 'DISCONNECT'
-      });
+      expect(dispatch).toHaveBeenNthCalledWith(2, disconnectAction());
       expect(mockWebSocket.mock.instances).toHaveLength(2);
     });
   });