@openmrs/esm-react-utils 4.0.0-pre.1 → 4.0.1-pre.204

package/src/useAssignedExtensions.ts

@@ -5,7 +5,7 @@ import {
   ExtensionStore,
   getExtensionStore,
 } from "@openmrs/esm-extensions";
-import { isEqual } from "lodash";
+import isEqual from "lodash/isEqual";
 
 /**
  * Gets the assigned extensions for a given extension slot name.

package/src/useConfig.test.tsx

@@ -1,5 +1,5 @@
 import React from "react";
-import { render, cleanup, screen, waitFor } from "@testing-library/react";
+import { render, cleanup, screen, waitFor, act } from "@testing-library/react";
 import {
   defineConfigSchema,
   temporaryConfigStore,
@@ -26,7 +26,6 @@ function clearConfig() {
 
 describe(`useConfig in root context`, () => {
   afterEach(clearConfig);
-  afterEach(cleanup);
 
   it(`can return config as a react hook`, async () => {
     defineConfigSchema("foo-module", {
@@ -103,9 +102,11 @@ describe(`useConfig in root context`, () => {
       expect(screen.findByText("The first thing")).toBeTruthy()
     );
 
-    temporaryConfigStore.setState({
-      config: { "foo-module": { thing: "A new thing" } },
-    });
+    act(() =>
+      temporaryConfigStore.setState({
+        config: { "foo-module": { thing: "A new thing" } },
+      })
+    );
 
     await waitFor(() => expect(screen.findByText("A new thing")).toBeTruthy());
   });
@@ -273,7 +274,7 @@ describe(`useConfig in an extension`, () => {
     );
 
     const newConfig = { "ext-module": { thing: "A new thing" } };
-    temporaryConfigStore.setState({ config: newConfig });
+    act(() => temporaryConfigStore.setState({ config: newConfig }));
 
     await waitFor(() => expect(screen.findByText("A new thing")).toBeTruthy());
 
@@ -290,7 +291,7 @@ describe(`useConfig in an extension`, () => {
         },
       },
     };
-    temporaryConfigStore.setState({ config: newConfig2 });
+    act(() => temporaryConfigStore.setState({ config: newConfig2 }));
 
     await waitFor(() =>
       expect(screen.findByText("Yet another thing")).toBeTruthy()

package/src/useConfig.ts

@@ -4,11 +4,11 @@ import {
   getConfigStore,
   getExtensionsConfigStore,
   ConfigStore,
+  ConfigObject,
   ExtensionsConfigStore,
   getExtensionConfigFromStore,
 } from "@openmrs/esm-config";
 import { ComponentContext, ExtensionData } from "./ComponentContext";
-import { ConfigObject } from "@openmrs/esm-config";
 import { Store } from "unistore";
 import isEqual from "lodash-es/isEqual";
 

package/.turbo/turbo-lint.log

@@ -1,2 +0,0 @@
-@openmrs/esm-react-utils:lint: cache hit, replaying output 79f18ce1ffc19bd2
-@openmrs/esm-react-utils:lint: $ eslint src

package/.turbo/turbo-test.log

@@ -1,45 +0,0 @@
-@openmrs/esm-react-utils:test: cache hit, replaying output 5c98fc05f32d4ded
-@openmrs/esm-react-utils:test: $ jest --passWithNoTests
-@openmrs/esm-react-utils:test: PASS src/openmrsComponentDecorator.test.tsx (12.76 s)
-@openmrs/esm-react-utils:test: PASS src/useSession.test.tsx (14.355 s)
-@openmrs/esm-react-utils:test: PASS src/useOnClickOutside.test.tsx (14.227 s)
-@openmrs/esm-react-utils:test: PASS src/useConfig.test.tsx (16.25 s)
-@openmrs/esm-react-utils:test:   ● Console
-@openmrs/esm-react-utils:test: 
-@openmrs/esm-react-utils:test:     console.error
-@openmrs/esm-react-utils:test:       Warning: An update to RenderConfig inside a test was not wrapped in act(...).
-@openmrs/esm-react-utils:test:       
-@openmrs/esm-react-utils:test:       When testing, code that causes React state updates should be wrapped into act(...):
-@openmrs/esm-react-utils:test:       
-@openmrs/esm-react-utils:test:       act(() => {
-@openmrs/esm-react-utils:test:         /* fire events that update state */
-@openmrs/esm-react-utils:test:       });
-@openmrs/esm-react-utils:test:       /* assert on the output */
-@openmrs/esm-react-utils:test:       
-@openmrs/esm-react-utils:test:       This ensures that you're testing the behavior the user would see in the browser. Learn more at https://reactjs.org/link/wrap-tests-with-act
-@openmrs/esm-react-utils:test:           at RenderConfig (/home/brandon/Code/pih/mf/openmrs-esm-core/packages/framework/esm-react-utils/src/useConfig.test.tsx:18:29)
-@openmrs/esm-react-utils:test:           at Suspense
-@openmrs/esm-react-utils:test: 
-@openmrs/esm-react-utils:test:       85 |     return store?.subscribe((state) => {
-@openmrs/esm-react-utils:test:       86 |       if (state.loaded && state.config) {
-@openmrs/esm-react-utils:test:     > 87 |         setState(state.config);
-@openmrs/esm-react-utils:test:          |         ^
-@openmrs/esm-react-utils:test:       88 |       }
-@openmrs/esm-react-utils:test:       89 |     });
-@openmrs/esm-react-utils:test:       90 |   }, [store]);
-@openmrs/esm-react-utils:test: 
-@openmrs/esm-react-utils:test:       at printWarning (../../../node_modules/react-dom/cjs/react-dom.development.js:86:30)
-@openmrs/esm-react-utils:test:       at error (../../../node_modules/react-dom/cjs/react-dom.development.js:60:7)
-@openmrs/esm-react-utils:test:       at warnIfUpdatesNotWrappedWithActDEV (../../../node_modules/react-dom/cjs/react-dom.development.js:27543:9)
-@openmrs/esm-react-utils:test:       at scheduleUpdateOnFiber (../../../node_modules/react-dom/cjs/react-dom.development.js:25404:5)
-@openmrs/esm-react-utils:test:       at dispatchSetState (../../../node_modules/react-dom/cjs/react-dom.development.js:17389:16)
-@openmrs/esm-react-utils:test:       at Array.setState (src/useConfig.ts:87:9)
-@openmrs/esm-react-utils:test:       at Object.e (../../../node_modules/unistore/src/index.js:16:19)
-@openmrs/esm-react-utils:test: 
-@openmrs/esm-react-utils:test: PASS src/ConfigurableLink.test.tsx (17.386 s)
-@openmrs/esm-react-utils:test: 
-@openmrs/esm-react-utils:test: Test Suites: 5 passed, 5 total
-@openmrs/esm-react-utils:test: Tests:       19 passed, 19 total
-@openmrs/esm-react-utils:test: Snapshots:   0 total
-@openmrs/esm-react-utils:test: Time:        27.609 s
-@openmrs/esm-react-utils:test: Ran all test suites.