@openmrs/esm-react-utils 5.3.3-pre.1359 → 5.3.3-pre.1370

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openmrs/esm-react-utils",
-  "version": "5.3.3-pre.1359",
+  "version": "5.3.3-pre.1370",
   "license": "MPL-2.0",
   "description": "React utilities for OpenMRS.",
   "browser": "dist/openmrs-esm-react-utils.js",
@@ -48,6 +48,7 @@
     "@openmrs/esm-error-handling": "5.x",
     "@openmrs/esm-extensions": "5.x",
     "@openmrs/esm-globals": "5.x",
+    "@openmrs/esm-navigation": "5.x",
     "dayjs": "1.x",
     "i18next": "19.x",
     "react": "18.x",
@@ -57,12 +58,13 @@
     "swr": "2.x"
   },
   "devDependencies": {
-    "@openmrs/esm-api": "5.3.3-pre.1359",
-    "@openmrs/esm-config": "5.3.3-pre.1359",
-    "@openmrs/esm-error-handling": "5.3.3-pre.1359",
-    "@openmrs/esm-extensions": "5.3.3-pre.1359",
-    "@openmrs/esm-feature-flags": "5.3.3-pre.1359",
-    "@openmrs/esm-globals": "5.3.3-pre.1359",
+    "@openmrs/esm-api": "5.3.3-pre.1370",
+    "@openmrs/esm-config": "5.3.3-pre.1370",
+    "@openmrs/esm-error-handling": "5.3.3-pre.1370",
+    "@openmrs/esm-extensions": "5.3.3-pre.1370",
+    "@openmrs/esm-feature-flags": "5.3.3-pre.1370",
+    "@openmrs/esm-globals": "5.3.3-pre.1370",
+    "@openmrs/esm-navigation": "5.3.3-pre.1370",
     "dayjs": "^1.10.8",
     "i18next": "^21.10.0",
     "react": "^18.1.0",

package/src/ConfigurableLink.test.tsx

@@ -2,18 +2,13 @@ import React from 'react';
 import '@testing-library/jest-dom';
 import { render, screen } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
-import { navigate, interpolateUrl } from '@openmrs/esm-config';
+import { navigate } from '@openmrs/esm-navigation';
 import { ConfigurableLink } from './ConfigurableLink';
 
 jest.mock('single-spa');
 
-jest.mock('@openmrs/esm-config');
 const mockNavigate = navigate as jest.Mock;
 
-const realInterpolate = jest.requireActual('@openmrs/esm-config').interpolateUrl;
-
-(interpolateUrl as jest.Mock).mockImplementation((...args) => realInterpolate(...args));
-
 describe(`ConfigurableLink`, () => {
   const path = '${openmrsSpaBase}/home';
   beforeEach(() => {

package/src/ConfigurableLink.tsx

@@ -1,7 +1,7 @@
 /** @module @category Navigation */
 import React, { type MouseEvent, type AnchorHTMLAttributes, type PropsWithChildren } from 'react';
-import type { TemplateParams } from '@openmrs/esm-config';
-import { navigate, interpolateUrl } from '@openmrs/esm-config';
+import type { TemplateParams } from '@openmrs/esm-navigation';
+import { navigate, interpolateUrl } from '@openmrs/esm-navigation';
 
 function handleClick(event: MouseEvent, to: string, templateParams?: TemplateParams, onBeforeNavigate?: () => void) {
   if (!event.metaKey && !event.ctrlKey && !event.shiftKey && event.button == 0) {

package/src/extensions.test.tsx

@@ -18,7 +18,7 @@ import {
 import userEvent from '@testing-library/user-event';
 import { registerFeatureFlag, setFeatureFlag } from '@openmrs/esm-feature-flags';
 
-// For some reason in the text context `isEqual` always returns true
+// For some reason in the test context `isEqual` always returns true
 // when using the import substitution in jest.config.js. Here's a custom
 // mock.
 jest.mock('lodash-es/isEqual', () => (a, b) => JSON.stringify(a) == JSON.stringify(b));

package/src/setup-tests.js

@@ -1,5 +1,11 @@
+import { jest } from '@jest/globals'
 import '@testing-library/jest-dom';
 
 global.window.openmrsBase = '/openmrs';
 global.window.spaBase = '/spa';
 global.window.getOpenmrsSpaBase = () => '/openmrs/spa/';
+
+jest.mock('@openmrs/esm-navigation', () => ({
+  ...jest.requireActual('@openmrs/esm-navigation'),
+  navigate: jest.fn(),
+}));