fix Combobox test so it properly catches previous Document bug

tested with the Document changes removed and verified that the test fails properly. Swore it failed when I first wrote it...

Author: LFDanLu

packages/@react-spectrum/s2/test/Combobox.test.tsx

@@ -39,6 +39,7 @@ describe('Combobox', () => {
     jest.useFakeTimers();
     jest.spyOn(window.HTMLElement.prototype, 'clientWidth', 'get').mockImplementation(() => 100);
     jest.spyOn(window.HTMLElement.prototype, 'clientHeight', 'get').mockImplementation(() => 100);
+    jest.spyOn(window.HTMLElement.prototype, 'scrollHeight', 'get').mockImplementation(() => 50);
     user = userEvent.setup({delay: null, pointerMap});
   });
 
@@ -154,7 +155,7 @@ describe('Combobox', () => {
     }
 
     // A bit contrived, but essentially testing a combinaiton of insertions/deletions along side some of the old entries remaining
-    let newItems = [{name: 'Chocolate Mint'}, {name: 'Chocolate'}, {name: 'Chocolate Chip'}, {name: 'Chocolate Chip Cookie Dough'}];
+    let newItems = [{name: 'Chocolate'}, {name: 'Chocolate Mint'}, {name: 'Chocolate Chip Cookie Dough'}, {name: 'Chocolate Chip'}];
     tree.rerender(<DynamicCombobox items={newItems} loadingState="idle" />);
 
     options = comboboxTester.options();