@truedat/dq 4.48.3 → 4.48.6

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## [4.48.6] 2022-07-15
+
+### Fixed
+
+- [TD-4576] Send business_concept_id in concept rules tab rule creation form
+
 ## [4.48.2] 2022-07-08
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@truedat/dq",
-  "version": "4.48.3",
+  "version": "4.48.6",
   "description": "Truedat Web Data Quality Module",
   "sideEffects": false,
   "jsnext:main": "src/index.js",
@@ -88,8 +88,8 @@
   },
   "dependencies": {
     "@apollo/client": "^3.6.4",
-    "@truedat/core": "4.48.3",
-    "@truedat/df": "4.48.3",
+    "@truedat/core": "4.48.6",
+    "@truedat/df": "4.48.6",
     "axios": "^0.19.2",
     "graphql": "^15.5.3",
     "path-to-regexp": "^1.7.0",
@@ -110,5 +110,5 @@
     "react-dom": ">= 16.8.6 < 17",
     "semantic-ui-react": ">= 0.88.2 < 2.1"
   },
-  "gitHead": "5b30312f30c532988985ab9271d1e39ad0ad5c43"
+  "gitHead": "06ed478bb99a0fb6ab5f5cc557808f8ee00d5e24"
 }

package/src/components/RuleForm.js

@@ -148,10 +148,29 @@ export class RuleForm extends React.Component {
     e.preventDefault();
     if (this.messagesInformed(this.generateValidationMessages())) return;
     const { onSubmit, match } = this.props;
-    const { concept, ...rule } = this.state;
+
+    const {
+      concept,
+      domain_id,
+      active,
+      description,
+      df_content,
+      df_name,
+      name,
+    } = this.state;
 
     onSubmit({
-      rule,
+      rule: {
+        ...(concept?.business_concept_id && {
+          business_concept_id: concept?.business_concept_id,
+        }),
+        domain_id,
+        active,
+        description,
+        df_content,
+        df_name,
+        name,
+      },
       ids: match.params,
     });
   };
@@ -204,7 +223,7 @@ export class RuleForm extends React.Component {
       isSubmitting,
       intl: { formatMessage },
     } = this.props;
-    const { template, valid, ...rule } = this.state;
+    const { valid, ...rule } = this.state;
     const { domain_id } = rule;
     const { concept, domainOptions } = this.props;
     const messages = this.generateValidationMessages();

package/src/components/__test_samples__/RuleForm.js

@@ -0,0 +1,118 @@
+export const globalState = {
+  ruleCreating: false,
+  ruleUpdating: false,
+  conceptRulesActions: {
+    domain_ids: [
+      {
+        external_id: "ARQUITECTURA",
+        id: 1,
+        name: "ARQ",
+        parent_id: null,
+        updated_at: "2020-04-01 11:30:42.177807Z",
+      },
+    ],
+  },
+  concept: {
+    id: 12157,
+    content: {
+      CHECKBOXES: "",
+      dependiente: "v4",
+    },
+    confidential: false,
+    description: {
+      document: {
+        data: {},
+        nodes: [
+          {
+            data: {},
+            nodes: [
+              {
+                marks: [],
+                object: "text",
+                text: "a",
+              },
+            ],
+            object: "block",
+            type: "paragraph",
+          },
+        ],
+        object: "document",
+      },
+      object: "value",
+    },
+    domain: {
+      id: 1,
+      name: "ARQ",
+    },
+    domain_parents: [
+      {
+        external_id: "ARQUITECTURA",
+        id: 1,
+        name: "ARQ",
+      },
+      {
+        external_id: "Bankia Prueba",
+        id: 191,
+        name: "Bankia Prueba",
+      },
+    ],
+    name: "abc-def",
+    business_concept_id: 11960,
+    status: "published",
+    type: "test_checkboxes",
+    version: 1,
+    template: {
+      content: [
+        {
+          fields: [
+            {
+              cardinality: "?",
+              default: "",
+              label: "dependiente",
+              name: "dependiente",
+              type: "string",
+              values: {
+                fixed: ["v1", "v2", "v3", "v4"],
+              },
+              widget: "radio",
+            },
+            {
+              cardinality: "*",
+              default: "",
+              label: "CHECKBOXES",
+              name: "CHECKBOXES",
+              type: "string",
+              values: {
+                switch: {
+                  on: "dependiente",
+                  values: {
+                    v1: ["v11", "v22"],
+                    v2: ["c2a", "c2b"],
+                    v3: ["v3x", "v3y"],
+                  },
+                },
+              },
+              widget: "checkbox",
+            },
+          ],
+          name: "New Group 1",
+        },
+      ],
+      label: "test_checkboxes",
+    },
+    last_change_at: "2021-12-28T09:35:25.624552Z",
+    last_change_by: 29,
+    last_change_user: {
+      email: "user@domain.tld",
+      external_id: "",
+      full_name: "Full Name",
+      id: 29,
+      user_name: "user@domain.tld",
+    },
+    completeness: 50,
+    in_progress: false,
+    link_count: 0,
+    rule_count: 11,
+    concept_count: 4,
+  },
+};

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

@@ -4,6 +4,7 @@ import { render } from "@truedat/test/render";
 import userEvent from "@testing-library/user-event";
 import { multipleTemplatesMock } from "@truedat/test/mocks";
 import RuleForm from "../RuleForm";
+import { globalState } from "../__test_samples__/RuleForm";
 
 const rule = {
   id: 123,
@@ -62,4 +63,41 @@ describe("<RuleForm />", () => {
       expect(getByRole("button", { name: /save/i })).not.toBeDisabled();
     });
   });
+
+  it("submit rule data", async () => {
+    const onSubmit = jest.fn();
+
+    const renderOpts = {
+      mocks: [multipleTemplatesMock({ scope: "dq", domainIds: [1] })],
+      state: globalState,
+      fallback: "lazy",
+    };
+
+    const { getByRole } = render(<RuleForm onSubmit={onSubmit} />, renderOpts);
+    await waitFor(() =>
+      expect(getByRole("button", { name: /save/i })).toBeDisabled()
+    );
+    userEvent.click(getByRole("option", { name: "ARQ" }));
+    const ruleNameTextbox = getByRole("textbox", { name: /rule name/i });
+    userEvent.paste(ruleNameTextbox, "rule1");
+    expect(ruleNameTextbox).toHaveValue("rule1");
+    await waitFor(() => {
+      expect(getByRole("button", { name: /save/i })).not.toBeDisabled();
+    });
+
+    userEvent.click(await getByRole("button", { name: /save/i }));
+
+    expect(onSubmit).toHaveBeenCalledWith({
+      ids: {},
+      rule: {
+        active: true,
+        business_concept_id: 11960,
+        description: {},
+        df_content: {},
+        df_name: undefined,
+        domain_id: 1,
+        name: "rule1",
+      },
+    });
+  });
 });

package/src/components/ruleImplementationForm/__tests__/DataSetForm.spec.js

@@ -1,4 +1,4 @@
-import React, { Suspense } from "react";
+import React from "react";
 import { render } from "@truedat/test/render";
 import { waitFor } from "@testing-library/react";
 import { REFERENCE_DATASETS_HEADERS_QUERY } from "@truedat/dd/api/queries";
@@ -23,7 +23,11 @@ const referenceDatasetsMock = {
 };
 const messages = { en };
 
-const renderOpts = { mocks: [referenceDatasetsMock], messages };
+const renderOpts = {
+  mocks: [referenceDatasetsMock],
+  messages,
+  fallback: "lazy",
+};
 
 describe("<DatasetForm />", () => {
   const setStructures = jest.fn();
@@ -31,7 +35,7 @@ describe("<DatasetForm />", () => {
   const selector = -1;
   const setImplementationKey = jest.fn();
   const implementationKey = "Impl";
-  const structures = [1, 2];
+  const structures = [{ id: 1 }, { id: 2 }];
 
   const props = {
     implementationKey,
@@ -44,9 +48,7 @@ describe("<DatasetForm />", () => {
 
   it("matches the latest snapshot", async () => {
     const { container, queryByText } = render(
-      <Suspense fallback={null}>
-        <DatasetForm {...props} />
-      </Suspense>,
+      <DatasetForm {...props} />,
       renderOpts
     );