@reltio/components 1.4.2070 → 1.4.2071

package/FacetViewHeader/FacetViewHeader.js

@@ -5,5 +5,5 @@ import { useStyles } from './styles';
 export var FacetViewHeader = function (_a) {
     var className = _a.className, title = _a.title, children = _a.children;
     var styles = useStyles();
-    return (React.createElement(BasicViewHeader, { title: title, classes: { root: classnames(styles.container, className), title: styles.title } }, children));
+    return (React.createElement(BasicViewHeader, { "data-reltio-id": "facet-view-header", title: title, classes: { root: classnames(styles.container, className), title: styles.title } }, children));
 };

package/ImageAttributesLine/imageAttributeValues.test-data.js

@@ -46,6 +46,14 @@ export var imageAttributeValues = [
                     value: '/test.jpg',
                     uri: 'entities/1Rd2I9Mt/attributes/ImageGallery/39V5GWJZv/CdnUrlThumbnail/39V5GWj9T'
                 }
+            ],
+            UrlPreview: [
+                {
+                    type: 'configuration/entityTypes/Series/attributes/ImageGallery/attributes/UrlPreview',
+                    ov: true,
+                    value: '/test.jpg',
+                    uri: 'entities/1Rd2I9Mt/attributes/ImageGallery/39V5GWJZv/UrlPreview/38Man0dAs'
+                }
             ]
         },
         ov: true,

package/ImageAttributesLine/index.d.ts

@@ -2,4 +2,4 @@ export { ImageAttributesLine } from './ImageAttributesLine';
 export type { RenderImage, ImageRenderProps } from './types';
 export { DEFAULT_IMAGE_SIZE } from './constants';
 export { downloadImagesByURLs } from './helpers';
-export { imageAttributeValues } from './imageAttributeValues.test-data';
+export { imageAttributeValues, attributeType } from './imageAttributeValues.test-data';

package/ImageAttributesLine/index.js

@@ -1,4 +1,4 @@
 export { ImageAttributesLine } from './ImageAttributesLine';
 export { DEFAULT_IMAGE_SIZE } from './constants';
 export { downloadImagesByURLs } from './helpers';
-export { imageAttributeValues } from './imageAttributeValues.test-data';
+export { imageAttributeValues, attributeType } from './imageAttributeValues.test-data';

package/ImageDetailsView/components/ImageDetailsViewCarousel/ImageDetailsViewCarousel.js

@@ -21,7 +21,7 @@ export var ImageDetailsViewCarousel = function (_a) {
     return (React.createElement("div", { className: classnames(styles.root, className) },
         React.createElement("div", { className: styles.main },
             attributeValueIndex > 0 && (React.createElement(SmallIconButton, { "data-reltio-id": "prev-image-button", icon: NavigateBeforeIcon, className: classnames(styles.arrowButton, styles.arrowButtonPrev), onClick: onPrevImageButtonClick, size: "XL" })),
-            React.createElement(Image, { containerClassName: styles.mainImageContainer, className: styles.mainImage, size: mainImageSrc && ImageSizesStyles.OVERRIDE, src: mainImageSrc }),
+            React.createElement(Image, { "data-reltio-id": "main-image", containerClassName: styles.mainImageContainer, className: styles.mainImage, size: mainImageSrc && ImageSizesStyles.OVERRIDE, src: mainImageSrc }),
             attributeValueIndex !== attributeValues.length - 1 && (React.createElement(SmallIconButton, { "data-reltio-id": "next-image-button", icon: NavigateNextIcon, className: classnames(styles.arrowButton, styles.arrowButtonNext), onClick: onNextImageButtonClick, size: "XL" }))),
         React.createElement(Thumbnails, { attributeValues: attributeValues, attributeValueIndex: attributeValueIndex, setAttributeValueIndex: setAttributeValueIndex })));
 };

package/MergeButton/MergeButton.spec.js

@@ -55,27 +55,32 @@ var __spreadArray = (this && this.__spreadArray) || function (to, from, pack) {
     return to.concat(ar || Array.prototype.slice.call(from));
 };
 import React from 'react';
-import { mount } from 'enzyme';
-import { act } from 'react-dom/test-utils';
+import { render, screen, within } from '@testing-library/react';
+import userEvent from '@testing-library/user-event';
 import { mergeCustomerTenantEntityWithMixedEntities, mergeDataTenantEntity } from '@reltio/mdm-sdk';
+import { MdmModuleProvider } from '../contexts/MdmModuleContext';
 import { MergeButton } from './MergeButton';
-import MergeIcon from '../icons/MergeDark';
-import { ActionButton } from '../ActionButton';
 import { ActionButtonMode } from '../types';
-import { MdmModuleProvider } from '../contexts/MdmModuleContext';
 jest.mock('@reltio/mdm-sdk', function () { return (__assign(__assign({}, jest.requireActual('@reltio/mdm-sdk')), { mergeCustomerTenantEntityWithMixedEntities: jest.fn(), mergeDataTenantEntity: jest.fn() })); });
-var mdmValues = {
-    tenant: { id: 'test', name: 'test' },
-    dtssPath: 'dtssPath/'
-};
-var updateSearchNavigationDataOnMerge = jest.fn();
+var mdmValues = { tenant: { id: 'test', name: 'test' }, dtssPath: 'dtssPath/' };
+var mdmActions = { updateSearchNavigationDataOnMerge: jest.fn() };
+var defaultEntity = { uri: 'entities/123', type: 'entityType' };
+var defaultEntities = [
+    { uri: 'entities/345', type: 'entityType' },
+    { uri: 'entities/567', type: 'entityType' }
+];
+var defaultProps = { entity: defaultEntity, entities: defaultEntities };
 var setUp = function (props) {
-    return mount(React.createElement(MdmModuleProvider, { values: mdmValues, actions: { updateSearchNavigationDataOnMerge: updateSearchNavigationDataOnMerge } },
-        React.createElement(MergeButton, __assign({}, props))));
+    if (props === void 0) { props = defaultProps; }
+    var user = userEvent.setup();
+    var Providers = function (_a) {
+        var children = _a.children;
+        return (React.createElement(MdmModuleProvider, { values: mdmValues, actions: mdmActions }, children));
+    };
+    return __assign(__assign({}, render(React.createElement(MergeButton, __assign({}, props)), { wrapper: Providers })), { user: user });
 };
-describe('MergeButton component test', function () {
+describe('Merge button tests', function () {
     beforeEach(function () {
-        jest.clearAllMocks();
         mergeCustomerTenantEntityWithMixedEntities.mockImplementation(function () {
             return Promise.all([
                 Promise.resolve(),
@@ -83,89 +88,51 @@ describe('MergeButton component test', function () {
                 Promise.resolve({ uris: ['entities/1234'] })
             ]);
         });
-        mergeDataTenantEntity.mockImplementation(function () { return Promise.resolve({ status: 'success' }); });
+        mergeDataTenantEntity.mockResolvedValue({ status: 'success' });
     });
     afterEach(function () {
         jest.clearAllMocks();
     });
-    var entity = { uri: 'entities/123', type: 'entityType' };
-    var entities = [
-        {
-            uri: 'entities/345',
-            type: 'entityType'
-        },
-        {
-            uri: 'entities/567',
-            type: 'entityType'
-        }
-    ];
     it('should render initial state correctly', function () {
-        var mode = ActionButtonMode.button;
-        var wrapper = setUp({ entity: entity, entities: entities, mode: mode });
-        var button = wrapper.find(ActionButton);
-        expect(button).toHaveLength(1);
-        expect(button.props()).toMatchObject({
-            disabled: false,
-            mode: mode,
-            label: 'Merge',
-            icon: MergeIcon
-        });
+        var props = __assign(__assign({}, defaultProps), { mode: ActionButtonMode.button });
+        setUp(props);
+        var button = screen.getByRole('button');
+        expect(button).not.toBeDisabled();
+        within(button).getByTestId('SvgMergeDark');
+        within(button).getByText('Merge');
     });
     it('should merge entities with customer tenant entity on button click', function () { return __awaiter(void 0, void 0, void 0, function () {
-        var wrapper, button;
+        var user;
         return __generator(this, function (_a) {
             switch (_a.label) {
                 case 0:
-                    wrapper = setUp({ entity: entity, entities: entities });
-                    button = wrapper.find(ActionButton);
-                    expect(button).toHaveLength(1);
-                    return [4 /*yield*/, act(function () { return __awaiter(void 0, void 0, void 0, function () {
-                            return __generator(this, function (_a) {
-                                button.prop('onClick')();
-                                return [2 /*return*/];
-                            });
-                        }); })];
+                    user = setUp().user;
+                    return [4 /*yield*/, user.click(screen.getByRole('button'))];
                 case 1:
                     _a.sent();
                     expect(mergeCustomerTenantEntityWithMixedEntities).toHaveBeenCalledWith({
-                        dtssPath: 'dtssPath/',
                         customerTenant: 'test',
-                        entity: entity,
-                        entities: entities
+                        dtssPath: 'dtssPath/',
+                        entities: defaultEntities,
+                        entity: defaultEntity
                     });
                     return [2 /*return*/];
             }
         });
     }); });
     it('should merge entities with data tenant entity on button click', function () { return __awaiter(void 0, void 0, void 0, function () {
-        var entity, entities, wrapper, button;
+        var entity, entities, props, user;
         return __generator(this, function (_a) {
             switch (_a.label) {
                 case 0:
-                    entity = {
-                        uri: 'entities/dt123',
-                        type: 'entityType',
-                        dataTenant: 'dt1'
-                    };
+                    entity = { uri: 'entities/dt123', type: 'entityType', dataTenant: 'dt1' };
                     entities = [
-                        {
-                            uri: 'entities/ct345',
-                            type: 'entityType'
-                        },
-                        {
-                            uri: 'entities/ct567',
-                            type: 'entityType'
-                        }
+                        { uri: 'entities/ct345', type: 'entityType' },
+                        { uri: 'entities/ct567', type: 'entityType' }
                     ];
-                    wrapper = setUp({ entity: entity, entities: entities });
-                    button = wrapper.find(ActionButton);
-                    expect(button).toHaveLength(1);
-                    return [4 /*yield*/, act(function () { return __awaiter(void 0, void 0, void 0, function () {
-                            return __generator(this, function (_a) {
-                                button.prop('onClick')();
-                                return [2 /*return*/];
-                            });
-                        }); })];
+                    props = { entity: entity, entities: entities };
+                    user = setUp(props).user;
+                    return [4 /*yield*/, user.click(screen.getByRole('button'))];
                 case 1:
                     _a.sent();
                     expect(mergeDataTenantEntity).toHaveBeenCalledWith({
@@ -179,70 +146,39 @@ describe('MergeButton component test', function () {
         });
     }); });
     it('should call props.onError on merge error', function () { return __awaiter(void 0, void 0, void 0, function () {
-        var entity, entities, error, onError, wrapper, button;
+        var entity, entities, props, user;
         return __generator(this, function (_a) {
             switch (_a.label) {
                 case 0:
-                    entity = {
-                        uri: 'entities/dt123',
-                        type: 'entityType',
-                        dataTenant: 'dt1'
-                    };
+                    mergeDataTenantEntity.mockRejectedValue({ errorMessage: 'merge failed' });
+                    entity = { uri: 'entities/dt123', type: 'entityType', dataTenant: 'dt1' };
                     entities = [
-                        {
-                            uri: 'entities/ct345',
-                            type: 'entityType'
-                        },
-                        {
-                            uri: 'entities/ct567',
-                            type: 'entityType'
-                        }
+                        { uri: 'entities/ct345', type: 'entityType' },
+                        { uri: 'entities/ct567', type: 'entityType' }
                     ];
-                    error = { errorMessage: 'merge failed' };
-                    mergeDataTenantEntity.mockImplementation(function () { return Promise.reject(error); });
-                    onError = jest.fn();
-                    wrapper = setUp({ entity: entity, entities: entities, onError: onError });
-                    button = wrapper.find(ActionButton);
-                    expect(button).toHaveLength(1);
-                    return [4 /*yield*/, act(function () { return __awaiter(void 0, void 0, void 0, function () {
-                            return __generator(this, function (_a) {
-                                button.prop('onClick')();
-                                return [2 /*return*/];
-                            });
-                        }); })];
+                    props = { entity: entity, entities: entities, onError: jest.fn() };
+                    user = setUp(props).user;
+                    return [4 /*yield*/, user.click(screen.getByRole('button'))];
                 case 1:
                     _a.sent();
-                    expect(onError).toHaveBeenCalledWith(error);
+                    expect(props.onError).toHaveBeenCalledWith({ errorMessage: 'merge failed' });
                     return [2 /*return*/];
             }
         });
     }); });
     it('should call props.onSuccess on merge success', function () { return __awaiter(void 0, void 0, void 0, function () {
-        var onSuccess, otherEntities, wrapper, button;
+        var entities, props, user;
         return __generator(this, function (_a) {
             switch (_a.label) {
                 case 0:
-                    onSuccess = jest.fn();
-                    otherEntities = __spreadArray(__spreadArray([], entities, true), [
-                        {
-                            uri: 'entities/1234',
-                            type: 'entityType',
-                            dataTenant: 'dt1'
-                        }
-                    ], false);
-                    wrapper = setUp({ entity: entity, entities: otherEntities, onSuccess: onSuccess });
-                    button = wrapper.find(ActionButton);
-                    expect(button).toHaveLength(1);
-                    return [4 /*yield*/, act(function () { return __awaiter(void 0, void 0, void 0, function () {
-                            return __generator(this, function (_a) {
-                                button.prop('onClick')();
-                                return [2 /*return*/];
-                            });
-                        }); })];
+                    entities = __spreadArray(__spreadArray([], defaultEntities, true), [{ uri: 'entities/1234', type: 'entityType', dataTenant: 'dt1' }], false);
+                    props = { entity: defaultEntity, entities: entities, onSuccess: jest.fn() };
+                    user = setUp(props).user;
+                    return [4 /*yield*/, user.click(screen.getByRole('button'))];
                 case 1:
                     _a.sent();
-                    expect(onSuccess).toHaveBeenCalled();
-                    expect(updateSearchNavigationDataOnMerge).toHaveBeenCalledWith({
+                    expect(props.onSuccess).toHaveBeenCalled();
+                    expect(mdmActions.updateSearchNavigationDataOnMerge).toHaveBeenCalledWith({
                         winnerUri: 'entities/123',
                         losersUris: ['entities/345', 'entities/567']
                     });
@@ -250,9 +186,4 @@ describe('MergeButton component test', function () {
             }
         });
     }); });
-    it('button should have right label', function () {
-        var wrapper = setUp({ entity: entity, entities: entities });
-        var button = wrapper.find(ActionButton);
-        expect(button.prop('label')).toBe('Merge');
-    });
 });

package/MetadataTypesSelector/MetadataTypesSelector.js

@@ -73,7 +73,7 @@ export var MetadataTypesSelector = memo(function (_a) {
     var noTypesSelected = selectedItemsUris.length === 0;
     return (React.createElement(React.Fragment, null,
         React.createElement(TextField, { label: label, ref: rootRef, InputProps: {
-                startAdornment: noTypesSelected && !placeholder ? null : (React.createElement("div", { className: classnames(styles.inputText, classes.inputText) }, noTypesSelected ? (React.createElement("div", null, placeholder)) : (React.createElement(ListLabel, __assign({}, ListLabelProps, { list: selectedLabels }))))),
+                startAdornment: noTypesSelected && !placeholder ? null : (React.createElement("div", { "data-reltio-id": "metadata-types-selector-start-adornment", className: classnames(styles.inputText, classes.inputText) }, noTypesSelected ? (React.createElement("div", null, placeholder)) : (React.createElement(ListLabel, __assign({}, ListLabelProps, { list: selectedLabels }))))),
                 classes: {
                     root: styles.inputRoot,
                     input: classnames(styles.input, classes.input, (_b = {},