fix(mentions): Fix tests

box · Jun 24, 2020 · e7d1063 · e7d1063
1 parent 2224ff1
commit e7d1063
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/components/ItemList/ItemList.tsx b/src/components/ItemList/ItemList.tsx
@@ -41,9 +41,9 @@ const ItemList = <T extends { id: string }>({
             }
 
             if (listEl.scrollTop > activeEl.offsetTop) {
-                listEl.scrollTop = activeEl.offsetTop;
+                listEl.scrollTop -= activeEl.offsetHeight;
             } else if (listEl.scrollTop + listHeight < activeEl.offsetTop + activeEl.offsetHeight) {
-                listEl.scrollTop = activeEl.offsetTop + activeEl.offsetHeight - listHeight;
+                listEl.scrollTop += activeEl.offsetHeight;
             }
         },
         [listRef],

diff --git a/src/components/ItemList/__tests__/ItemList-test.tsx b/src/components/ItemList/__tests__/ItemList-test.tsx
@@ -1,5 +1,5 @@
 import React from 'react';
-import { shallow, ShallowWrapper } from 'enzyme';
+import { mount, ReactWrapper } from 'enzyme';
 import ItemList, { Props } from '../ItemList';
 import { Collaborator } from '../../../@types';
 
@@ -12,7 +12,7 @@ describe('components/ItemList/ItemList', () => {
         onSelect: jest.fn(),
     };
 
-    const getWrapper = (props = {}): ShallowWrapper => shallow(<ItemList {...defaults} {...props} />);
+    const getWrapper = (props = {}): ReactWrapper => mount(<ItemList {...defaults} {...props} />);
 
     describe('render()', () => {
         test('should render elements with correct props', () => {
@@ -41,7 +41,7 @@ describe('components/ItemList/ItemList', () => {
             const firstChild = wrapper.find('[data-testid="ba-ItemList"]').childAt(0);
 
             firstChild.simulate('click', mockEvent);
-            expect(defaults.onSelect).toBeCalledWith(0, mockEvent);
+            expect(defaults.onSelect).toBeCalledWith(0, expect.objectContaining(mockEvent));
 
             firstChild.simulate('mousedown', mockEvent);
             expect(mockEvent.preventDefault).toBeCalled();