@reltio/components 1.4.2070 → 1.4.2072

package/ReadOnlyAttributesPager/ReadOnlyAttributesPager.test.js

@@ -60,12 +60,16 @@ import userEvent from '@testing-library/user-event';
 import { EntityAttrTypes } from '@reltio/mdm-sdk';
 import ReadOnlyAttributesPager from './ReadOnlyAttributesPager';
 import { MdmModuleProvider } from '../contexts/MdmModuleContext';
+import { FeaturesContext } from '../contexts/FeaturesContext';
+import { CollaborationContext } from '../contexts/CollaborationContext';
 describe('ReadOnlyAttributesPager tests', function () {
+    var collaboration = { commentsMap: {} };
     var attributeType = {
         uri: 'configuration/entityTypes/HCP/attributes/Simple',
         type: 'String',
         name: 'Simple',
-        label: 'Simple label'
+        label: 'Simple label',
+        description: 'Simple description'
     };
     var values = [
         { type: attributeType.uri, uri: 'entities/sIeHttl/attributes/Simple/u123', value: '123', ov: true },
@@ -83,57 +87,108 @@ describe('ReadOnlyAttributesPager tests', function () {
         jest.clearAllMocks();
     });
     var setUp = function (_a) {
-        var _b = _a === void 0 ? {} : _a, _c = _b.props, props = _c === void 0 ? defaultProps : _c, _d = _b.mdmValues, mdmValues = _d === void 0 ? {} : _d;
+        var _b = _a === void 0 ? {} : _a, _c = _b.props, props = _c === void 0 ? defaultProps : _c, _d = _b.mdmValues, mdmValues = _d === void 0 ? {} : _d, _e = _b.featuresContextValue, featuresContextValue = _e === void 0 ? { showDescription: true } : _e;
         var user = userEvent.setup();
         var Providers = function (_a) {
             var children = _a.children;
-            return (React.createElement(MdmModuleProvider, { values: mdmValues, actions: actions }, children));
+            return (React.createElement(MdmModuleProvider, { values: mdmValues, actions: actions },
+                React.createElement(CollaborationContext.Provider, { value: collaboration },
+                    React.createElement(FeaturesContext.Provider, { value: featuresContextValue }, children))));
         };
         return __assign({ user: user }, render(React.createElement(ReadOnlyAttributesPager, __assign({}, props)), { wrapper: Providers }));
     };
     it('should use OneLineRenderer if there is only one ov value of Simple attribute', function () {
         var props = __assign(__assign({}, defaultProps), { values: [values[0]] });
         setUp({ props: props });
-        expect(screen.queryByTestId('branch-decorator')).not.toBeInTheDocument();
         expect(screen.getByTestId('one-line-renderer')).toBeInTheDocument();
+        expect(screen.getByLabelText('Simple description')).toBeInTheDocument();
         expect(screen.getByText('Simple label')).toBeInTheDocument();
+        expect(screen.getByLabelText('Add comment')).toBeInTheDocument();
         expect(screen.getByText('123')).toBeInTheDocument();
-    });
-    it('should use OneLineRenderer if there is only one ov value (and some non ov) of Simple attribute', function () {
-        var props = __assign(__assign({}, defaultProps), { values: [values[0], __assign(__assign({}, values[1]), { ov: false })] });
-        setUp({ props: props });
         expect(screen.queryByTestId('branch-decorator')).not.toBeInTheDocument();
-        expect(screen.getByTestId('one-line-renderer')).toBeInTheDocument();
-        expect(screen.getByText('Simple label')).toBeInTheDocument();
-        expect(screen.getByText('123')).toBeInTheDocument();
-        expect(screen.queryByText('456')).not.toBeInTheDocument();
     });
+    it('should use OneLineRenderer if there is only one ov value (and some non ov) of Simple attribute', function () { return __awaiter(void 0, void 0, void 0, function () {
+        var props, user;
+        return __generator(this, function (_a) {
+            switch (_a.label) {
+                case 0:
+                    props = __assign(__assign({}, defaultProps), { values: [values[0], __assign(__assign({}, values[1]), { ov: false })] });
+                    user = setUp({ props: props }).user;
+                    expect(screen.getByTestId('one-line-renderer')).toBeInTheDocument();
+                    expect(screen.getByLabelText('Simple description')).toBeInTheDocument();
+                    expect(screen.getByText('Simple label')).toBeInTheDocument();
+                    expect(screen.getByLabelText('Add comment')).toBeInTheDocument();
+                    expect(screen.getByText('123')).toBeInTheDocument();
+                    return [4 /*yield*/, user.hover(screen.getByText('+ 1'))];
+                case 1:
+                    _a.sent();
+                    expect(screen.getByText('456')).toBeInTheDocument();
+                    return [2 /*return*/];
+            }
+        });
+    }); });
     it('should use MultiLineRenderer for Complex attribute', function () { return __awaiter(void 0, void 0, void 0, function () {
-        var ovValues, props, user;
+        var ovValues, props, user, labels;
         return __generator(this, function (_a) {
             switch (_a.label) {
                 case 0:
                     ovValues = [
-                        __assign(__assign({}, values[0]), { value: { Name: [{ value: '123', uri: '123' }] } }),
-                        __assign(__assign({}, values[1]), { value: { Name: [{ value: '456', uri: '456' }] } }),
-                        __assign(__assign({}, values[2]), { value: { Name: [{ value: '444', uri: '444' }] } })
+                        __assign(__assign({}, values[0]), { label: 'First value', value: { Name: [{ value: '123', uri: '123' }] } }),
+                        __assign(__assign({}, values[1]), { label: 'Second value', value: { Name: [{ value: '456', uri: '456' }] } }),
+                        __assign(__assign({}, values[2]), { label: 'Third value', value: { Name: [{ value: '444', uri: '444' }] } })
                     ];
                     props = __assign(__assign({}, defaultProps), { attributeType: __assign(__assign({}, defaultProps.attributeType), { type: 'Nested' }), max: 1, paging: { totalOvValues: 5 }, values: __spreadArray(__spreadArray([], ovValues, true), [__assign(__assign({}, values[1]), { ov: false })], false) });
                     user = setUp({ props: props }).user;
                     expect(screen.queryByTestId('branch-decorator')).not.toBeInTheDocument();
                     expect(screen.getByTestId('multi-line-renderer')).toBeInTheDocument();
+                    expect(screen.getByLabelText('Add comment')).toBeInTheDocument();
+                    expect(screen.getByLabelText('Simple description')).toBeInTheDocument();
                     expect(screen.getByText('Simple label')).toBeInTheDocument();
                     expect(screen.getByText('+ 1')).toBeInTheDocument();
-                    return [4 /*yield*/, user.click(screen.getByTestId('show-more'))];
+                    labels = screen.getAllByTestId('reltio-attribute-complex-label');
+                    expect(labels).toHaveLength(1);
+                    expect(labels[0]).toHaveTextContent('First value');
+                    return [4 /*yield*/, user.click(screen.getByText('Show 1 more of remaining 4 value'))];
                 case 1:
                     _a.sent();
-                    expect(actions.requestNextPageOfAttributeValues).toHaveBeenCalledWith({
+                    expect(actions.requestNextPageOfAttributeValues).toHaveBeenLastCalledWith({
                         attributeTypeName: 'Simple',
                         attributeTypeUri: 'configuration/entityTypes/HCP/attributes/Simple',
                         defaultMaxValues: 1,
                         parentUri: 'entities/sIeHttl',
                         values: ovValues
                     });
+                    labels = screen.getAllByTestId('reltio-attribute-complex-label');
+                    expect(labels).toHaveLength(2);
+                    expect(labels[0]).toHaveTextContent('First value');
+                    expect(labels[1]).toHaveTextContent('Second value');
+                    return [4 /*yield*/, user.click(screen.getByText('Show 1 more of remaining 3 value'))];
+                case 2:
+                    _a.sent();
+                    expect(actions.requestNextPageOfAttributeValues).toHaveBeenLastCalledWith({
+                        attributeTypeName: 'Simple',
+                        attributeTypeUri: 'configuration/entityTypes/HCP/attributes/Simple',
+                        defaultMaxValues: 1,
+                        parentUri: 'entities/sIeHttl',
+                        values: ovValues
+                    });
+                    labels = screen.getAllByTestId('reltio-attribute-complex-label');
+                    expect(labels).toHaveLength(3);
+                    expect(labels[0]).toHaveTextContent('First value');
+                    expect(labels[1]).toHaveTextContent('Second value');
+                    expect(labels[2]).toHaveTextContent('Third value');
+                    return [4 /*yield*/, user.click(screen.getByText('Show 1 more of remaining 2 value'))];
+                case 3:
+                    _a.sent();
+                    return [4 /*yield*/, user.click(screen.getByText('Show 1 more of remaining 1 value'))];
+                case 4:
+                    _a.sent();
+                    return [4 /*yield*/, user.click(screen.getByText('Show less'))];
+                case 5:
+                    _a.sent();
+                    labels = screen.getAllByTestId('reltio-attribute-complex-label');
+                    expect(labels).toHaveLength(1);
+                    expect(labels[0]).toHaveTextContent('First value');
                     return [2 /*return*/];
             }
         });
@@ -143,6 +198,8 @@ describe('ReadOnlyAttributesPager tests', function () {
         setUp({ props: props });
         expect(screen.queryByTestId('branch-decorator')).not.toBeInTheDocument();
         expect(screen.getByTestId('multi-line-renderer')).toBeInTheDocument();
+        expect(screen.getByLabelText('Add comment')).toBeInTheDocument();
+        expect(screen.getByLabelText('Simple description')).toBeInTheDocument();
         expect(screen.getByText('Simple label')).toBeInTheDocument();
         expect(screen.getByText('123')).toBeInTheDocument();
         expect(screen.queryByText('+ 1')).not.toBeInTheDocument();

package/ReadOnlyAttributesView/ReadOnlyAttributesView.js

@@ -6,7 +6,7 @@ import Box from '@mui/material/Box';
 import Typography from '@mui/material/Typography';
 import { PivotingAttributeContext } from '../contexts/PivotingAttributeContext';
 import { useMdmMetadata, useMdmPivotingAttributes } from '../contexts/MdmModuleContext';
-import { AlwaysVisibleAttributesContext } from '../contexts/AllwaysVisibleAttributesContext';
+import { AlwaysVisibleAttributesContext } from '../contexts/AlwaysVisibleAttributesContext';
 import { BasicView } from '../BasicView';
 import { FacetViewHeader } from '../FacetViewHeader';
 import { ReadOnlyAttributesList } from '../ReadOnlyAttributesList';