From 0da9e32a5a2a0ab6aefaecf9623a66e22ce519eb Mon Sep 17 00:00:00 2001 From: Jan Kassens Date: Mon, 4 Nov 2024 15:23:17 -0500 Subject: [PATCH] Followup remove dead test code from #30346 I missed that this was a constant false check when making the broader cleanup changes in https://github.com/facebook/react/pull/30346 --- packages/react/src/__tests__/ReactElementClone-test.js | 6 ------ 1 file changed, 6 deletions(-) diff --git a/packages/react/src/__tests__/ReactElementClone-test.js b/packages/react/src/__tests__/ReactElementClone-test.js index 435af8e1180..4e7d0c5ca2a 100644 --- a/packages/react/src/__tests__/ReactElementClone-test.js +++ b/packages/react/src/__tests__/ReactElementClone-test.js @@ -273,9 +273,6 @@ describe('ReactElementClone', () => { if (gate(flags => flags.disableStringRefs)) { expect(component.childRef).toEqual({current: null}); expect(component.parentRef.current.xyzRef.current.tagName).toBe('SPAN'); - } else if (gate(flags => false)) { - expect(component.childRef).toEqual({current: null}); - expect(component.parentRef.current.xyzRef.current.tagName).toBe('SPAN'); } else if (gate(flags => !flags.disableStringRefs)) { expect(component.childRef).toEqual({current: null}); expect(component.parentRef.current.xyzRef.current.tagName).toBe('SPAN'); @@ -413,9 +410,6 @@ describe('ReactElementClone', () => { {withoutStack: true}, ); expect(clone.props).toEqual({foo: 'ef', ref: '34'}); - } else if (gate(flags => false)) { - expect(clone.ref).toBe(element.ref); - expect(clone.props).toEqual({foo: 'ef'}); } else if (gate(flags => !flags.disableStringRefs)) { expect(() => { expect(clone.ref).toBe(element.ref);