Skip to content

Commit b452e92

Browse files
committed
style(useQueries): remove unnecessary comments
1 parent bf48183 commit b452e92

File tree

1 file changed

+0
-12
lines changed

1 file changed

+0
-12
lines changed

packages/react-query/src/__tests__/useQueries.test.tsx

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1742,7 +1742,6 @@ describe('useQueries', () => {
17421742
expect(spy).toHaveBeenCalledTimes(3)
17431743
})
17441744

1745-
// Regression test for issue #9638
17461745
it('should not cause infinite re-renders when removing last query', async () => {
17471746
let renderCount = 0
17481747

@@ -1767,7 +1766,6 @@ describe('useQueries', () => {
17671766
<div data-testid="query-count">queries: {result.length}</div>
17681767
<button
17691768
onClick={() => {
1770-
// This should NOT cause infinite re-renders (removing last query)
17711769
setQueries([{
17721770
queryKey: ['query1'],
17731771
queryFn: () => 'data1',
@@ -1778,7 +1776,6 @@ describe('useQueries', () => {
17781776
</button>
17791777
<button
17801778
onClick={() => {
1781-
// This should work fine (removing first query)
17821779
setQueries([{
17831780
queryKey: ['query2'],
17841781
queryFn: () => 'data2',
@@ -1794,28 +1791,19 @@ describe('useQueries', () => {
17941791
const rendered = renderWithClient(queryClient, <Page />)
17951792

17961793
await vi.advanceTimersByTimeAsync(0)
1797-
1798-
// Reset render count after initial render
17991794
renderCount = 0
18001795

1801-
// Remove last query - this was causing infinite re-renders
18021796
fireEvent.click(rendered.getByRole('button', { name: /remove last/i }))
1803-
18041797
await vi.advanceTimersByTimeAsync(100)
18051798

1806-
// Should have a reasonable number of renders (not infinite)
18071799
expect(renderCount).toBeLessThan(10)
18081800
expect(rendered.getByTestId('query-count').textContent).toBe('queries: 1')
18091801

1810-
// Reset render count
18111802
renderCount = 0
18121803

1813-
// Remove first query - this should also work fine
18141804
fireEvent.click(rendered.getByRole('button', { name: /remove first/i }))
1815-
18161805
await vi.advanceTimersByTimeAsync(100)
18171806

1818-
// Should have a reasonable number of renders
18191807
expect(renderCount).toBeLessThan(10)
18201808
expect(rendered.getByTestId('query-count').textContent).toBe('queries: 1')
18211809
})

0 commit comments

Comments
 (0)