From cb3579b2ada32f1f8de9ab244a396da3a4ab987f Mon Sep 17 00:00:00 2001 From: eps1lon Date: Thu, 23 Apr 2026 17:22:32 +0200 Subject: [PATCH] Fix formatting --- .../__tests__/ReactDOMFragmentRefs-test.js | 284 +++++++++--------- 1 file changed, 136 insertions(+), 148 deletions(-) diff --git a/packages/react-dom/src/__tests__/ReactDOMFragmentRefs-test.js b/packages/react-dom/src/__tests__/ReactDOMFragmentRefs-test.js index 7b41e1a2a87..ae934b914e2 100644 --- a/packages/react-dom/src/__tests__/ReactDOMFragmentRefs-test.js +++ b/packages/react-dom/src/__tests__/ReactDOMFragmentRefs-test.js @@ -814,85 +814,79 @@ describe('FragmentRefs', () => { expect(logs).toEqual([]); }); - // @gate enableFragmentRefs - it( - 'removes a capture listener registered with boolean when removed with options object', - async () => { - const fragmentRef = React.createRef(null); - function Test() { - return ( - -
- - ); - } - const root = ReactDOMClient.createRoot(container); - await act(() => { - root.render(); - }); + // @gate enableFragmentRefs + it('removes a capture listener registered with boolean when removed with options object', async () => { + const fragmentRef = React.createRef(null); + function Test() { + return ( + +
+ + ); + } + const root = ReactDOMClient.createRoot(container); + await act(() => { + root.render(); + }); - const logs = []; - function logCapture() { - logs.push('capture'); - } + const logs = []; + function logCapture() { + logs.push('capture'); + } - // Register with boolean `true` (capture phase) - fragmentRef.current.addEventListener('click', logCapture, true); - document.querySelector('#child-a').click(); - expect(logs).toEqual(['capture']); + // Register with boolean `true` (capture phase) + fragmentRef.current.addEventListener('click', logCapture, true); + document.querySelector('#child-a').click(); + expect(logs).toEqual(['capture']); - logs.length = 0; + logs.length = 0; - // Remove with equivalent options object {capture: true} - // Per DOM spec, these are identical - the listener MUST be removed - fragmentRef.current.removeEventListener('click', logCapture, { - capture: true, - }); - document.querySelector('#child-a').click(); - // Listener should have been removed - logs must remain empty - expect(logs).toEqual([]); - }, - ); + // Remove with equivalent options object {capture: true} + // Per DOM spec, these are identical - the listener MUST be removed + fragmentRef.current.removeEventListener('click', logCapture, { + capture: true, + }); + document.querySelector('#child-a').click(); + // Listener should have been removed - logs must remain empty + expect(logs).toEqual([]); + }); - // @gate enableFragmentRefs - it( - 'removes a capture listener registered with options object when removed with boolean', - async () => { - const fragmentRef = React.createRef(null); - function Test() { - return ( - -
- - ); - } - const root = ReactDOMClient.createRoot(container); - await act(() => { - root.render(); - }); + // @gate enableFragmentRefs + it('removes a capture listener registered with options object when removed with boolean', async () => { + const fragmentRef = React.createRef(null); + function Test() { + return ( + +
+ + ); + } + const root = ReactDOMClient.createRoot(container); + await act(() => { + root.render(); + }); - const logs = []; - function logCapture() { - logs.push('capture'); - } + const logs = []; + function logCapture() { + logs.push('capture'); + } - // Register with options object {capture: true} - fragmentRef.current.addEventListener('click', logCapture, { - capture: true, - }); - document.querySelector('#child-b').click(); - expect(logs).toEqual(['capture']); + // Register with options object {capture: true} + fragmentRef.current.addEventListener('click', logCapture, { + capture: true, + }); + document.querySelector('#child-b').click(); + expect(logs).toEqual(['capture']); - logs.length = 0; + logs.length = 0; - // Remove with boolean `true` - // Per DOM spec, these are identical - the listener MUST be removed - fragmentRef.current.removeEventListener('click', logCapture, true); - document.querySelector('#child-b').click(); - // Listener should have been removed - logs must remain empty - expect(logs).toEqual([]); - }, - ); + // Remove with boolean `true` + // Per DOM spec, these are identical - the listener MUST be removed + fragmentRef.current.removeEventListener('click', logCapture, true); + document.querySelector('#child-b').click(); + // Listener should have been removed - logs must remain empty + expect(logs).toEqual([]); + }); // @gate enableFragmentRefs it('applies event listeners to portaled children', async () => { @@ -2762,87 +2756,81 @@ describe('FragmentRefs', () => { }); // @gate enableFragmentRefs - it( - 'treats passive:true and passive:false as same listener per DOM spec', - async () => { - const fragmentRef = React.createRef(); - const root = ReactDOMClient.createRoot(container); - - await act(() => { - root.render( - -
- , - ); - }); - - const logs = []; - const handler = () => logs.push('fired'); - - const child = document.querySelector('#child'); - const spy = jest.spyOn(child, 'addEventListener'); - // Per DOM spec, listener identity is (type, callback, capture). - // passive is NOT part of the key, so these are the SAME listener. - fragmentRef.current.addEventListener('click', handler, {passive: false}); - // Second add is a no-op: same (type, callback, capture) identity. - fragmentRef.current.addEventListener('click', handler, {passive: true}); - expect(spy).toHaveBeenCalledTimes(1); - expect(spy).toHaveBeenCalledWith('click', handler, {passive: false}); + it('treats passive:true and passive:false as same listener per DOM spec', async () => { + const fragmentRef = React.createRef(); + const root = ReactDOMClient.createRoot(container); - document.querySelector('#child').click(); - // First handler fires once (second add was a no-op). - expect(logs).toEqual(['fired']); + await act(() => { + root.render( + +
+ , + ); + }); - // removeEventListener also ignores passive when matching - fragmentRef.current.removeEventListener('click', handler, { - passive: true, - }); + const logs = []; + const handler = () => logs.push('fired'); + + const child = document.querySelector('#child'); + const spy = jest.spyOn(child, 'addEventListener'); + // Per DOM spec, listener identity is (type, callback, capture). + // passive is NOT part of the key, so these are the SAME listener. + fragmentRef.current.addEventListener('click', handler, {passive: false}); + // Second add is a no-op: same (type, callback, capture) identity. + fragmentRef.current.addEventListener('click', handler, {passive: true}); + expect(spy).toHaveBeenCalledTimes(1); + expect(spy).toHaveBeenCalledWith('click', handler, {passive: false}); + + document.querySelector('#child').click(); + // First handler fires once (second add was a no-op). + expect(logs).toEqual(['fired']); + + // removeEventListener also ignores passive when matching + fragmentRef.current.removeEventListener('click', handler, { + passive: true, + }); - logs.length = 0; - document.querySelector('#child').click(); - expect(logs).toEqual([]); - }, - ); + logs.length = 0; + document.querySelector('#child').click(); + expect(logs).toEqual([]); + }); // @gate enableFragmentRefs - it( - 'removes a listener registered with passive:false when removed with passive:true', - async () => { - const fragmentRef = React.createRef(null); - function Test() { - return ( - <> -
- - ); - } - const root = ReactDOMClient.createRoot(container); - await act(() => { - root.render( - - - , - ); - }); - const logs = []; - function handler() { - logs.push('fired'); - } - // Register with passive: false - fragmentRef.current.addEventListener('click', handler, { - passive: false, - }); - document.querySelector('#child-x').click(); - expect(logs).toEqual(['fired']); - logs.length = 0; - // Remove with passive: true - per DOM spec, passive is NOT part of identity - // so this MUST remove the listener regardless of passive mismatch. - fragmentRef.current.removeEventListener('click', handler, { - passive: true, - }); - document.querySelector('#child-x').click(); - // Listener removed - no more invocations - expect(logs).toEqual([]); - }, - ); + it('removes a listener registered with passive:false when removed with passive:true', async () => { + const fragmentRef = React.createRef(null); + function Test() { + return ( + <> +
+ + ); + } + const root = ReactDOMClient.createRoot(container); + await act(() => { + root.render( + + + , + ); + }); + const logs = []; + function handler() { + logs.push('fired'); + } + // Register with passive: false + fragmentRef.current.addEventListener('click', handler, { + passive: false, + }); + document.querySelector('#child-x').click(); + expect(logs).toEqual(['fired']); + logs.length = 0; + // Remove with passive: true - per DOM spec, passive is NOT part of identity + // so this MUST remove the listener regardless of passive mismatch. + fragmentRef.current.removeEventListener('click', handler, { + passive: true, + }); + document.querySelector('#child-x').click(); + // Listener removed - no more invocations + expect(logs).toEqual([]); + }); }); });