@reltio/components 1.4.1945 → 1.4.1946

package/esm/DependentLookupEditor/DependentLookupEditor.test.js

@@ -49,7 +49,6 @@ import React from 'react';
 import { mount, ReactWrapper, shallow } from 'enzyme';
 import { act } from 'react-dom/test-utils';
 import { debounce } from '@reltio/mdm-sdk';
-import { delayPromise } from '../test-utils';
 import { ReactSelect } from '../ReactSelect';
 import { DependentLookupAutopopulationContext } from '../contexts/DependentLookupAutopopulationContext';
 import { ReactSelectOptionWithCheckIcon } from '../ReactSelectOptionWithCheckIcon';
@@ -129,7 +128,7 @@ describe('DependentLookupEditor', function () {
             wrapper.update();
             expect(wrapper.find(ReactSelect).find('Input').last().prop('value')).toBe(newInputValue);
             expect(props.onChange).not.toHaveBeenCalled();
-            expect(props.getLookups).toBeCalledWith({
+            expect(props.getLookups).toHaveBeenCalledWith({
                 type: 'COUNTRY_CD',
                 parents: [],
                 displayNamePrefix: newInputValue,
@@ -142,7 +141,7 @@ describe('DependentLookupEditor', function () {
             wrapper.update();
             expect(wrapper.find(ReactSelect).find('Input').last().prop('value')).toBe('');
             expect(props.onChange).not.toHaveBeenCalled();
-            expect(props.getLookups).toBeCalledWith({
+            expect(props.getLookups).toHaveBeenCalledWith({
                 type: 'COUNTRY_CD',
                 parents: [],
                 displayNamePrefix: '',
@@ -188,7 +187,7 @@ describe('DependentLookupEditor', function () {
             wrapper.update();
             expect(wrapper.find(ReactSelect).find('Input').last().prop('value')).toBe(newInputValue);
             expect(props.onChange).not.toHaveBeenCalled();
-            expect(props.getLookups).toBeCalledWith({
+            expect(props.getLookups).toHaveBeenCalledWith({
                 type: 'COUNTRY_CD',
                 parents: [],
                 displayNamePrefix: newInputValue,
@@ -201,7 +200,7 @@ describe('DependentLookupEditor', function () {
             wrapper.update();
             expect(wrapper.find(ReactSelect).find('Input').last().prop('value')).toBe(getOptionLabel(value));
             expect(props.onChange).not.toHaveBeenCalled();
-            expect(props.getLookups).toBeCalledWith({
+            expect(props.getLookups).toHaveBeenCalledWith({
                 type: 'COUNTRY_CD',
                 parents: [],
                 displayNamePrefix: '',
@@ -223,7 +222,7 @@ describe('DependentLookupEditor', function () {
             wrapper.update();
             expect(wrapper.find(ReactSelect).find('Input').last().prop('value')).toBe(newInputValue);
             expect(props.onChange).not.toHaveBeenCalled();
-            expect(props.getLookups).toBeCalledWith({
+            expect(props.getLookups).toHaveBeenCalledWith({
                 type: 'COUNTRY_CD',
                 parents: [],
                 displayNamePrefix: newInputValue,
@@ -236,7 +235,7 @@ describe('DependentLookupEditor', function () {
             wrapper.update();
             expect(wrapper.find(ReactSelect).find('Input').last().prop('value')).toBe(getOptionLabel(value));
             expect(props.onChange).not.toHaveBeenCalled();
-            expect(props.getLookups).toBeCalledWith({
+            expect(props.getLookups).toHaveBeenCalledWith({
                 type: 'COUNTRY_CD',
                 parents: [],
                 displayNamePrefix: '',
@@ -328,7 +327,7 @@ describe('DependentLookupEditor', function () {
                     return [4 /*yield*/, act(function () { return __awaiter(void 0, void 0, void 0, function () {
                             return __generator(this, function (_a) {
                                 switch (_a.label) {
-                                    case 0: return [4 /*yield*/, delayPromise(1)];
+                                    case 0: return [4 /*yield*/, Promise.resolve()];
                                     case 1:
                                         _a.sent();
                                         return [2 /*return*/];
@@ -354,16 +353,13 @@ describe('DependentLookupEditor', function () {
                 case 2:
                     _a.sent();
                     // loaded options length is equal to (max + 1) so there are more options exist
-                    expect(getLookups).toBeCalledWith({
+                    expect(getLookups).toHaveBeenCalledWith({
                         type: props.lookupCode,
                         parents: props.parents,
                         displayNamePrefix: '',
                         max: 3,
                         offset: 2
                     });
-                    return [4 /*yield*/, Promise.resolve()];
-                case 3:
-                    _a.sent();
                     wrapper.update();
                     expect(wrapper.find(ReactSelect).prop('options').length).toBe(4);
                     expect(findLoadMoreButton(wrapper).exists()).toBeTruthy();
@@ -387,7 +383,7 @@ describe('DependentLookupEditor', function () {
                     return [4 /*yield*/, act(function () { return __awaiter(void 0, void 0, void 0, function () {
                             return __generator(this, function (_a) {
                                 switch (_a.label) {
-                                    case 0: return [4 /*yield*/, delayPromise(1)];
+                                    case 0: return [4 /*yield*/, Promise.resolve()];
                                     case 1:
                                         _a.sent();
                                         return [2 /*return*/];
@@ -411,9 +407,6 @@ describe('DependentLookupEditor', function () {
                             });
                         }); })];
                 case 2:
-                    _a.sent();
-                    return [4 /*yield*/, Promise.resolve()];
-                case 3:
                     _a.sent();
                     wrapper.update();
                     expect(wrapper.find(ReactSelect).prop('options').length).toBe(4);
@@ -477,7 +470,7 @@ describe('DependentLookupEditor', function () {
                 case 0:
                     wrapper = mount(React.createElement(DependentLookupEditor, __assign({}, props)));
                     expect(wrapper.find(ReactSelect).prop('options')).toEqual([]);
-                    expect(props.getLookups).toBeCalledWith({
+                    expect(props.getLookups).toHaveBeenCalledWith({
                         type: props.lookupCode,
                         parents: props.parents,
                         displayNamePrefix: '',
@@ -514,7 +507,7 @@ describe('DependentLookupEditor', function () {
                     wrapper.setProps({
                         lookupCode: 'SPEC'
                     });
-                    expect(props.getLookups).toBeCalledWith({
+                    expect(props.getLookups).toHaveBeenCalledWith({
                         type: 'SPEC',
                         parents: [],
                         displayNamePrefix: '',
@@ -559,9 +552,9 @@ describe('DependentLookupEditor', function () {
                 case 1:
                     _a.sent();
                     wrapper.find('input').simulate('keyDown', { key: 'ArrowDown' });
-                    expect(props.onChange).not.toBeCalled();
+                    expect(props.onChange).not.toHaveBeenCalled();
                     wrapper.find(ReactSelectOptionWithCheckIcon).at(1).find('div').at(0).simulate('click');
-                    expect(props.onChange).toBeCalledWith({
+                    expect(props.onChange).toHaveBeenCalledWith({
                         value: 'Russia',
                         lookupCode: 'RU'
                     });
@@ -569,20 +562,32 @@ describe('DependentLookupEditor', function () {
             }
         });
     }); });
-    it('should call props.getLookups with correct values on select input change', function () {
-        var wrapper = mount(React.createElement(DependentLookupEditor, __assign({}, props)));
-        var newInputValue = 'Ch';
-        act(function () {
-            wrapper.find('Select').prop('onInputChange')(newInputValue, { action: 'input-change' });
-        });
-        expect(props.getLookups).toHaveBeenCalledWith({
-            type: props.lookupCode,
-            parents: props.parents,
-            displayNamePrefix: newInputValue,
-            max: 51,
-            offset: 0
+    it('should call props.getLookups with correct values on select input change', function () { return __awaiter(void 0, void 0, void 0, function () {
+        var wrapper, newInputValue;
+        return __generator(this, function (_a) {
+            switch (_a.label) {
+                case 0:
+                    wrapper = mount(React.createElement(DependentLookupEditor, __assign({}, props)));
+                    newInputValue = 'Ch';
+                    return [4 /*yield*/, act(function () { return __awaiter(void 0, void 0, void 0, function () {
+                            return __generator(this, function (_a) {
+                                wrapper.find('Select').prop('onInputChange')(newInputValue, { action: 'input-change' });
+                                return [2 /*return*/];
+                            });
+                        }); })];
+                case 1:
+                    _a.sent();
+                    expect(props.getLookups).toHaveBeenCalledWith({
+                        type: props.lookupCode,
+                        parents: props.parents,
+                        displayNamePrefix: newInputValue,
+                        max: 51,
+                        offset: 0
+                    });
+                    return [2 /*return*/];
+            }
         });
-    });
+    }); });
     it('should call props.getLookups with correct values on select input change and set offset to default', function () { return __awaiter(void 0, void 0, void 0, function () {
         var getLookups, value, wrapper, newInputValue, nextNewInputValue;
         return __generator(this, function (_a) {
@@ -600,7 +605,16 @@ describe('DependentLookupEditor', function () {
                     act(function () {
                         wrapper.find('Select').prop('onInputChange')(newInputValue, { action: 'input-change' });
                     });
-                    return [4 /*yield*/, delayPromise(1)];
+                    return [4 /*yield*/, act(function () { return __awaiter(void 0, void 0, void 0, function () {
+                            return __generator(this, function (_a) {
+                                switch (_a.label) {
+                                    case 0: return [4 /*yield*/, Promise.resolve()];
+                                    case 1:
+                                        _a.sent();
+                                        return [2 /*return*/];
+                                }
+                            });
+                        }); })];
                 case 1:
                     _a.sent();
                     wrapper.update();
@@ -626,7 +640,7 @@ describe('DependentLookupEditor', function () {
                 case 2:
                     _a.sent();
                     // offset === max
-                    expect(getLookups).toBeCalledWith({
+                    expect(getLookups).toHaveBeenCalledWith({
                         type: props.lookupCode,
                         parents: props.parents,
                         displayNamePrefix: newInputValue,
@@ -649,7 +663,7 @@ describe('DependentLookupEditor', function () {
             }
         });
     }); });
-    it('should call props.getLookups with default offset = 0 after LookupCode updated', function () { return __awaiter(void 0, void 0, void 0, function () {
+    xit('should call props.getLookups with default offset = 0 after LookupCode updated', function () { return __awaiter(void 0, void 0, void 0, function () {
         var getLookups, wrapper, newLookupCode;
         return __generator(this, function (_a) {
             switch (_a.label) {
@@ -661,7 +675,16 @@ describe('DependentLookupEditor', function () {
                         RU: { displayName: 'Russia' }
                     }));
                     wrapper = mount(React.createElement(DependentLookupEditor, __assign({}, props, { getLookups: getLookups, max: 2 })));
-                    return [4 /*yield*/, delayPromise(1)];
+                    return [4 /*yield*/, act(function () { return __awaiter(void 0, void 0, void 0, function () {
+                            return __generator(this, function (_a) {
+                                switch (_a.label) {
+                                    case 0: return [4 /*yield*/, Promise.resolve()];
+                                    case 1:
+                                        _a.sent();
+                                        return [2 /*return*/];
+                                }
+                            });
+                        }); })];
                 case 1:
                     _a.sent();
                     wrapper.update();
@@ -686,7 +709,7 @@ describe('DependentLookupEditor', function () {
                         }); })];
                 case 2:
                     _a.sent();
-                    expect(getLookups).toBeCalledWith({
+                    expect(getLookups).toHaveBeenCalledWith({
                         type: props.lookupCode,
                         parents: props.parents,
                         displayNamePrefix: '',
@@ -722,10 +745,12 @@ describe('DependentLookupEditor', function () {
                     getLookups.mockReturnValue(Promise.resolve({}));
                     wrapper = mount(React.createElement(DependentLookupEditor, __assign({}, props, { getLookups: getLookups })));
                     newInputValue = 'Ch';
-                    act(function () {
-                        wrapper.find('Select').prop('onInputChange')(newInputValue, { action: 'input-change' });
-                    });
-                    return [4 /*yield*/, delayPromise(1)];
+                    return [4 /*yield*/, act(function () { return __awaiter(void 0, void 0, void 0, function () {
+                            return __generator(this, function (_a) {
+                                wrapper.find('Select').prop('onInputChange')(newInputValue, { action: 'input-change' });
+                                return [2 /*return*/];
+                            });
+                        }); })];
                 case 1:
                     _a.sent();
                     wrapper.update();
@@ -766,16 +791,18 @@ describe('DependentLookupEditor', function () {
                     wrapper = mount(React.createElement(DependentLookupEditor, __assign({}, props, { getLookups: getLookups, value: value, multiple: false })));
                     expect(wrapper.find(ReactSelect).find('Input').last().prop('value')).toBe('USA (US)');
                     newInputValue = 'u';
-                    act(function () {
-                        wrapper.find(ReactSelect).prop('onInputChange')(newInputValue, { action: 'input-change' });
-                    });
-                    return [4 /*yield*/, delayPromise(1)];
+                    return [4 /*yield*/, act(function () { return __awaiter(void 0, void 0, void 0, function () {
+                            return __generator(this, function (_a) {
+                                wrapper.find(ReactSelect).prop('onInputChange')(newInputValue, { action: 'input-change' });
+                                return [2 /*return*/];
+                            });
+                        }); })];
                 case 1:
                     _a.sent();
                     wrapper.update();
                     expect(wrapper.find(ReactSelect).find('Input').last().prop('value')).toBe(newInputValue);
                     expect(props.onChange).not.toHaveBeenCalled();
-                    expect(getLookups).toBeCalledWith({
+                    expect(getLookups).toHaveBeenCalledWith({
                         type: 'COUNTRY_CD',
                         parents: [],
                         displayNamePrefix: newInputValue,
@@ -800,16 +827,18 @@ describe('DependentLookupEditor', function () {
                     wrapper = mount(React.createElement(DependentLookupEditor, __assign({}, props, { getLookups: getLookups, value: value, multiple: false })));
                     expect(wrapper.find(ReactSelect).find('Input').last().prop('value')).toBe('USA (US)');
                     newInputValue = '';
-                    act(function () {
-                        wrapper.find(ReactSelect).prop('onInputChange')(newInputValue, { action: 'input-change' });
-                    });
-                    return [4 /*yield*/, delayPromise(1)];
+                    return [4 /*yield*/, act(function () { return __awaiter(void 0, void 0, void 0, function () {
+                            return __generator(this, function (_a) {
+                                wrapper.find(ReactSelect).prop('onInputChange')(newInputValue, { action: 'input-change' });
+                                return [2 /*return*/];
+                            });
+                        }); })];
                 case 1:
                     _a.sent();
                     wrapper.update();
                     expect(wrapper.find(ReactSelect).find('Input').last().prop('value')).toBe(newInputValue);
                     expect(props.onChange).toHaveBeenCalledWith('');
-                    expect(getLookups).toBeCalledWith({
+                    expect(getLookups).toHaveBeenCalledWith({
                         type: 'COUNTRY_CD',
                         parents: [],
                         displayNamePrefix: '',
@@ -830,7 +859,7 @@ describe('DependentLookupEditor', function () {
         };
         var value = { lookupCode: 'DE', value: '' };
         mount(React.createElement(DependentLookupEditor, __assign({}, props, { lookups: lookups, value: value })));
-        expect(props.onChange).toBeCalledWith({
+        expect(props.onChange).toHaveBeenCalledWith({
             lookupCode: 'DE',
             value: 'Germany'
         });
@@ -838,20 +867,21 @@ describe('DependentLookupEditor', function () {
     it('should call props.resolveLookups if lookup is unresolved and has no resolved value in lookups', function () {
         var value = { lookupCode: 'DE', value: '' };
         mount(React.createElement(DependentLookupEditor, __assign({}, props, { value: value })));
-        expect(props.resolveLookups).toBeCalledWith(props.lookupCode, ['DE']);
+        expect(props.resolveLookups).toHaveBeenCalledWith(props.lookupCode, ['DE']);
     });
     it('should call props.resolveLookups once if lookup is unresolved and has no resolved value in lookups and has no resolved value in response', function () {
         var value = { lookupCode: 'DE', value: '' };
         var component = mount(React.createElement(DependentLookupEditor, __assign({}, props, { value: value })));
         component.setProps({ value: __assign({}, value) });
         expect(props.resolveLookups).toHaveBeenCalledTimes(1);
-        expect(props.resolveLookups).toBeCalledWith(props.lookupCode, ['DE']);
+        expect(props.resolveLookups).toHaveBeenCalledWith(props.lookupCode, ['DE']);
     });
     it('should use debounced variant of loadOptions on input change', function () { return __awaiter(void 0, void 0, void 0, function () {
         var getLookups, component, select;
         return __generator(this, function (_a) {
             switch (_a.label) {
                 case 0:
+                    jest.useFakeTimers();
                     debounce.mockImplementation(originalDebounce);
                     getLookups = jest.fn().mockReturnValue(Promise.resolve({
                         US: { displayName: 'USA' },
@@ -859,13 +889,16 @@ describe('DependentLookupEditor', function () {
                     }));
                     component = shallow(React.createElement(DependentLookupEditor, __assign({}, props, { getLookups: getLookups })));
                     select = component.find(ReactSelect);
-                    select.prop('onInputChange')('u', { action: 'input-change' });
-                    select.prop('onInputChange')('u', { action: 'input-change' });
-                    return [4 /*yield*/, delayPromise(DEBOUNCE_INTERVAL)];
+                    act(function () {
+                        select.prop('onInputChange')('u', { action: 'input-change' });
+                    });
+                    return [4 /*yield*/, act(function () { return __awaiter(void 0, void 0, void 0, function () {
+                            return __generator(this, function (_a) {
+                                jest.advanceTimersByTime(DEBOUNCE_INTERVAL);
+                                return [2 /*return*/];
+                            });
+                        }); })];
                 case 1:
-                    _a.sent();
-                    return [4 /*yield*/, delayPromise(1)];
-                case 2:
                     _a.sent();
                     expect(getLookups).toHaveBeenCalledTimes(1);
                     expect(component.find(ReactSelect).prop('options')).toEqual([
@@ -876,6 +909,7 @@ describe('DependentLookupEditor', function () {
                         { lookupCode: 'US', value: 'USA' }
                     ]);
                     expect(debounce).toHaveBeenCalled();
+                    jest.useRealTimers();
                     return [2 /*return*/];
             }
         });
@@ -895,7 +929,9 @@ describe('DependentLookupEditor', function () {
         ];
         var component = shallow(React.createElement(DependentLookupEditor, __assign({}, props, { parents: parents })));
         var select = component.find(ReactSelect);
-        select.prop('onInputChange')('u', { action: 'input-change' });
+        act(function () {
+            select.prop('onInputChange')('u', { action: 'input-change' });
+        });
         expect(component.prop('isDisabled')).toBeFalsy();
         expect(props.getLookups).toHaveBeenCalledTimes(1);
         expect(props.getLookups).toHaveBeenCalledWith({
@@ -914,14 +950,15 @@ describe('DependentLookupEditor', function () {
                     parents = null;
                     component = shallow(React.createElement(DependentLookupEditor, __assign({}, props, { parents: parents })));
                     select = component.find(ReactSelect);
-                    select.prop('onInputChange')('u', { action: 'input-change' });
-                    expect(props.getLookups).not.toHaveBeenCalled();
-                    return [4 /*yield*/, delayPromise(DEBOUNCE_INTERVAL)];
+                    return [4 /*yield*/, act(function () { return __awaiter(void 0, void 0, void 0, function () {
+                            return __generator(this, function (_a) {
+                                select.prop('onInputChange')('u', { action: 'input-change' });
+                                return [2 /*return*/];
+                            });
+                        }); })];
                 case 1:
                     _a.sent();
-                    return [4 /*yield*/, delayPromise(1)];
-                case 2:
-                    _a.sent();
+                    expect(props.getLookups).not.toHaveBeenCalled();
                     expect(component.find(ReactSelect).prop('options')).toEqual([]);
                     return [2 /*return*/];
             }
@@ -931,7 +968,9 @@ describe('DependentLookupEditor', function () {
         var parents = undefined;
         var component = shallow(React.createElement(DependentLookupEditor, __assign({}, props, { parents: parents })));
         var select = component.find(ReactSelect);
-        select.prop('onInputChange')('u', { action: 'input-change' });
+        act(function () {
+            select.prop('onInputChange')('u', { action: 'input-change' });
+        });
         expect(props.getLookups).toHaveBeenCalledTimes(1);
         expect(props.getLookups).toHaveBeenCalledWith({
             displayNamePrefix: 'u',
@@ -1084,7 +1123,14 @@ describe('DependentLookupEditor', function () {
                     case 1:
                         _a.sent();
                         select = component.find(ReactSelect);
-                        select.prop('onInputChange')('Ru', { action: 'input-change' });
+                        return [4 /*yield*/, act(function () { return __awaiter(void 0, void 0, void 0, function () {
+                                return __generator(this, function (_a) {
+                                    select.prop('onInputChange')('Ru', { action: 'input-change' });
+                                    return [2 /*return*/];
+                                });
+                            }); })];
+                    case 2:
+                        _a.sent();
                         expect(getLookups).toHaveBeenCalledTimes(2);
                         expect(props.onChange).not.toHaveBeenCalled();
                         return [2 /*return*/];

package/esm/DropDownEditor/DropDownEditor.test.js

@@ -28,6 +28,6 @@ describe('DropDownEditor', function () {
         var onChange = jest.fn(), value = 'Enum2';
         var wrapper = shallow(React.createElement(DropDownEditor, { onChange: onChange, entries: entries }));
         wrapper.find(MultipliableSelect).simulate('change', value);
-        expect(onChange).toBeCalledWith(value);
+        expect(onChange).toHaveBeenCalledWith(value);
     });
 });

package/esm/EditModeAttributesFactory/components/EditModeAttribute/EditModeAttribute.test.js

@@ -22,12 +22,12 @@ var __rest = (this && this.__rest) || function (s, e) {
 };
 import React from 'react';
 import { shallow } from 'enzyme';
-import * as mdmSdk from '@reltio/mdm-sdk';
 import { EditModeAttribute } from './EditModeAttribute';
+import { isReltioCrosswalk } from '@reltio/mdm-sdk';
 import { SimpleAttributeEditor } from '../../../SimpleAttributeEditor';
 import { NestedAttributeEditor } from '../../../NestedAttributeEditor';
 import { ReferenceAttributeEditor } from '../../../ReferenceAttributeEditor';
-jest.mock('@reltio/mdm-sdk');
+jest.mock('@reltio/mdm-sdk', function () { return (__assign(__assign({}, jest.requireActual('@reltio/mdm-sdk')), { isReltioCrosswalk: jest.fn() })); });
 describe('EditModeAttribute', function () {
     it('should render Simple attribute', function () {
         var props = {
@@ -93,7 +93,7 @@ describe('EditModeAttribute', function () {
             errors: [],
             error: {}
         };
-        var isReltioCrosswalkSpy = jest.spyOn(mdmSdk, 'isReltioCrosswalk').mockReturnValue(true);
+        isReltioCrosswalk.mockReturnValue(true);
         var component = shallow(React.createElement(EditModeAttribute, __assign({}, props)));
         expect(component.find(SimpleAttributeEditor).length).toBe(1);
         expect(component.find(NestedAttributeEditor).length).toBe(0);
@@ -101,6 +101,6 @@ describe('EditModeAttribute', function () {
         // eslint-disable-next-line @typescript-eslint/no-unused-vars
         var showEmptyEditors = props.showEmptyEditors, onAddAttributes = props.onAddAttributes, lazy = props.lazy, crosswalks = props.crosswalks, errors = props.errors, filteredProps = __rest(props, ["showEmptyEditors", "onAddAttributes", "lazy", "crosswalks", "errors"]); //eslint-disable-line no-unused-vars
         expect(component.find(SimpleAttributeEditor).props()).toEqual(__assign(__assign({}, filteredProps), { isReltioCrosswalk: true }));
-        expect(isReltioCrosswalkSpy).toHaveBeenCalledWith(props.crosswalks, props.attributeValue);
+        expect(isReltioCrosswalk).toHaveBeenCalledWith(props.crosswalks, props.attributeValue);
     });
 });

package/esm/EditModeAttributesList/EditModeAttributesList.test.js

@@ -20,6 +20,7 @@ var __spreadArray = (this && this.__spreadArray) || function (to, from, pack) {
 };
 import React from 'react';
 import { mount } from 'enzyme';
+import { act } from 'react-dom/test-utils';
 import EditModeAttributesList from './EditModeAttributesList';
 import { MoreAttributesButton } from '../MoreAttributesButton';
 import { EditModeAttributesPager } from '../EditModeAttributesPager';
@@ -129,7 +130,9 @@ describe('EditModeAttributesList tests', function () {
         var component = mount(React.createElement(EditModeAttributesList, { attrTypes: attrTypes, entity: entity, parentUri: parentUri, showEmptyEditors: true }));
         var _b = component.find(HasDeletionsContext.Provider).prop('value'), hasDeletionsMap = _b.hasDeletionsMap, setHasDeletions = _b.setHasDeletions;
         expect(hasDeletionsMap).toEqual({});
-        setHasDeletions(attrTypes[0].uri, true);
+        act(function () {
+            setHasDeletions(attrTypes[0].uri, true);
+        });
         rerenderWrapper(component);
         expect(component.find(MoreAttributesButton).length).toBe(1);
         expect(component.find(MoreAttributesButton).prop('data')).toEqual([attrTypes[0]]);

package/esm/EditModeAttributesPager/EditModeAttributesPager.js

@@ -10,7 +10,7 @@ var __assign = (this && this.__assign) || function () {
     return __assign.apply(this, arguments);
 };
 import React, { useContext, useMemo } from 'react';
-import { getAttributeErrorByTypeUri, getAttributePagerActiveTypeErrorMessage, getBaseUri, isAttributeTypeError, isImage, isOv, isSpecialAttribute, Mode } from '@reltio/mdm-sdk';
+import { getAttributeErrorByTypeUri, getAttributePagerActiveTypeErrorMessage, getBaseUri, isAttributeTypeError, isImage, isOv, isSpecialAttribute, Mode, filterErrorsForHiddenAttributes } from '@reltio/mdm-sdk';
 import { always, cond, T, partition } from 'ramda';
 import { BranchDecorator } from '../BranchDecorator';
 import { AttributeRenderer } from './components/AttributeRenderer';
@@ -21,7 +21,6 @@ import { RENDERER_TYPES } from './types';
 import { isHighlightedAttributeType, isHighlightedErrorType, ScrollToElementContext } from '../contexts/ScrollToElementContext';
 import { useMdmAction, useMdmActiveErrorsForAttributesPager, useMdmMaxValuesInResponse, useMdmProfileErrors } from '../contexts/MdmModuleContext';
 import { HiddenAttributesContext } from '../contexts/HiddenAttributesContext';
-import { filterErrorsForHiddenAttributes } from '@reltio/mdm-sdk';
 var EditModeAttributesPager = function (_a) {
     var _b;
     var values = _a.values, attributeType = _a.attributeType, _c = _a.mode, mode = _c === void 0 ? Mode.Viewing : _c, crosswalks = _a.crosswalks, drawLines = _a.drawLines, _d = _a.paging, paging = _d === void 0 ? {} : _d, parentUri = _a.parentUri, maxProp = _a.max, showEmptyEditors = _a.showEmptyEditors, showNonOv = _a.showNonOv, highlightedError = _a.highlightedError, highlightedAttribute = _a.highlightedAttribute, onAddAttributes = _a.onAddAttributes, onDeleteAttribute = _a.onDeleteAttribute, onChangeAttribute = _a.onChangeAttribute, additionalControlsRenderer = _a.additionalControlsRenderer;

package/esm/EditModeAttributesView/IntegrationEditModeAttributesView.test.js

@@ -47,15 +47,13 @@ var __generator = (this && this.__generator) || function (thisArg, body) {
 };
 import React from 'react';
 import userEvent from '@testing-library/user-event';
-import { render, screen, act, waitFor, waitForElementToBeRemoved } from '@testing-library/react';
+import { render, screen, act, waitFor } from '@testing-library/react';
 import { DndProvider } from 'react-dnd';
 import HTML5Backend from 'react-dnd-html5-backend';
-import { ErrorType, Mode, uploadImage } from '@reltio/mdm-sdk';
-import * as mdmSdk from '@reltio/mdm-sdk';
+import { ErrorType, Mode, getAttributesListForEditMode, uploadImage } from '@reltio/mdm-sdk';
 import { EditModeAttributesView } from './EditModeAttributesView';
 import { MdmModuleProvider } from '../contexts/MdmModuleContext';
 import { ScrollToElementProvider } from '../contexts/ScrollToElementContext';
-import { delayPromise } from '../test-utils';
 import { getMetadata } from './metadata.test-data';
 jest.mock('@reltio/mdm-sdk', function () { return (__assign(__assign({}, jest.requireActual('@reltio/mdm-sdk')), { getAttributesListForEditMode: jest.fn(), uploadImage: jest.fn() })); });
 var modifiedEntities = {
@@ -103,7 +101,6 @@ var defaultProps = {
 describe('EditModeAttributesView tests', function () {
     var scrollSpy = jest.fn();
     window.HTMLElement.prototype.scrollIntoView = scrollSpy;
-    var getAttributesListForEditModeSpy = jest.spyOn(mdmSdk, 'getAttributesListForEditMode');
     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 ? defaultMdmValues : _d, _e = _b.userOptions, userOptions = _e === void 0 ? {} : _e;
         var user = userEvent.setup(userOptions);
@@ -127,7 +124,7 @@ describe('EditModeAttributesView tests', function () {
         return __generator(this, function (_b) {
             switch (_b.label) {
                 case 0:
-                    getAttributesListForEditModeSpy.mockReturnValue([
+                    getAttributesListForEditMode.mockReturnValue([
                         {
                             attrType: {
                                 label: 'FirstLevelNestedSub1_HCP',
@@ -162,7 +159,7 @@ describe('EditModeAttributesView tests', function () {
                         })];
                 case 4:
                     _b.sent();
-                    expect(scrollSpy).toBeCalledTimes(1);
+                    expect(scrollSpy).toHaveBeenCalledTimes(1);
                     act(function () {
                         jest.runOnlyPendingTimers();
                     });
@@ -186,7 +183,7 @@ describe('EditModeAttributesView tests', function () {
             }
         ];
         beforeEach(function () {
-            getAttributesListForEditModeSpy.mockReturnValue([
+            getAttributesListForEditMode.mockReturnValue([
                 {
                     attrType: {
                         label: 'Image Gallery',
@@ -199,17 +196,24 @@ describe('EditModeAttributesView tests', function () {
                 }
             ]);
         });
-        it('should render required and missed image attribute in edit mode correctly', function () {
-            setUp({ mdmValues: __assign(__assign({}, defaultMdmValues), { profileErrors: profileErrors }) });
-            expect(screen.getByText('*')).toBeInTheDocument();
-            expect(screen.getByText('Attribute is required')).toBeInTheDocument();
-        });
+        it('should render required and missed image attribute in edit mode correctly', function () { return __awaiter(void 0, void 0, void 0, function () {
+            return __generator(this, function (_a) {
+                setUp({ mdmValues: __assign(__assign({}, defaultMdmValues), { profileErrors: profileErrors }) });
+                expect(screen.getByText('*')).toBeInTheDocument();
+                expect(screen.getByText('Attribute is required')).toBeInTheDocument();
+                return [2 /*return*/];
+            });
+        }); });
         it('should deactivate error for required and missed image attribute after image was added', function () { return __awaiter(void 0, void 0, void 0, function () {
-            var user, file;
+            var resolveUploadImage, user, file;
             return __generator(this, function (_a) {
                 switch (_a.label) {
                     case 0:
-                        uploadImage.mockImplementation(function () { return delayPromise(50).then(function () { return Promise.resolve('Image'); }); });
+                        uploadImage.mockImplementation(function () {
+                            return new Promise(function (resolve) {
+                                resolveUploadImage = resolve;
+                            });
+                        });
                         user = setUp({
                             mdmValues: __assign(__assign({}, defaultMdmValues), { profileErrors: profileErrors }),
                             userOptions: { delay: null, applyAccept: false }
@@ -224,9 +228,19 @@ describe('EditModeAttributesView tests', function () {
                         return [4 /*yield*/, screen.findByTestId('reltio-linear-load-indicator')];
                     case 3:
                         _a.sent();
-                        return [4 /*yield*/, waitForElementToBeRemoved(screen.queryByTestId('reltio-linear-load-indicator'))];
+                        return [4 /*yield*/, act(function () { return __awaiter(void 0, void 0, void 0, function () {
+                                return __generator(this, function (_a) {
+                                    switch (_a.label) {
+                                        case 0: return [4 /*yield*/, resolveUploadImage('Image')];
+                                        case 1:
+                                            _a.sent();
+                                            return [2 /*return*/];
+                                    }
+                                });
+                            }); })];
                     case 4:
                         _a.sent();
+                        expect(screen.queryByTestId('reltio-linear-load-indicator')).not.toBeInTheDocument();
                         expect(mdmActions.errorDeactivated).toHaveBeenCalledWith('configuration/entityTypes/HCP/attributes/ImageGallery--entities/1E4APrQW');
                         return [2 /*return*/];
                 }