fix tests and lint

Author: LFDanLu

packages/@react-aria/virtualizer/src/ScrollView.tsx

@@ -213,7 +213,7 @@ export function useScrollView(props: ScrollViewProps, ref: RefObject<HTMLElement
         // due to strict mode.
         setUpdate({});
         lastContentSize.current = contentSize;
-        return
+        return;
       } else {
         queueMicrotask(() => updateSize(flushSync));
       }
@@ -225,7 +225,7 @@ export function useScrollView(props: ScrollViewProps, ref: RefObject<HTMLElement
   // Will only run in tests, needs to be in separate effect so it is properly run in the next render in strict mode.
   useLayoutEffect(() => {
     updateSize(fn => fn());
-  }, [update])
+  }, [update]);
 
   let onResize = useCallback(() => {
     updateSize(flushSync);

packages/react-aria-components/test/GridList.test.js

@@ -946,15 +946,14 @@ describe('GridList', () => {
 
       let tree = render(<AsyncGridList items={items} onLoadMore={onLoadMore} isLoading />);
       let sentinel = tree.getByTestId('loadMoreSentinel');
-      expect(observe).toHaveBeenCalledTimes(2);
       expect(observe).toHaveBeenLastCalledWith(sentinel);
       expect(onLoadMore).toHaveBeenCalledTimes(0);
 
       act(() => {observer.instance.triggerCallback([{isIntersecting: true}]);});
       expect(onLoadMore).toHaveBeenCalledTimes(1);
+      observe.mockClear();
 
       tree.rerender(<AsyncGridList items={items} onLoadMore={onLoadMore} />);
-      expect(observe).toHaveBeenCalledTimes(3);
       expect(observe).toHaveBeenLastCalledWith(sentinel);
       expect(onLoadMore).toHaveBeenCalledTimes(1);
 

packages/react-aria-components/test/ListBox.test.js

@@ -1417,15 +1417,14 @@ describe('ListBox', () => {
 
       let tree = render(<AsyncListbox items={items} onLoadMore={onLoadMore} isLoading />);
       let sentinel = tree.getByTestId('loadMoreSentinel');
-      expect(observe).toHaveBeenCalledTimes(2);
       expect(observe).toHaveBeenLastCalledWith(sentinel);
       expect(onLoadMore).toHaveBeenCalledTimes(0);
 
       act(() => {observer.instance.triggerCallback([{isIntersecting: true}]);});
       expect(onLoadMore).toHaveBeenCalledTimes(1);
+      observe.mockClear();
 
       tree.rerender(<AsyncListbox items={items} onLoadMore={onLoadMore} />);
-      expect(observe).toHaveBeenCalledTimes(3);
       expect(observe).toHaveBeenLastCalledWith(sentinel);
       expect(onLoadMore).toHaveBeenCalledTimes(1);
 

packages/react-aria-components/test/Table.test.js

@@ -1931,15 +1931,14 @@ describe('Table', () => {
 
       let tree = render(<LoadMoreTable items={items} onLoadMore={onLoadMore} isLoading />);
       let sentinel = tree.getByTestId('loadMoreSentinel');
-      expect(observe).toHaveBeenCalledTimes(2);
       expect(observe).toHaveBeenLastCalledWith(sentinel);
       expect(onLoadMore).toHaveBeenCalledTimes(0);
+      observe.mockClear();
 
       act(() => {observer.instance.triggerCallback([{isIntersecting: true}]);});
       expect(onLoadMore).toHaveBeenCalledTimes(1);
 
       tree.rerender(<LoadMoreTable items={items} onLoadMore={onLoadMore} />);
-      expect(observe).toHaveBeenCalledTimes(3);
       expect(observe).toHaveBeenLastCalledWith(sentinel);
       expect(onLoadMore).toHaveBeenCalledTimes(1);