@truedat/dq 8.3.2 → 8.3.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@truedat/dq",
-  "version": "8.3.2",
+  "version": "8.3.3",
   "description": "Truedat Web Data Quality Module",
   "sideEffects": false,
   "module": "src/index.js",
@@ -53,7 +53,7 @@
     "@testing-library/jest-dom": "^6.6.3",
     "@testing-library/react": "^16.3.0",
     "@testing-library/user-event": "^14.6.1",
-    "@truedat/test": "8.3.2",
+    "@truedat/test": "8.3.3",
     "identity-obj-proxy": "^3.0.0",
     "jest": "^29.7.0",
     "redux-saga-test-plan": "^4.0.6"
@@ -86,5 +86,5 @@
     "semantic-ui-react": "^3.0.0-beta.2",
     "swr": "^2.3.3"
   },
-  "gitHead": "ed0c289882710d6df3483692fab5833747130cbf"
+  "gitHead": "4659d78d53d18027bcfb8f8cdb221378e78d7ef6"
 }

package/src/components/Subscription.js

@@ -105,6 +105,7 @@ export const Subscription = ({
 }) => {
   const [subscriptionField, setSubscription] = useState(searchSubscription);
   const [resourceTypeId, setResourceTypeId] = useState();
+  const [isOpen, setIsOpen] = useState(false);
 
   useEffect(() => {
     setSubscription(searchSubscription);
@@ -143,6 +144,7 @@ export const Subscription = ({
     _.prop("id")(subscriptionField)
       ? updateSubscription(payload)
       : createSubscription(payload);
+    setIsOpen(false);
   };
 
   const resourceName = (resource, resourceType) => {
@@ -182,7 +184,10 @@ export const Subscription = ({
       },
       ...resourceTypeId,
     };
-    _.prop("id")(subscriptionField) && deleteSubscription(payload);
+    if (_.prop("id")(subscriptionField)) {
+      deleteSubscription(payload);
+      setIsOpen(false);
+    }
   };
 
   const handleChange = (e, { name, value }) => {
@@ -200,6 +205,9 @@ export const Subscription = ({
       position="bottom right"
       basic
       flowing
+      open={isOpen}
+      onOpen={() => setIsOpen(true)}
+      onClose={() => setIsOpen(false)}
       trigger={
         <Button
           basic

package/src/components/__tests__/Subscription.spec.js

@@ -64,11 +64,18 @@ describe("<Subscription />", () => {
   };
   const subscriptionUpdating = false;
 
+  const createSubscription = jest.fn();
+  const updateSubscription = jest.fn();
+  const deleteSubscription = jest.fn();
+
   const props = {
     resource: rule,
     resourceType: "rule",
     searchSubscription,
     subscriptionUpdating,
+    createSubscription,
+    updateSubscription,
+    deleteSubscription,
   };
 
   it("matches the latest snapshot", async () => {
@@ -76,4 +83,23 @@ describe("<Subscription />", () => {
     await waitForLoad(rendered);
     expect(rendered.container).toMatchSnapshot();
   });
+
+  it("closes the popup after saving the subscription", async () => {
+    const rendered = render(<Subscription {...props} />);
+    await waitForLoad(rendered);
+
+    const user = userEvent.setup({ delay: null });
+
+    // abre el popup haciendo click en el botón de suscripción
+    await user.click(rendered.getByRole("button"));
+
+    // click en guardar
+    const saveButton = await rendered.findByText(/actions.save/i);
+    await user.click(saveButton);
+
+    // el popup se cierra (contenido ya no está en el DOM)
+    await waitFor(() =>
+      expect(rendered.queryByText(/actions.save/i)).not.toBeInTheDocument()
+    );
+  });
 });