@k-int/stripes-kint-components 5.36.1 → 5.37.0

package/es/lib/hooks/useMutateGeneric/useMutateGeneric.js

@@ -12,54 +12,76 @@ function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e
 // A utility function to provide this all-in-one functionality used in various places
 
 const useMutateGeneric = function () {
-  let {
-    afterQueryCalls: {
-      delete: afterQueryDelete = _noop.default,
-      post: afterQueryPost = _noop.default,
-      put: afterQueryPut = _noop.default
-    } = {},
-    catchQueryCalls: {
-      delete: catchQueryDelete = _noop.default,
-      post: catchQueryPost = _noop.default,
-      put: catchQueryPut = _noop.default
-    } = {},
-    endpoint,
-    endpointMutators: {
-      delete: endpointMutatorDelete = id => `${endpoint}/${id}`,
-      post: endpointMutatorPost = () => endpoint,
-      put: endpointMutatorPut = data => `${endpoint}/${data.id}`
-    } = {},
-    payloadMutators: {
-      post: payloadMutatorPost = data => ({
-        json: data
-      }),
-      put: payloadMutatorPut = data => ({
-        json: data
-      })
-    } = {},
-    promiseReturns: {
-      delete: promiseReturnDelete = (id, ky) => ky.delete(endpointMutatorDelete(id)).json(),
-      post: promiseReturnPost = (data, ky) => ky.post(endpointMutatorPost(data), payloadMutatorPost(data)).json(),
-      put: promiseReturnPut = (data, ky) => ky.put(endpointMutatorPut(data), payloadMutatorPut(data)).json()
-    } = {},
-    queryKey = [],
-    // Must be type: array
-    queryKeyMutators: {
-      delete: queryKeyMutatorDelete = () => [...queryKey, 'delete'],
-      post: queryKeyMutatorPost = () => [...queryKey, 'create'],
-      put: queryKeyMutatorPut = () => [...queryKey, 'edit']
-    } = {},
-    queryParams: {
-      delete: queryParamsDelete = {},
-      post: queryParamsPost = {},
-      put: queryParamsPut = {}
-    } = {},
-    returnQueryObject: {
-      delete: returnQueryObjectDelete = false,
-      post: returnQueryObjectPost = false,
-      put: returnQueryObjectPut = false
-    } = {}
-  } = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
+  let _ref = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {},
+    _ref$afterQueryCalls = _ref.afterQueryCalls,
+    _ref$afterQueryCalls2 = _ref$afterQueryCalls === void 0 ? {} : _ref$afterQueryCalls,
+    _ref$afterQueryCalls3 = _ref$afterQueryCalls2.delete,
+    afterQueryDelete = _ref$afterQueryCalls3 === void 0 ? _noop.default : _ref$afterQueryCalls3,
+    _ref$afterQueryCalls4 = _ref$afterQueryCalls2.post,
+    afterQueryPost = _ref$afterQueryCalls4 === void 0 ? _noop.default : _ref$afterQueryCalls4,
+    _ref$afterQueryCalls5 = _ref$afterQueryCalls2.put,
+    afterQueryPut = _ref$afterQueryCalls5 === void 0 ? _noop.default : _ref$afterQueryCalls5,
+    _ref$catchQueryCalls = _ref.catchQueryCalls,
+    _ref$catchQueryCalls2 = _ref$catchQueryCalls === void 0 ? {} : _ref$catchQueryCalls,
+    _ref$catchQueryCalls3 = _ref$catchQueryCalls2.delete,
+    catchQueryDelete = _ref$catchQueryCalls3 === void 0 ? _noop.default : _ref$catchQueryCalls3,
+    _ref$catchQueryCalls4 = _ref$catchQueryCalls2.post,
+    catchQueryPost = _ref$catchQueryCalls4 === void 0 ? _noop.default : _ref$catchQueryCalls4,
+    _ref$catchQueryCalls5 = _ref$catchQueryCalls2.put,
+    catchQueryPut = _ref$catchQueryCalls5 === void 0 ? _noop.default : _ref$catchQueryCalls5,
+    endpoint = _ref.endpoint,
+    _ref$endpointMutators = _ref.endpointMutators,
+    _ref$endpointMutators2 = _ref$endpointMutators === void 0 ? {} : _ref$endpointMutators,
+    _ref$endpointMutators3 = _ref$endpointMutators2.delete,
+    endpointMutatorDelete = _ref$endpointMutators3 === void 0 ? id => "".concat(endpoint, "/").concat(id) : _ref$endpointMutators3,
+    _ref$endpointMutators4 = _ref$endpointMutators2.post,
+    endpointMutatorPost = _ref$endpointMutators4 === void 0 ? () => endpoint : _ref$endpointMutators4,
+    _ref$endpointMutators5 = _ref$endpointMutators2.put,
+    endpointMutatorPut = _ref$endpointMutators5 === void 0 ? data => "".concat(endpoint, "/").concat(data.id) : _ref$endpointMutators5,
+    _ref$payloadMutators = _ref.payloadMutators,
+    _ref$payloadMutators2 = _ref$payloadMutators === void 0 ? {} : _ref$payloadMutators,
+    _ref$payloadMutators3 = _ref$payloadMutators2.post,
+    payloadMutatorPost = _ref$payloadMutators3 === void 0 ? data => ({
+      json: data
+    }) : _ref$payloadMutators3,
+    _ref$payloadMutators4 = _ref$payloadMutators2.put,
+    payloadMutatorPut = _ref$payloadMutators4 === void 0 ? data => ({
+      json: data
+    }) : _ref$payloadMutators4,
+    _ref$promiseReturns = _ref.promiseReturns,
+    _ref$promiseReturns2 = _ref$promiseReturns === void 0 ? {} : _ref$promiseReturns,
+    _ref$promiseReturns2$ = _ref$promiseReturns2.delete,
+    promiseReturnDelete = _ref$promiseReturns2$ === void 0 ? (id, ky) => ky.delete(endpointMutatorDelete(id)).json() : _ref$promiseReturns2$,
+    _ref$promiseReturns2$2 = _ref$promiseReturns2.post,
+    promiseReturnPost = _ref$promiseReturns2$2 === void 0 ? (data, ky) => ky.post(endpointMutatorPost(data), payloadMutatorPost(data)).json() : _ref$promiseReturns2$2,
+    _ref$promiseReturns2$3 = _ref$promiseReturns2.put,
+    promiseReturnPut = _ref$promiseReturns2$3 === void 0 ? (data, ky) => ky.put(endpointMutatorPut(data), payloadMutatorPut(data)).json() : _ref$promiseReturns2$3,
+    _ref$queryKey = _ref.queryKey,
+    queryKey = _ref$queryKey === void 0 ? [] : _ref$queryKey,
+    _ref$queryKeyMutators = _ref.queryKeyMutators,
+    _ref$queryKeyMutators2 = _ref$queryKeyMutators === void 0 ? {} : _ref$queryKeyMutators,
+    _ref$queryKeyMutators3 = _ref$queryKeyMutators2.delete,
+    queryKeyMutatorDelete = _ref$queryKeyMutators3 === void 0 ? () => [...queryKey, 'delete'] : _ref$queryKeyMutators3,
+    _ref$queryKeyMutators4 = _ref$queryKeyMutators2.post,
+    queryKeyMutatorPost = _ref$queryKeyMutators4 === void 0 ? () => [...queryKey, 'create'] : _ref$queryKeyMutators4,
+    _ref$queryKeyMutators5 = _ref$queryKeyMutators2.put,
+    queryKeyMutatorPut = _ref$queryKeyMutators5 === void 0 ? () => [...queryKey, 'edit'] : _ref$queryKeyMutators5,
+    _ref$queryParams = _ref.queryParams,
+    _ref$queryParams2 = _ref$queryParams === void 0 ? {} : _ref$queryParams,
+    _ref$queryParams2$del = _ref$queryParams2.delete,
+    queryParamsDelete = _ref$queryParams2$del === void 0 ? {} : _ref$queryParams2$del,
+    _ref$queryParams2$pos = _ref$queryParams2.post,
+    queryParamsPost = _ref$queryParams2$pos === void 0 ? {} : _ref$queryParams2$pos,
+    _ref$queryParams2$put = _ref$queryParams2.put,
+    queryParamsPut = _ref$queryParams2$put === void 0 ? {} : _ref$queryParams2$put,
+    _ref$returnQueryObjec = _ref.returnQueryObject,
+    _ref$returnQueryObjec2 = _ref$returnQueryObjec === void 0 ? {} : _ref$returnQueryObjec,
+    _ref$returnQueryObjec3 = _ref$returnQueryObjec2.delete,
+    returnQueryObjectDelete = _ref$returnQueryObjec3 === void 0 ? false : _ref$returnQueryObjec3,
+    _ref$returnQueryObjec4 = _ref$returnQueryObjec2.post,
+    returnQueryObjectPost = _ref$returnQueryObjec4 === void 0 ? false : _ref$returnQueryObjec4,
+    _ref$returnQueryObjec5 = _ref$returnQueryObjec2.put,
+    returnQueryObjectPut = _ref$returnQueryObjec5 === void 0 ? false : _ref$returnQueryObjec5;
   const ky = (0, _core.useOkapiKy)();
 
   // DELETE Object

package/es/lib/hooks/useMutateGeneric/useMutateGeneric.test.js

@@ -0,0 +1,192 @@
+"use strict";
+
+var _reactQuery = require("react-query");
+var _core = require("@folio/stripes/core");
+var _react = require("@folio/jest-config-stripes/testing-library/react");
+var _useMutateGeneric = _interopRequireDefault(require("./useMutateGeneric"));
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
+// EXAMPLE -- Using the mockKy spies from useOkapiKy mock
+
+describe('useMutateGeneric', () => {
+  const mockEndpoint = 'test/endpoint';
+  const mockQueryKey = ['test', 'key'];
+  beforeEach(() => {
+    jest.clearAllMocks();
+  });
+  describe.each([{
+    verb: 'delete',
+    callIndex: 0,
+    payload: '12345',
+    expectedKeySuffix: 'delete',
+    expectedUrl: 'test/endpoint/12345',
+    mockSpy: _core.mockDelete
+  }, {
+    verb: 'put',
+    callIndex: 1,
+    payload: {
+      id: '12345',
+      name: 'test'
+    },
+    expectedKeySuffix: 'edit',
+    expectedUrl: 'test/endpoint/12345',
+    expectedPayload: {
+      json: {
+        id: '12345',
+        name: 'test'
+      }
+    },
+    mockSpy: _core.mockPut
+  }, {
+    verb: 'post',
+    callIndex: 2,
+    payload: {
+      name: 'new test'
+    },
+    expectedKeySuffix: 'create',
+    expectedUrl: 'test/endpoint',
+    expectedPayload: {
+      json: {
+        name: 'new test'
+      }
+    },
+    mockSpy: _core.mockPost
+  }])('Verb: $verb', _ref => {
+    let verb = _ref.verb,
+      callIndex = _ref.callIndex,
+      payload = _ref.payload,
+      expectedKeySuffix = _ref.expectedKeySuffix,
+      expectedUrl = _ref.expectedUrl,
+      expectedPayload = _ref.expectedPayload,
+      mockSpy = _ref.mockSpy;
+    describe("initialization for ".concat(verb), () => {
+      let hookResult;
+      beforeEach(() => {
+        const _renderHook = (0, _react.renderHook)(() => (0, _useMutateGeneric.default)({
+            endpoint: mockEndpoint,
+            queryKey: mockQueryKey
+          })),
+          result = _renderHook.result;
+        hookResult = result;
+      });
+      test("returns the ".concat(verb, " mutation function"), () => {
+        expect(typeof hookResult.current[verb]).toBe('function');
+      });
+      test("configures useMutation with the correct ".concat(verb, " queryKey"), () => {
+        // useMutation call index: 0=delete, 1=put, 2=post
+        expect(_reactQuery.useMutation.mock.calls[callIndex][0]).toEqual([...mockQueryKey, expectedKeySuffix]);
+      });
+    });
+    describe("with returnQueryObject: true, return full mutation object for ".concat(verb), () => {
+      let hookResult;
+      beforeEach(() => {
+        const _renderHook2 = (0, _react.renderHook)(() => (0, _useMutateGeneric.default)({
+            endpoint: mockEndpoint,
+            returnQueryObject: {
+              [verb]: true
+            }
+          })),
+          result = _renderHook2.result;
+        hookResult = result;
+      });
+      test("returns the full mutation object for ".concat(verb, " when requested"), () => {
+        expect(hookResult.current[verb]).toHaveProperty('mutateAsync');
+      });
+    });
+    describe("execution for ".concat(verb), () => {
+      const afterCall = jest.fn();
+      const catchCall = jest.fn();
+      const customMutator = jest.fn(p => verb === 'post' ? 'custom/post' : "custom/".concat((p === null || p === void 0 ? void 0 : p.id) || p));
+      describe('successful lifecycle', () => {
+        const successData = {
+          id: 'some-id'
+        };
+        beforeEach(async () => {
+          _core.mockKyJson.mockResolvedValueOnce(successData);
+          const _renderHook3 = (0, _react.renderHook)(() => (0, _useMutateGeneric.default)({
+              endpoint: mockEndpoint,
+              afterQueryCalls: {
+                [verb]: afterCall
+              },
+              endpointMutators: {
+                [verb]: customMutator
+              }
+            })),
+            result = _renderHook3.result;
+          await (0, _react.waitFor)(async () => {
+            await result.current[verb](payload);
+          });
+        });
+        test('calls the custom endpointMutator', () => {
+          expect(customMutator).toHaveBeenCalledWith(payload);
+        });
+        test('calls the correct ky verb with the mutated URL', () => {
+          const expectedCustomUrl = verb === 'post' ? 'custom/post' : 'custom/12345';
+          if (expectedPayload) {
+            expect(mockSpy).toHaveBeenCalledWith(expectedCustomUrl, expectedPayload);
+          } else {
+            expect(mockSpy).toHaveBeenCalledWith(expectedCustomUrl);
+          }
+        });
+        test('executes the afterQuery success callback', () => {
+          expect(afterCall).toHaveBeenCalledWith(successData);
+        });
+      });
+      describe('failure lifecycle', () => {
+        beforeEach(async () => {
+          mockSpy.mockImplementationOnce(() => ({
+            json: () => Promise.reject(new Error('Network Error'))
+          }));
+          const _renderHook4 = (0, _react.renderHook)(() => (0, _useMutateGeneric.default)({
+              endpoint: mockEndpoint,
+              catchQueryCalls: {
+                [verb]: catchCall
+              }
+            })),
+            result = _renderHook4.result;
+          await (0, _react.waitFor)(async () => {
+            try {
+              await result.current[verb](payload);
+            } catch (e) {/* error handled in hook */}
+          });
+        });
+        test('executes the catchQuery failure callback', () => {
+          expect(catchCall).toHaveBeenCalled();
+        });
+      });
+      describe('default mutators fallback', () => {
+        beforeEach(async () => {
+          const _renderHook5 = (0, _react.renderHook)(() => (0, _useMutateGeneric.default)({
+              endpoint: mockEndpoint
+            })),
+            result = _renderHook5.result;
+          await (0, _react.waitFor)(async () => {
+            await result.current[verb](payload);
+          });
+        });
+        test('uses default URL construction logic', () => {
+          if (expectedPayload) {
+            expect(mockSpy).toHaveBeenCalledWith(expectedUrl, expectedPayload);
+          } else {
+            expect(mockSpy).toHaveBeenCalledWith(expectedUrl);
+          }
+        });
+      });
+    });
+    describe("returnQueryObject toggle for ".concat(verb), () => {
+      let hookResult;
+      beforeEach(() => {
+        const _renderHook6 = (0, _react.renderHook)(() => (0, _useMutateGeneric.default)({
+            endpoint: mockEndpoint,
+            returnQueryObject: {
+              [verb]: true
+            }
+          })),
+          result = _renderHook6.result;
+        hookResult = result;
+      });
+      test('returns the full mutation object', () => {
+        expect(hookResult.current[verb]).toHaveProperty('mutateAsync');
+      });
+    });
+  });
+});

package/es/lib/hooks/useMutateModConfigEntry.js

@@ -12,11 +12,9 @@ var _endpoints = require("../constants/endpoints");
 // DEPRECATED -- As far as I can see this is now unused anyway
 // KInt modules are swapping to AppSettings instead of mod-settings, so no centralised components should be needed
 const useMutateModConfigEntry = _ref => {
-  let {
-    configName,
-    moduleName,
-    id
-  } = _ref;
+  let configName = _ref.configName,
+    moduleName = _ref.moduleName,
+    id = _ref.id;
   // eslint-disable-next-line no-console
   console.warn('useMutateModConfigEntry is deprecated, as it utilises mod-config');
   const ky = (0, _core.useOkapiKy)();
@@ -25,12 +23,12 @@ const useMutateModConfigEntry = _ref => {
     configName,
     moduleName
   });
-  const putQueryObject = (0, _reactQuery.useMutation)([...baseNamespace, 'putConfigEntry', id], async data => ky.put(`${_endpoints.MOD_SETTINGS_ENDPOINT}/${id}`, {
+  const putQueryObject = (0, _reactQuery.useMutation)([...baseNamespace, 'putConfigEntry', id], async data => ky.put("".concat(_endpoints.MOD_SETTINGS_ENDPOINT, "/").concat(id), {
     json: data
   }).json().then(() => {
     queryClient.invalidateQueries(baseNamespace);
   }));
-  const postQueryObject = (0, _reactQuery.useMutation)([...baseNamespace, 'postConfigEntry'], async data => ky.post(`${_endpoints.MOD_SETTINGS_ENDPOINT}`, {
+  const postQueryObject = (0, _reactQuery.useMutation)([...baseNamespace, 'postConfigEntry'], async data => ky.post("".concat(_endpoints.MOD_SETTINGS_ENDPOINT), {
     json: data
   }).json().then(() => {
     queryClient.invalidateQueries(baseNamespace);

package/es/lib/hooks/useMutateRefdataCategory/useMutateRefdataCategory.js

@@ -7,19 +7,28 @@ exports.default = void 0;
 var _noop = _interopRequireDefault(require("lodash/noop"));
 var _useInvalidateRefdata = _interopRequireDefault(require("../useInvalidateRefdata"));
 var _useMutateGeneric = _interopRequireDefault(require("../useMutateGeneric"));
+const _excluded = ["afterQueryCalls", "endpoint"];
 function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
+function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
+function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
+function _defineProperty(e, r, t) { return (r = _toPropertyKey(r)) in e ? Object.defineProperty(e, r, { value: t, enumerable: !0, configurable: !0, writable: !0 }) : e[r] = t, e; }
+function _toPropertyKey(t) { var i = _toPrimitive(t, "string"); return "symbol" == typeof i ? i : i + ""; }
+function _toPrimitive(t, r) { if ("object" != typeof t || !t) return t; var e = t[Symbol.toPrimitive]; if (void 0 !== e) { var i = e.call(t, r || "default"); if ("object" != typeof i) return i; throw new TypeError("@@toPrimitive must return a primitive value."); } return ("string" === r ? String : Number)(t); }
+function _objectWithoutProperties(e, t) { if (null == e) return {}; var o, r, i = _objectWithoutPropertiesLoose(e, t); if (Object.getOwnPropertySymbols) { var n = Object.getOwnPropertySymbols(e); for (r = 0; r < n.length; r++) o = n[r], -1 === t.indexOf(o) && {}.propertyIsEnumerable.call(e, o) && (i[o] = e[o]); } return i; }
+function _objectWithoutPropertiesLoose(r, e) { if (null == r) return {}; var t = {}; for (var n in r) if ({}.hasOwnProperty.call(r, n)) { if (-1 !== e.indexOf(n)) continue; t[n] = r[n]; } return t; }
 const useMutateRefdataCategory = _ref => {
-  let {
-    afterQueryCalls: {
-      delete: afterQueryDelete = _noop.default,
-      post: afterQueryPost = _noop.default,
-      put: afterQueryPut = _noop.default
-    } = {},
-    endpoint,
-    ...mutateGenericProps
-  } = _ref;
+  let _ref$afterQueryCalls = _ref.afterQueryCalls,
+    _ref$afterQueryCalls2 = _ref$afterQueryCalls === void 0 ? {} : _ref$afterQueryCalls,
+    _ref$afterQueryCalls3 = _ref$afterQueryCalls2.delete,
+    afterQueryDelete = _ref$afterQueryCalls3 === void 0 ? _noop.default : _ref$afterQueryCalls3,
+    _ref$afterQueryCalls4 = _ref$afterQueryCalls2.post,
+    afterQueryPost = _ref$afterQueryCalls4 === void 0 ? _noop.default : _ref$afterQueryCalls4,
+    _ref$afterQueryCalls5 = _ref$afterQueryCalls2.put,
+    afterQueryPut = _ref$afterQueryCalls5 === void 0 ? _noop.default : _ref$afterQueryCalls5,
+    endpoint = _ref.endpoint,
+    mutateGenericProps = _objectWithoutProperties(_ref, _excluded);
   const invalidateRefdata = (0, _useInvalidateRefdata.default)();
-  return (0, _useMutateGeneric.default)({
+  return (0, _useMutateGeneric.default)(_objectSpread({
     afterQueryCalls: {
       delete: res => {
         invalidateRefdata().then(() => {
@@ -40,14 +49,12 @@ const useMutateRefdataCategory = _ref => {
     endpoint,
     payloadMutators: {
       post: data => ({
-        json: {
-          ...data,
+        json: _objectSpread(_objectSpread({}, data), {}, {
           values: []
-        }
+        })
       })
     },
-    queryKey: ['stripes-kint-components', 'useMutateRefdataCategory'],
-    ...mutateGenericProps
-  });
+    queryKey: ['stripes-kint-components', 'useMutateRefdataCategory']
+  }, mutateGenericProps));
 };
 var _default = exports.default = useMutateRefdataCategory;

package/es/lib/hooks/useMutateRefdataCategory/useMutateRefdataCategory.test.js

@@ -0,0 +1,69 @@
+"use strict";
+
+var _react = require("@folio/jest-config-stripes/testing-library/react");
+var _useMutateGeneric = _interopRequireDefault(require("../useMutateGeneric"));
+var _useInvalidateRefdata = _interopRequireDefault(require("../useInvalidateRefdata"));
+var _useMutateRefdataCategory = _interopRequireDefault(require("./useMutateRefdataCategory"));
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
+// Mocking dependencies
+jest.mock('../useMutateGeneric');
+jest.mock('../useInvalidateRefdata');
+describe('useMutateRefdataCategory', () => {
+  const mockEndpoint = 'erm/refdata';
+  const mockInvalidate = jest.fn(() => Promise.resolve());
+  beforeEach(() => {
+    jest.clearAllMocks();
+    _useInvalidateRefdata.default.mockReturnValue(mockInvalidate);
+  });
+  describe('initialization', () => {
+    let sentConfig;
+    beforeEach(() => {
+      (0, _react.renderHook)(() => (0, _useMutateRefdataCategory.default)({
+        endpoint: mockEndpoint
+      }));
+      sentConfig = _useMutateGeneric.default.mock.calls[0][0];
+    });
+    test('passes the correct endpoint', () => {
+      expect(sentConfig.endpoint).toBe(mockEndpoint);
+    });
+    test('constructs the correct queryKey', () => {
+      expect(sentConfig.queryKey).toEqual(['stripes-kint-components', 'useMutateRefdataCategory']);
+    });
+    test('payloadMutator for post adds an empty values array', () => {
+      const result = sentConfig.payloadMutators.post({
+        desc: 'new category'
+      });
+      expect(result.json).toEqual({
+        desc: 'new category',
+        values: []
+      });
+    });
+  });
+  describe('afterQueryCalls lifecycle', () => {
+    const mockRes = {
+      id: '123'
+    };
+    let sentConfig;
+    describe.each(['delete', 'post', 'put'])('Verb: %s', verb => {
+      const externalCallback = jest.fn();
+      beforeEach(async () => {
+        (0, _react.renderHook)(() => (0, _useMutateRefdataCategory.default)({
+          endpoint: mockEndpoint,
+          afterQueryCalls: {
+            [verb]: externalCallback
+          }
+        }));
+        sentConfig = _useMutateGeneric.default.mock.calls[0][0];
+
+        // Manually trigger the wrapper function defined in the hook
+        await sentConfig.afterQueryCalls[verb](mockRes);
+      });
+      test("calls invalidateRefdata for ".concat(verb), () => {
+        expect(mockInvalidate).toHaveBeenCalled();
+      });
+      test("calls the provided afterQuery".concat(verb.charAt(0).toUpperCase() + verb.slice(1), " callback"), () => {
+        expect(externalCallback).toHaveBeenCalledWith(mockRes);
+      });
+    });
+  });
+});

package/es/lib/hooks/useMutateRefdataValue/useMutateRefdataValue.js

@@ -7,20 +7,29 @@ exports.default = void 0;
 var _noop = _interopRequireDefault(require("lodash/noop"));
 var _useInvalidateRefdata = _interopRequireDefault(require("../useInvalidateRefdata"));
 var _useMutateGeneric = _interopRequireDefault(require("../useMutateGeneric"));
+const _excluded = ["afterQueryCalls", "endpoint", "id"];
 function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
+function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
+function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
+function _defineProperty(e, r, t) { return (r = _toPropertyKey(r)) in e ? Object.defineProperty(e, r, { value: t, enumerable: !0, configurable: !0, writable: !0 }) : e[r] = t, e; }
+function _toPropertyKey(t) { var i = _toPrimitive(t, "string"); return "symbol" == typeof i ? i : i + ""; }
+function _toPrimitive(t, r) { if ("object" != typeof t || !t) return t; var e = t[Symbol.toPrimitive]; if (void 0 !== e) { var i = e.call(t, r || "default"); if ("object" != typeof i) return i; throw new TypeError("@@toPrimitive must return a primitive value."); } return ("string" === r ? String : Number)(t); }
+function _objectWithoutProperties(e, t) { if (null == e) return {}; var o, r, i = _objectWithoutPropertiesLoose(e, t); if (Object.getOwnPropertySymbols) { var n = Object.getOwnPropertySymbols(e); for (r = 0; r < n.length; r++) o = n[r], -1 === t.indexOf(o) && {}.propertyIsEnumerable.call(e, o) && (i[o] = e[o]); } return i; }
+function _objectWithoutPropertiesLoose(r, e) { if (null == r) return {}; var t = {}; for (var n in r) if ({}.hasOwnProperty.call(r, n)) { if (-1 !== e.indexOf(n)) continue; t[n] = r[n]; } return t; }
 const useMutateRefdataValue = _ref => {
-  let {
-    afterQueryCalls: {
-      delete: afterQueryDelete = _noop.default,
-      post: afterQueryPost = _noop.default,
-      put: afterQueryPut = _noop.default
-    } = {},
-    endpoint,
-    id,
-    ...mutateGenericProps
-  } = _ref;
+  let _ref$afterQueryCalls = _ref.afterQueryCalls,
+    _ref$afterQueryCalls2 = _ref$afterQueryCalls === void 0 ? {} : _ref$afterQueryCalls,
+    _ref$afterQueryCalls3 = _ref$afterQueryCalls2.delete,
+    afterQueryDelete = _ref$afterQueryCalls3 === void 0 ? _noop.default : _ref$afterQueryCalls3,
+    _ref$afterQueryCalls4 = _ref$afterQueryCalls2.post,
+    afterQueryPost = _ref$afterQueryCalls4 === void 0 ? _noop.default : _ref$afterQueryCalls4,
+    _ref$afterQueryCalls5 = _ref$afterQueryCalls2.put,
+    afterQueryPut = _ref$afterQueryCalls5 === void 0 ? _noop.default : _ref$afterQueryCalls5,
+    endpoint = _ref.endpoint,
+    id = _ref.id,
+    mutateGenericProps = _objectWithoutProperties(_ref, _excluded);
   const invalidateRefdata = (0, _useInvalidateRefdata.default)();
-  return (0, _useMutateGeneric.default)({
+  return (0, _useMutateGeneric.default)(_objectSpread({
     afterQueryCalls: {
       delete: res => {
         invalidateRefdata().then(() => {
@@ -41,7 +50,7 @@ const useMutateRefdataValue = _ref => {
     endpoint,
     id,
     promiseReturns: {
-      delete: (delId, ky) => ky.put(`${endpoint}/${id}`, {
+      delete: (delId, ky) => ky.put("".concat(endpoint, "/").concat(id), {
         json: {
           id,
           values: [{
@@ -50,21 +59,20 @@ const useMutateRefdataValue = _ref => {
           }]
         }
       }).json(),
-      post: (data, ky) => ky.put(`${endpoint}/${id}`, {
+      post: (data, ky) => ky.put("".concat(endpoint, "/").concat(id), {
         json: {
           id,
           values: [data]
         }
       }).json(),
-      put: (data, ky) => ky.put(`${endpoint}/${id}`, {
+      put: (data, ky) => ky.put("".concat(endpoint, "/").concat(id), {
         json: {
           id,
           values: [data]
         }
       }).json()
     },
-    queryKey: ['stripes-kint-components', 'useMutateRefdataValue', id],
-    ...mutateGenericProps
-  });
+    queryKey: ['stripes-kint-components', 'useMutateRefdataValue', id]
+  }, mutateGenericProps));
 };
 var _default = exports.default = useMutateRefdataValue;

package/es/lib/hooks/useMutateRefdataValue/useMutateRefdataValue.test.js

@@ -0,0 +1,95 @@
+"use strict";
+
+var _react = require("@folio/jest-config-stripes/testing-library/react");
+var _useMutateGeneric = _interopRequireDefault(require("../useMutateGeneric"));
+var _useInvalidateRefdata = _interopRequireDefault(require("../useInvalidateRefdata"));
+var _useMutateRefdataValue = _interopRequireDefault(require("./useMutateRefdataValue"));
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
+jest.mock('../useMutateGeneric');
+jest.mock('../useInvalidateRefdata');
+describe('useMutateRefdataValue', () => {
+  const mockEndpoint = 'erm/refdata/values';
+  const mockId = 'category-123';
+  const mockInvalidate = jest.fn(() => Promise.resolve());
+  const mockKy = {
+    put: jest.fn(() => ({
+      json: () => Promise.resolve({
+        success: true
+      })
+    }))
+  };
+  beforeEach(() => {
+    jest.clearAllMocks();
+    _useInvalidateRefdata.default.mockReturnValue(mockInvalidate);
+  });
+  describe('initialization', () => {
+    test('constructs the correct queryKey using the ID', () => {
+      (0, _react.renderHook)(() => (0, _useMutateRefdataValue.default)({
+        endpoint: mockEndpoint,
+        id: mockId
+      }));
+      const sentConfig = _useMutateGeneric.default.mock.calls[0][0];
+      expect(sentConfig.queryKey).toEqual(['stripes-kint-components', 'useMutateRefdataValue', mockId]);
+    });
+  });
+  describe('custom promiseReturns logic', () => {
+    let promiseReturns;
+    beforeEach(() => {
+      (0, _react.renderHook)(() => (0, _useMutateRefdataValue.default)({
+        endpoint: mockEndpoint,
+        id: mockId
+      }));
+      promiseReturns = _useMutateGeneric.default.mock.calls[0][0].promiseReturns;
+    });
+    test('delete: sends a PUT to the parent category with _delete: true', async () => {
+      await promiseReturns.delete('value-abc', mockKy);
+      expect(mockKy.put).toHaveBeenCalledWith("".concat(mockEndpoint, "/").concat(mockId), {
+        json: {
+          id: mockId,
+          values: [{
+            id: 'value-abc',
+            _delete: true
+          }]
+        }
+      });
+    });
+    test('post: sends a PUT to the parent category with the new value data', async () => {
+      const newData = {
+        label: 'New Value'
+      };
+      await promiseReturns.post(newData, mockKy);
+      expect(mockKy.put).toHaveBeenCalledWith("".concat(mockEndpoint, "/").concat(mockId), {
+        json: {
+          id: mockId,
+          values: [newData]
+        }
+      });
+    });
+    test('put: sends a PUT to the parent category with the updated value data', async () => {
+      const updateData = {
+        id: 'val-1',
+        label: 'Updated'
+      };
+      await promiseReturns.put(updateData, mockKy);
+      expect(mockKy.put).toHaveBeenCalledWith("".concat(mockEndpoint, "/").concat(mockId), {
+        json: {
+          id: mockId,
+          values: [updateData]
+        }
+      });
+    });
+  });
+  describe('afterQueryCalls lifecycle', () => {
+    test.each(['delete', 'post', 'put'])('Verb: %s triggers invalidation', async verb => {
+      (0, _react.renderHook)(() => (0, _useMutateRefdataValue.default)({
+        endpoint: mockEndpoint,
+        id: mockId
+      }));
+      const afterQueryCalls = _useMutateGeneric.default.mock.calls[0][0].afterQueryCalls;
+      await afterQueryCalls[verb]({
+        success: true
+      });
+      expect(mockInvalidate).toHaveBeenCalled();
+    });
+  });
+});