@truedat/dq 4.38.7 → 4.40.0

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## [4.39.0] 2022-03-01
+
+### Changed
+
+- [TD-4528] Restyle quality results table
+
 ## [4.38.7] 2022-02-22
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@truedat/dq",
-  "version": "4.38.7",
+  "version": "4.40.0",
   "description": "Truedat Web Data Quality Module",
   "sideEffects": false,
   "jsnext:main": "src/index.js",
@@ -31,7 +31,7 @@
     "@babel/plugin-transform-modules-commonjs": "^7.15.0",
     "@babel/preset-env": "^7.15.0",
     "@babel/preset-react": "^7.14.5",
-    "@truedat/test": "4.38.7",
+    "@truedat/test": "4.39.0",
     "babel-jest": "^27.0.6",
     "babel-plugin-dynamic-import-node": "^2.3.3",
     "babel-plugin-lodash": "^3.3.4",
@@ -82,8 +82,8 @@
   },
   "dependencies": {
     "@apollo/client": "^3.4.10",
-    "@truedat/core": "4.38.7",
-    "@truedat/df": "4.38.7",
+    "@truedat/core": "4.40.0",
+    "@truedat/df": "4.40.0",
     "axios": "^0.19.2",
     "graphql": "^15.5.3",
     "path-to-regexp": "^1.7.0",
@@ -103,5 +103,5 @@
     "react-dom": ">= 16.8.6 < 17",
     "semantic-ui-react": ">= 0.88.2 < 2.1"
   },
-  "gitHead": "840c799ecdd6b8440112ac803119482e688142ed"
+  "gitHead": "0bd677584da73a5021a5c96f2e5525fde9a1958a"
 }

package/src/components/ExecutionDetails.js

@@ -91,14 +91,7 @@ export const ExecutionDetails = ({ ruleImplementation, ruleResultId }) => {
     ruleImplementation.results
   );
 
-  return _.isEmpty(ruleResult.details) ? (
-    <Message
-      style={{ marginTop: "14px" }}
-      header={formatMessage({
-        id: "rule.ruleImplementation.results.details.empty",
-      })}
-    />
-  ) : (
+  return (
     <Table className="implementation-results medium">
       <Table.Header>
         <Table.Row>
@@ -116,20 +109,23 @@ export const ExecutionDetails = ({ ruleImplementation, ruleResultId }) => {
           ruleResult={ruleResult}
         />
       </Table.Body>
-
-      <Table.Header>
-        <Table.Row>
-          <Table.HeaderCell
-            colSpan={2}
-            content={formatMessage({
-              id: "ruleResult.props.header.details",
-            })}
-          />
-        </Table.Row>
-      </Table.Header>
-      <Table.Body>
-        <DetailRow details={ruleResult.details} />
-      </Table.Body>
+      {!_.isEmpty(ruleResult.details) ? (
+        <>
+          <Table.Header>
+            <Table.Row>
+              <Table.HeaderCell
+                colSpan={2}
+                content={formatMessage({
+                  id: "ruleResult.props.header.details",
+                })}
+              />
+            </Table.Row>
+          </Table.Header>
+          <Table.Body>
+            <DetailRow details={ruleResult?.details} />
+          </Table.Body>
+        </>
+      ) : null}
     </Table>
   );
 };

package/src/components/RuleImplementationResults.js

@@ -35,7 +35,7 @@ export const RuleImplementationResults = ({
 
   return (
     <>
-      {ruleImplementation.event_type === "FAILED" && (
+      {ruleImplementation.event_type === "FAILED" ? (
         <Message negative style={{ marginTop: "14px" }}>
           <Message.Header>
             {formatMessage({
@@ -61,7 +61,7 @@ export const RuleImplementationResults = ({
             {ruleImplementation.event_message}
           </p>
         </Message>
-      )}
+      ) : null}
       {_.isEmpty(ruleResults) ? (
         <Message
           style={{ marginTop: "14px" }}
@@ -70,25 +70,25 @@ export const RuleImplementationResults = ({
           })}
         />
       ) : (
-        <Table className="implementation-results small" selectable>
+        <Table className="implementation-results small">
           <Table.Header>
             <Table.Row>
               <Table.HeaderCell
-                content={formatMessage({ id: "ruleResult.props.quality" })}
+                content={formatMessage({ id: "ruleResult.props.date" })}
               />
               <Table.HeaderCell
-                content={formatMessage({ id: "ruleResult.props.date" })}
+                content={formatMessage({ id: "ruleResult.props.quality" })}
               />
-              {_.includes("records")(optionalColumns) && (
+              {_.includes("records")(optionalColumns) ? (
                 <Table.HeaderCell
                   content={formatMessage({ id: "ruleResult.props.records" })}
                 />
-              )}
-              {_.includes("errors")(optionalColumns) && (
+              ) : null}
+              {_.includes("errors")(optionalColumns) ? (
                 <Table.HeaderCell
                   content={formatMessage({ id: "ruleResult.props.errors" })}
                 />
-              )}
+              ) : null}
               {customColumns.map((column, index) => (
                 <Table.HeaderCell
                   key={index}
@@ -99,7 +99,7 @@ export const RuleImplementationResults = ({
                 />
               ))}
               {<Table.HeaderCell />}
-              {isAdmin && <Table.HeaderCell />}
+              {isAdmin ? <Table.HeaderCell /> : null}
             </Table.Row>
           </Table.Header>
           <Table.Body>

package/src/components/RuleResultRow.js

@@ -3,13 +3,14 @@ import React from "react";
 import PropTypes from "prop-types";
 import { connect } from "react-redux";
 import { useIntl } from "react-intl";
-import { useHistory } from "react-router-dom";
+import { Link } from "react-router-dom";
 import { Table, Icon } from "semantic-ui-react";
 import { ConfirmModal, DateTime } from "@truedat/core/components";
 import { columnDecorator } from "@truedat/core/services";
 import { linkTo } from "@truedat/core/routes";
 import { selectColor } from "../functions/selectors";
 import { deleteRuleResult } from "../routines";
+import "../styles/ruleResultRow.less";
 
 export const RuleResultRow = ({
   deleteRuleResult,
@@ -22,24 +23,25 @@ export const RuleResultRow = ({
 }) => {
   const { formatMessage, formatNumber: _formatNumber } = useIntl();
   const formatNumber = (num) => (_.isNil(num) ? num : _formatNumber(num));
-  const history = useHistory();
 
   return (
-    <Table.Row
-      onClick={() =>
-        !_.isEmpty(ruleResult.details)
-          ? history.push(
-              linkTo.RULE_IMPLEMENTATION_RESULT_DETAILS({
-                id: rule.id,
-                implementation_id: ruleImplementation.id,
-                rule,
-                ruleImplementation,
-                rule_result_id: ruleResult.id,
-              })
-            )
-          : null
-      }
-    >
+    <Table.Row>
+      <Table.Cell>
+        <Link
+          to={linkTo.RULE_IMPLEMENTATION_RESULT_DETAILS({
+            id: rule.id,
+            implementation_id: ruleImplementation.id,
+            rule,
+            ruleImplementation,
+            rule_result_id: ruleResult.id,
+          })}
+        >
+          <DateTime className="rule-result-date" value={ruleResult.date} />
+          {!_.isEmpty(ruleResult.details) ? (
+            <Icon name="info circle" color="blue" />
+          ) : null}
+        </Link>
+      </Table.Cell>
       <Table.Cell>
         <Icon
           name="circle"
@@ -47,28 +49,18 @@ export const RuleResultRow = ({
         />
         {`${parseFloat(ruleResult.result)} %`}
       </Table.Cell>
-      <Table.Cell>
-        <DateTime value={ruleResult.date} />
-      </Table.Cell>
-      {_.includes("records")(optionalColumns) && (
+      {_.includes("records")(optionalColumns) ? (
         <Table.Cell>{formatNumber(ruleResult.records)}</Table.Cell>
-      )}
-      {_.includes("errors")(optionalColumns) && (
+      ) : null}
+      {_.includes("errors")(optionalColumns) ? (
         <Table.Cell>{formatNumber(ruleResult.errors)}</Table.Cell>
-      )}
-      {_.includes("details")(optionalColumns) && (
-        <Table.Cell>
-          {!_.isEmpty(ruleResult.details) && (
-            <Icon name="info circle" color="blue" />
-          )}
-        </Table.Cell>
-      )}
+      ) : null}
       {customColumns?.map((column, index) => (
         <Table.Cell key={index}>
           {columnDecorator(column)(ruleResult)}
         </Table.Cell>
       ))}
-      {isAdmin && (
+      {isAdmin ? (
         <Table.Cell onClick={(e) => e.stopPropagation()}>
           <ConfirmModal
             icon="trash"
@@ -88,7 +80,7 @@ export const RuleResultRow = ({
             }
           />
         </Table.Cell>
-      )}
+      ) : null}
     </Table.Row>
   );
 };

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

@@ -11,8 +11,12 @@ jest.spyOn(React, "useContext").mockImplementation(() => intl);
 describe("<RuleResultRow />", () => {
   it("renders customColumns values when passed", () => {
     const ruleResultRowProps = {
-      id: 10,
-      ruleResult: { result: 80, params: { p1: "1", p2: "2", p3: "no" } },
+      rule: { id: 10 },
+      ruleResult: {
+        id: 54,
+        result: 80,
+        params: { p1: "1", p2: "2", p3: "no" },
+      },
       customColumns: [
         {
           name: "m_parm1",
@@ -24,7 +28,12 @@ describe("<RuleResultRow />", () => {
         },
       ],
       date: "2019-08-12T02:00:00Z",
-      ruleImplementation: { minimum: 1, goal: 10, result_type: "percentage" },
+      ruleImplementation: {
+        id: 23,
+        minimum: 1,
+        goal: 10,
+        result_type: "percentage",
+      },
     };
 
     const wrapper = shallow(<RuleResultRow {...ruleResultRowProps} />);
@@ -50,10 +59,11 @@ describe("<RuleResultRow />", () => {
 
   it("green icon when result is over goal", () => {
     const ruleResultRowProps = {
-      id: 10,
-      ruleResult: { result: 80 },
+      rule: { id: 10 },
+      ruleResult: { id: 54, result: 80 },
       date: "2019-08-12T02:00:00Z",
       ruleImplementation: {
+        id: 23,
         minimum: 1,
         goal: 10,
         result_type: "percentage",
@@ -61,7 +71,7 @@ describe("<RuleResultRow />", () => {
     };
 
     const wrapper = shallow(<RuleResultRow {...ruleResultRowProps} />);
-    const cell = wrapper.find("TableRow").find("TableCell").at(0).dive();
+    const cell = wrapper.find("TableRow").find("TableCell").at(1).dive();
 
     expect(cell.find("td").find("Icon")).toHaveLength(1);
 
@@ -72,10 +82,11 @@ describe("<RuleResultRow />", () => {
 
   it("red icon when result is under minimum", () => {
     const ruleResultRowProps = {
-      id: 10,
-      ruleResult: { result: 80 },
+      rule: { id: 10 },
+      ruleResult: { id: 54, result: 80 },
       date: "2019-08-12T02:00:00Z",
       ruleImplementation: {
+        id: 23,
         minimum: 81,
         goal: 90,
         result_type: "percentage",
@@ -83,7 +94,7 @@ describe("<RuleResultRow />", () => {
     };
 
     const wrapper = shallow(<RuleResultRow {...ruleResultRowProps} />);
-    const cell = wrapper.find("TableRow").find("TableCell").at(0).dive();
+    const cell = wrapper.find("TableRow").find("TableCell").at(1).dive();
 
     expect(cell.find("td").find("Icon")).toHaveLength(1);
 
@@ -94,18 +105,19 @@ describe("<RuleResultRow />", () => {
 
   it("yellow icon when result is between minimum and goal", () => {
     const ruleResultRowProps = {
-      id: 10,
+      rule: { id: 10 },
       ruleImplementation: {
+        id: 23,
         minimum: 81,
         goal: 90,
         result_type: "percentage",
       },
-      ruleResult: { result: 82 },
+      ruleResult: { id: 54, result: 82 },
       date: "2019-08-12T02:00:00Z",
     };
 
     const wrapper = shallow(<RuleResultRow {...ruleResultRowProps} />);
-    const cell = wrapper.find("TableRow").find("TableCell").at(0).dive();
+    const cell = wrapper.find("TableRow").find("TableCell").at(1).dive();
 
     expect(cell.find("td").find("Icon")).toHaveLength(1);
 
@@ -116,13 +128,14 @@ describe("<RuleResultRow />", () => {
 
   it("renders delete cell in RuleResultRow when user isAdmin is true", () => {
     const ruleResultRowProps = {
-      id: 10,
+      rule: { id: 10 },
       ruleImplementation: {
+        id: 23,
         minimum: 81,
         goal: 90,
         result_type: "percentage",
       },
-      ruleResult: { result: 82 },
+      ruleResult: { id: 54, result: 82 },
       date: "2019-08-12T02:00:00Z",
       isAdmin: true,
       deleteRuleResult: jest.fn(),
@@ -138,13 +151,14 @@ describe("<RuleResultRow />", () => {
 
   it("does not render delete cell in RuleResultRow when user isAdmin is false", () => {
     const ruleResultRowProps = {
-      id: 10,
+      rule: { id: 10 },
       ruleImplementation: {
+        id: 23,
         minimum: 81,
         goal: 90,
         result_type: "percentage",
       },
-      ruleResult: { result: 82 },
+      ruleResult: { id: 54, result: 82 },
       date: "2019-08-12T02:00:00Z",
       isAdmin: false,
       deleteRuleResult: jest.fn(),

package/src/components/__tests__/__snapshots__/ExecutionDetails.spec.js.snap

@@ -115,19 +115,81 @@ exports[`<ExecutionDetails> matches the lastest snapshot 1`] = `
 
 exports[`<ExecutionDetails> without details 1`] = `
 <div>
-  <div
-    class="ui message"
-    style="margin-top: 14px;"
+  <table
+    class="ui table implementation-results medium"
   >
-    <div
-      class="content"
+    <thead
+      class=""
     >
-      <div
-        class="header"
+      <tr
+        class=""
       >
-        No details found for current result
-      </div>
-    </div>
-  </div>
+        <th
+          class=""
+          colspan="2"
+        >
+          Information
+        </th>
+      </tr>
+    </thead>
+    <tbody
+      class=""
+    >
+      <tr
+        class=""
+      >
+        <td
+          class=""
+        >
+          Quality
+        </td>
+        <td
+          class=""
+        >
+          <i
+            aria-hidden="true"
+            class="grey circle icon"
+          />
+          NaN %
+        </td>
+      </tr>
+      <tr
+        class=""
+      >
+        <td
+          class=""
+        >
+          Date
+        </td>
+        <td
+          class=""
+        />
+      </tr>
+      <tr
+        class=""
+      >
+        <td
+          class=""
+        >
+          Records
+        </td>
+        <td
+          class=""
+        />
+      </tr>
+      <tr
+        class=""
+      >
+        <td
+          class=""
+        >
+          Errors
+        </td>
+        <td
+          class=""
+        />
+      </tr>
+    </tbody>
+  </table>
 </div>
 `;

package/src/components/__tests__/__snapshots__/RuleImplementationResults.spec.js.snap

@@ -5,7 +5,6 @@ exports[`<RuleImplementationResults /> matches the latest snapshot 1`] = `
   <Table
     as="table"
     className="implementation-results small"
-    selectable={true}
   >
     <TableHeader
       as="thead"
@@ -16,11 +15,11 @@ exports[`<RuleImplementationResults /> matches the latest snapshot 1`] = `
       >
         <TableHeaderCell
           as="th"
-          content="ruleResult.props.quality"
+          content="ruleResult.props.date"
         />
         <TableHeaderCell
           as="th"
-          content="ruleResult.props.date"
+          content="ruleResult.props.quality"
         />
         <TableHeaderCell
           as="th"

package/src/styles/ruleResultRow.less

@@ -0,0 +1,3 @@
+.rule-result-date {
+  margin-right: 5px;
+}