@reltio/components 1.4.2070 → 1.4.2072

package/cjs/ImageDetailsView/components/ImageDetailsViewCarousel/ImageDetailsViewCarousel.js

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

package/cjs/MergeButton/MergeButton.spec.js

@@ -60,27 +60,32 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 var react_1 = __importDefault(require("react"));
-var enzyme_1 = require("enzyme");
-var test_utils_1 = require("react-dom/test-utils");
+var react_2 = require("@testing-library/react");
+var user_event_1 = __importDefault(require("@testing-library/user-event"));
 var mdm_sdk_1 = require("@reltio/mdm-sdk");
+var MdmModuleContext_1 = require("../contexts/MdmModuleContext");
 var MergeButton_1 = require("./MergeButton");
-var MergeDark_1 = __importDefault(require("../icons/MergeDark"));
-var ActionButton_1 = require("../ActionButton");
 var types_1 = require("../types");
-var MdmModuleContext_1 = require("../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 (0, enzyme_1.mount)(react_1.default.createElement(MdmModuleContext_1.MdmModuleProvider, { values: mdmValues, actions: { updateSearchNavigationDataOnMerge: updateSearchNavigationDataOnMerge } },
-        react_1.default.createElement(MergeButton_1.MergeButton, __assign({}, props))));
+    if (props === void 0) { props = defaultProps; }
+    var user = user_event_1.default.setup();
+    var Providers = function (_a) {
+        var children = _a.children;
+        return (react_1.default.createElement(MdmModuleContext_1.MdmModuleProvider, { values: mdmValues, actions: mdmActions }, children));
+    };
+    return __assign(__assign({}, (0, react_2.render)(react_1.default.createElement(MergeButton_1.MergeButton, __assign({}, props)), { wrapper: Providers })), { user: user });
 };
-describe('MergeButton component test', function () {
+describe('Merge button tests', function () {
     beforeEach(function () {
-        jest.clearAllMocks();
         mdm_sdk_1.mergeCustomerTenantEntityWithMixedEntities.mockImplementation(function () {
             return Promise.all([
                 Promise.resolve(),
@@ -88,89 +93,51 @@ describe('MergeButton component test', function () {
                 Promise.resolve({ uris: ['entities/1234'] })
             ]);
         });
-        mdm_sdk_1.mergeDataTenantEntity.mockImplementation(function () { return Promise.resolve({ status: 'success' }); });
+        mdm_sdk_1.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 = types_1.ActionButtonMode.button;
-        var wrapper = setUp({ entity: entity, entities: entities, mode: mode });
-        var button = wrapper.find(ActionButton_1.ActionButton);
-        expect(button).toHaveLength(1);
-        expect(button.props()).toMatchObject({
-            disabled: false,
-            mode: mode,
-            label: 'Merge',
-            icon: MergeDark_1.default
-        });
+        var props = __assign(__assign({}, defaultProps), { mode: types_1.ActionButtonMode.button });
+        setUp(props);
+        var button = react_2.screen.getByRole('button');
+        expect(button).not.toBeDisabled();
+        (0, react_2.within)(button).getByTestId('SvgMergeDark');
+        (0, react_2.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_1.ActionButton);
-                    expect(button).toHaveLength(1);
-                    return [4 /*yield*/, (0, test_utils_1.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(react_2.screen.getByRole('button'))];
                 case 1:
                     _a.sent();
                     expect(mdm_sdk_1.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_1.ActionButton);
-                    expect(button).toHaveLength(1);
-                    return [4 /*yield*/, (0, test_utils_1.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(react_2.screen.getByRole('button'))];
                 case 1:
                     _a.sent();
                     expect(mdm_sdk_1.mergeDataTenantEntity).toHaveBeenCalledWith({
@@ -184,70 +151,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'
-                    };
+                    mdm_sdk_1.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' };
-                    mdm_sdk_1.mergeDataTenantEntity.mockImplementation(function () { return Promise.reject(error); });
-                    onError = jest.fn();
-                    wrapper = setUp({ entity: entity, entities: entities, onError: onError });
-                    button = wrapper.find(ActionButton_1.ActionButton);
-                    expect(button).toHaveLength(1);
-                    return [4 /*yield*/, (0, test_utils_1.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(react_2.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_1.ActionButton);
-                    expect(button).toHaveLength(1);
-                    return [4 /*yield*/, (0, test_utils_1.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(react_2.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']
                     });
@@ -255,9 +191,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_1.ActionButton);
-        expect(button.prop('label')).toBe('Merge');
-    });
 });

package/cjs/MetadataTypesSelector/MetadataTypesSelector.d.ts

@@ -8,8 +8,9 @@ type Classes = {
     root?: string;
     label?: string;
 };
+type Item = TEntityType | TRelationType;
 type Props = {
-    items: (TEntityType | TRelationType)[];
+    items: Item[];
     selectedItemsUris: string[];
     onChange: (values: string[]) => void;
     label: string;

package/cjs/MetadataTypesSelector/MetadataTypesSelector.js

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