@reltio/components 1.4.1944 → 1.4.1946

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*/];
                 }

package/esm/EntitySelector/EntitySelector.test.js

@@ -48,7 +48,7 @@ var __generator = (this && this.__generator) || function (thisArg, body) {
 import React from 'react';
 import { shallow, mount } from 'enzyme';
 import { act } from 'react-dom/test-utils';
-import * as mdmSdkModule from '@reltio/mdm-sdk';
+import { typeAheadSearch } from '@reltio/mdm-sdk';
 import { MdmModuleProvider } from '../contexts/MdmModuleContext';
 import { EntitySelector } from './EntitySelector';
 var metadata = {
@@ -72,14 +72,13 @@ var mockOnChange = jest.fn();
 var mockEntities = [];
 jest.mock('@reltio/mdm-sdk', function () { return (__assign(__assign({}, jest.requireActual('@reltio/mdm-sdk')), { typeAheadSearch: jest.fn() })); });
 describe('EntitySelector', function () {
-    var typeAheadSearchSpy;
     var globalSearchRequestOptions = {
         searchOptions: 'searchByOv',
         activityFilter: undefined,
         globalFilter: '123'
     };
     beforeAll(function () {
-        typeAheadSearchSpy = jest.spyOn(mdmSdkModule, 'typeAheadSearch').mockImplementation(function () {
+        typeAheadSearch.mockImplementation(function () {
             return new Promise(function (resolve) {
                 resolve(mockEntities);
             });
@@ -134,7 +133,7 @@ describe('EntitySelector', function () {
             .then(function () {
             var entityTypes = [metadata.entityTypes[0]];
             var options = __assign(__assign({}, globalSearchRequestOptions), { max: 5, sendMasked: false });
-            expect(typeAheadSearchSpy).toHaveBeenCalledWith(entityTypes, undefined, options);
+            expect(typeAheadSearch).toHaveBeenCalledWith(entityTypes, undefined, options);
             done();
         });
     });
@@ -154,7 +153,7 @@ describe('EntitySelector', function () {
                 max: 20,
                 sendMasked: false
             };
-            expect(typeAheadSearchSpy).toHaveBeenCalledWith(entityTypes, undefined, options);
+            expect(typeAheadSearch).toHaveBeenCalledWith(entityTypes, undefined, options);
             done();
         });
     });
@@ -185,7 +184,6 @@ describe('EntitySelector', function () {
         return __generator(this, function (_b) {
             switch (_b.label) {
                 case 0:
-                    expect.assertions(1);
                     entityTypesUris = ['configuration/entityTypes/HCP'];
                     wrapper = shallow(React.createElement(EntitySelector, { onChange: mockOnChange, entityTypesUris: entityTypesUris, metadata: metadata }));
                     _a = expect;
@@ -207,7 +205,6 @@ describe('EntitySelector', function () {
         return __generator(this, function (_b) {
             switch (_b.label) {
                 case 0:
-                    expect.assertions(1);
                     entityTypesUris = ['configuration/entityTypes/HCP'];
                     mockEntities = [
                         {
@@ -254,7 +251,6 @@ describe('EntitySelector', function () {
         return __generator(this, function (_b) {
             switch (_b.label) {
                 case 0:
-                    expect.assertions(1);
                     entityTypesUris = ['configuration/entityTypes/HCP'];
                     mockEntities = [
                         {
@@ -288,7 +284,6 @@ describe('EntitySelector', function () {
         return __generator(this, function (_b) {
             switch (_b.label) {
                 case 0:
-                    expect.assertions(1);
                     entityTypesUris = ['configuration/entityTypes/HCP', 'configuration/entityTypes/HCA'];
                     mockEntities = [
                         {
@@ -347,7 +342,6 @@ describe('EntitySelector', function () {
         return __generator(this, function (_b) {
             switch (_b.label) {
                 case 0:
-                    expect.assertions(1);
                     onCreateMock = jest.fn();
                     entityTypesUris = ['configuration/entityTypes/HCP', 'configuration/entityTypes/HCA'];
                     mockEntities = [
@@ -394,9 +388,9 @@ describe('EntitySelector', function () {
         });
     }); });
     it('should process entities list to get options in case two types (one is abstract) and with entities only for one type', function () { return __awaiter(void 0, void 0, void 0, function () {
-        var onCreateMock, entityTypesUris, wrapper, _a;
-        return __generator(this, function (_b) {
-            switch (_b.label) {
+        var onCreateMock, entityTypesUris, wrapper, options;
+        return __generator(this, function (_a) {
+            switch (_a.label) {
                 case 0:
                     expect.assertions(1);
                     onCreateMock = jest.fn();
@@ -409,10 +403,10 @@ describe('EntitySelector', function () {
                         }
                     ];
                     wrapper = shallow(React.createElement(EntitySelector, { onCreate: onCreateMock, onChange: mockOnChange, entityTypesUris: entityTypesUris, metadata: metadata }));
-                    _a = expect;
                     return [4 /*yield*/, wrapper.find('DropDownSelector').prop('getOptions')()];
                 case 1:
-                    _a.apply(void 0, [_b.sent()]).toEqual([
+                    options = _a.sent();
+                    expect(options).toEqual([
                         {
                             entityType: {
                                 label: 'HCP',
@@ -440,7 +434,7 @@ describe('EntitySelector', function () {
         act(function () { return wrapper.find('DropDownSelector').prop('onInputChange')('someValue'); });
         wrapper.update();
         act(function () { return wrapper.find('DropDownSelector').prop('onCreate')('someValue'); });
-        expect(onCreateMock).toBeCalledWith('configuration/entityTypes/HCP', 'someValue');
+        expect(onCreateMock).toHaveBeenCalledWith('configuration/entityTypes/HCP', 'someValue');
         expect(wrapper.find('DropDownSelector').prop('createLabel')).toEqual('Create "someValue" as new HCP');
     });
     it('should run onCreate for several types', function () {
@@ -450,7 +444,7 @@ describe('EntitySelector', function () {
         act(function () { return wrapper.find('DropDownSelector').prop('onInputChange')('someValue'); });
         wrapper.update();
         act(function () { return wrapper.find('DropDownSelector').prop('onCreate')('someValue', 'configuration/entityTypes/HCA'); });
-        expect(onCreateMock).toBeCalledWith('configuration/entityTypes/HCA', 'someValue');
+        expect(onCreateMock).toHaveBeenCalledWith('configuration/entityTypes/HCA', 'someValue');
     });
     it('should run onChange', function () {
         var entityTypesUris = ['configuration/entityTypes/HCP'];
@@ -460,7 +454,7 @@ describe('EntitySelector', function () {
             wrapper.find('DropDownSelector').prop('onChange')('newValue');
         });
         wrapper.update();
-        expect(mockOnChange).toBeCalledWith('newValue');
+        expect(mockOnChange).toHaveBeenCalledWith('newValue');
         expect(wrapper.find('Input').prop('value')).toBe('');
     });
     it('should run onClear', function () {
@@ -471,7 +465,7 @@ describe('EntitySelector', function () {
             wrapper.find('DropDownSelector').prop('onClear')();
         });
         wrapper.update();
-        expect(mockOnChange).toBeCalled();
+        expect(mockOnChange).toHaveBeenCalled();
         expect(wrapper.find('Input').prop('value')).toBe('');
     });
 });

package/esm/EntitySelector/components/ClearIndicator/ClearIndicator.test.js

@@ -22,6 +22,6 @@ describe('ClearIndicator', function () {
         };
         var wrapper = shallow(React.createElement(ClearIndicator, __assign({}, props)));
         wrapper.simulate('click');
-        expect(onClear).toBeCalled();
+        expect(onClear).toHaveBeenCalled();
     });
 });

package/esm/EntitySelector/components/EntityOption/EntityOption.test.js

@@ -74,6 +74,6 @@ describe('EntityOption', function () {
         };
         var wrapper = mount(React.createElement(EntityOption, __assign({}, props)));
         wrapper.find('li').simulate('click');
-        expect(selectOptionMock).toBeCalledWith(props.data);
+        expect(selectOptionMock).toHaveBeenCalledWith(props.data);
     });
 });

package/esm/EntitySelector/components/Group/Group.test.js

@@ -60,6 +60,6 @@ describe('Group', function () {
             } }, defaultProps);
         var wrapper = mount(React.createElement(Group, __assign({}, props)));
         wrapper.find('.addLabel').simulate('click');
-        expect(onCreateMock).toBeCalledWith('Input Value', props.data.entityType);
+        expect(onCreateMock).toHaveBeenCalledWith('Input Value', props.data.entityType);
     });
 });

package/esm/EntityUriLink/EntityUriLink.test.js

@@ -142,6 +142,7 @@ describe('EntityUriLink', function () {
                 case 0:
                     user = setUp().user;
                     link = screen.getByText('label');
+                    link.addEventListener('click', function (event) { return event.preventDefault(); }, false);
                     return [4 /*yield*/, user.keyboard('{Control>}')];
                 case 1:
                     _a.sent();

package/esm/ErrorBoundary/ErrorBoundary.test.js

@@ -10,11 +10,13 @@ describe('ErrorBoundary test', function () {
         expect(wrapper.find('.children').length).toBe(1);
     });
     it('it should be render error component after click', function () {
+        jest.spyOn(console, 'error').mockReturnValue();
         var wrapper = mount(React.createElement(ErrorBoundary, { error: errorComponent },
             React.createElement(Children, null)));
         var error = new Error('error');
         wrapper.find(Children).simulateError(error);
         wrapper.update();
         expect(wrapper.find('.error').length).toBe(1);
+        expect(console.error).toHaveBeenCalledWith(error);
     });
 });

package/esm/ExpandedValueTooltip/ExpandedValueTooltip.test.js

@@ -1,6 +1,7 @@
 import React from 'react';
 import ReactResizeDetector from 'react-resize-detector';
 import { mount } from 'enzyme';
+import { act } from 'react-dom/test-utils';
 import Tooltip from '@mui/material/Tooltip';
 import { ExpandedValueTooltip } from './ExpandedValueTooltip';
 describe('ExpandedValueTooltip', function () {
@@ -15,7 +16,9 @@ describe('ExpandedValueTooltip', function () {
             clientWidth: 50,
             scrollWidth: 100
         };
-        wrapper.find(ReactResizeDetector).prop('onResize')();
+        act(function () {
+            wrapper.find(ReactResizeDetector).prop('onResize')();
+        });
         wrapper.update();
         expect(wrapper.find(Tooltip).prop('title')).toEqual(expandedValue);
     });

package/esm/FilterButton/FilterButton.test.js

@@ -64,7 +64,7 @@ describe('FilterButton tests', function () {
                     return [4 /*yield*/, user.click(screen.getByRole('button'))];
                 case 1:
                     _a.sent();
-                    expect(onClick).toBeCalled();
+                    expect(onClick).toHaveBeenCalled();
                     return [2 /*return*/];
             }
         });

package/esm/HOCs/withFilter/components/ColumnFilter/ColumnFilter.spec.js

@@ -50,7 +50,7 @@ import { render, screen } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 import { DataTypes, getFacetedAttributeData } from '@reltio/mdm-sdk';
 import { ColumnFilter } from './ColumnFilter';
-jest.mock('@reltio/mdm-sdk', function () { return (__assign(__assign({}, jest.requireActual('@reltio/mdm-sdk')), { getFacetedAttributeData: jest.fn() })); });
+jest.mock('@reltio/mdm-sdk', function () { return (__assign(__assign({}, jest.requireActual('@reltio/mdm-sdk')), { getFacetedAttributeData: jest.fn(), debounce: function (x) { return x; } })); });
 var defaultProps = {
     columnId: 'columnId',
     dataTypeDefinition: { name: 'TextField', type: DataTypes.TYPE_TYPEAHEAD },
@@ -83,7 +83,7 @@ describe('ColumnFilter behaviour', function () {
                     return [4 /*yield*/, user.click(screen.getByText('starts with'))];
                 case 3:
                     _a.sent();
-                    return [4 /*yield*/, user.type(screen.getByRole('textbox'), 'filter value')];
+                    return [4 /*yield*/, user.type(screen.getByRole('textbox'), 'val')];
                 case 4:
                     _a.sent();
                     popover = screen.getByRole('presentation');
@@ -93,7 +93,7 @@ describe('ColumnFilter behaviour', function () {
                     _a.sent();
                     expect(defaultProps.onFilter).toHaveBeenCalledWith({
                         columnId: 'columnId',
-                        filter: { filter: 'startsWith', value: 'filter value' }
+                        filter: { filter: 'startsWith', value: 'val' }
                     });
                     return [2 /*return*/];
             }

package/esm/HOCs/withTableContext/withTableContext.test.js

@@ -36,6 +36,6 @@ describe('withTableContext behavior', function () {
         var ownProps = { someProp: true };
         mount(React.createElement(BasicTableContext.Provider, { value: { cellContext: cellContext } },
             React.createElement(CellRendererWithContext, __assign({}, ownProps))));
-        expect(contextSelector).toBeCalledWith({ cellContext: cellContext }, ownProps);
+        expect(contextSelector).toHaveBeenCalledWith({ cellContext: cellContext }, ownProps);
     });
 });

package/esm/HierarchyNodeTitle/HierarchyNodeTitle.test.js

@@ -46,14 +46,14 @@ var __generator = (this && this.__generator) || function (thisArg, body) {
     }
 };
 import React from 'react';
-import * as mdmSdk from '@reltio/mdm-sdk';
 import { act } from 'react-dom/test-utils';
 import { mount } from 'enzyme';
-import { HierarchyNodeTitle } from './HierarchyNodeTitle';
+import { getEntity } from '@reltio/mdm-sdk';
 import { Link } from '../Link';
 import { Popper } from '../Popper';
 import { ProfileCard } from '../ProfileCard';
 import { MdmModuleProvider } from '../contexts/MdmModuleContext';
+import { HierarchyNodeTitle } from './HierarchyNodeTitle';
 jest.mock('../ProfileCard', function () { return ({ ProfileCard: function () { return null; } }); });
 jest.mock('@reltio/mdm-sdk', function () { return (__assign(__assign({}, jest.requireActual('@reltio/mdm-sdk')), { getEntity: jest.fn() })); });
 var entity = {
@@ -75,7 +75,7 @@ var setUp = function (props) {
 };
 describe('HierarchyNodeTitle tests', function () {
     beforeEach(function () {
-        jest.spyOn(mdmSdk, 'getEntity').mockResolvedValue(entity);
+        getEntity.mockResolvedValue(entity);
     });
     afterEach(function () {
         jest.clearAllMocks();
@@ -131,7 +131,7 @@ describe('HierarchyNodeTitle tests', function () {
                         entity: null,
                         variant: 'loading'
                     });
-                    expect(mdmSdk.getEntity).toHaveBeenCalledWith(defaultProps.entityUri);
+                    expect(getEntity).toHaveBeenCalledWith(defaultProps.entityUri);
                     return [4 /*yield*/, act(function () { return __awaiter(void 0, void 0, void 0, function () {
                             return __generator(this, function (_a) {
                                 return [2 /*return*/, Promise.resolve()];

package/esm/HierarchyNodeTitle/useEntityDetails.test.js

@@ -1,3 +1,14 @@
+var __assign = (this && this.__assign) || function () {
+    __assign = Object.assign || function(t) {
+        for (var s, i = 1, n = arguments.length; i < n; i++) {
+            s = arguments[i];
+            for (var p in s) if (Object.prototype.hasOwnProperty.call(s, p))
+                t[p] = s[p];
+        }
+        return t;
+    };
+    return __assign.apply(this, arguments);
+};
 var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
     function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
     return new (P || (P = Promise))(function (resolve, reject) {
@@ -35,17 +46,15 @@ var __generator = (this && this.__generator) || function (thisArg, body) {
     }
 };
 import { renderHook, act } from '@testing-library/react-hooks';
-import * as mdmSdk from '@reltio/mdm-sdk';
+import { getEntity, debounce } from '@reltio/mdm-sdk';
 import { useEntityDetails } from './useEntityDetails';
-jest.mock('@reltio/mdm-sdk');
+jest.mock('@reltio/mdm-sdk', function () { return (__assign(__assign({}, jest.requireActual('@reltio/mdm-sdk')), { getEntity: jest.fn(), debounce: jest.fn() })); });
 describe('useEntityDetails', function () {
     var entity = { uri: 'entities/123', type: 'configuration/entityTypes/HCP' };
     beforeEach(function () {
         jest.useFakeTimers();
-        jest.spyOn(mdmSdk, 'getEntity').mockResolvedValue(entity);
-        jest.spyOn(mdmSdk, 'debounce').mockImplementation(function (fn) {
-            return jest.requireActual('@reltio/mdm-sdk').debounce(fn, 10);
-        });
+        getEntity.mockResolvedValue(entity);
+        debounce.mockImplementation(function (fn) { return jest.requireActual('@reltio/mdm-sdk').debounce(fn, 10); });
     });
     afterEach(function () {
         jest.useRealTimers();
@@ -68,7 +77,7 @@ describe('useEntityDetails', function () {
                     });
                     expect(result.current.isLoading).toBe(true);
                     expect(result.current.entityDetails).toBe(null);
-                    expect(mdmSdk.getEntity).toHaveBeenCalledWith(entity.uri);
+                    expect(getEntity).toHaveBeenCalledWith(entity.uri);
                     return [4 /*yield*/, act(function () { return __awaiter(void 0, void 0, void 0, function () {
                             return __generator(this, function (_a) {
                                 return [2 /*return*/, Promise.resolve()];
@@ -116,7 +125,7 @@ describe('useEntityDetails', function () {
                         result.current.showEntityDetails();
                         jest.runAllTimers();
                     });
-                    expect(mdmSdk.getEntity).toHaveBeenCalled();
+                    expect(getEntity).toHaveBeenCalled();
                     act(function () {
                         result.current.hideEntityDetails();
                     });
@@ -148,7 +157,7 @@ describe('useEntityDetails', function () {
         act(function () {
             jest.runAllTimers();
         });
-        expect(mdmSdk.getEntity).not.toHaveBeenCalled();
+        expect(getEntity).not.toHaveBeenCalled();
         expect(result.current.isLoading).toBe(false);
         expect(result.current.entityDetails).toBe(null);
     });
@@ -162,7 +171,7 @@ describe('useEntityDetails', function () {
                         result.current.showEntityDetails();
                         jest.runAllTimers();
                     });
-                    expect(mdmSdk.getEntity).toHaveBeenCalledWith('entities/123');
+                    expect(getEntity).toHaveBeenCalledWith('entities/123');
                     expect(result.current.isLoading).toBe(true);
                     expect(result.current.entityDetails).toBe(null);
                     act(function () {

package/esm/ImageAttributesGallery/ImageAttributesGallery.test.js

@@ -145,7 +145,9 @@ describe('ImageAttributesGallery tests', function () {
         var id = 'entities/1Rd2I9Mt/attributes/ImageGallery/39V5GWJZ0';
         var component = setUp();
         expect(component.find(ImageGalleryDialog).prop('mode')).toBe('gallery');
-        component.find(GalleryView).prop('onClickImage')(id);
+        act(function () {
+            component.find(GalleryView).prop('onClickImage')(id);
+        });
         expect(defaultProps.setCurrentAttributeValueUri).toHaveBeenCalledWith(id);
         component.update();
         expect(component.find(ImageGalleryDialog).prop('mode')).toBe('details');
@@ -277,7 +279,7 @@ describe('ImageAttributesGallery tests', function () {
                     onDelete();
                 }
             });
-            expect(onDeleteAttribute).toBeCalledWith({
+            expect(onDeleteAttribute).toHaveBeenCalledWith({
                 uri: 'entities/1Rd2I9Mt/attributes/ImageGallery/39V5GWJZ1',
                 attributeType: defaultProps.attributeType
             });
@@ -299,7 +301,7 @@ describe('ImageAttributesGallery tests', function () {
                     onDelete();
                 }
             });
-            expect(onDeleteAttribute).toBeCalledWith({
+            expect(onDeleteAttribute).toHaveBeenCalledWith({
                 uri: 'entities/1Rd2I9Mt/attributes/ImageGallery/39V5GWJZ1',
                 attributeType: defaultProps.attributeType
             });
@@ -312,7 +314,7 @@ describe('ImageAttributesGallery tests', function () {
                 var onDelete = component.find(GalleryView).prop('onDelete');
                 onDelete('entities/1Rd2I9Mt/attributes/ImageGallery/39V5GWJZ1');
             });
-            expect(onDeleteAttribute).toBeCalledWith({
+            expect(onDeleteAttribute).toHaveBeenCalledWith({
                 uri: 'entities/1Rd2I9Mt/attributes/ImageGallery/39V5GWJZ1',
                 attributeType: defaultProps.attributeType
             });

package/esm/ImageAttributesGallery/components/HeaderRight/HeaderRight.test.js

@@ -70,7 +70,7 @@ describe('HeaderRight tests', function () {
                     return [4 /*yield*/, user.click(uploadButton)];
                 case 1:
                     _a.sent();
-                    expect(onUpload).toBeCalledTimes(1);
+                    expect(onUpload).toHaveBeenCalledTimes(1);
                     return [2 /*return*/];
             }
         });