diff --git a/src/__tests__/render.test.tsx b/src/__tests__/render.test.tsx index 145fc6248..088d476bd 100644 --- a/src/__tests__/render.test.tsx +++ b/src/__tests__/render.test.tsx @@ -3,8 +3,6 @@ import * as React from 'react'; import { View, Text, TextInput, Pressable, SafeAreaView } from 'react-native'; import { render, fireEvent, RenderAPI } from '..'; -type ConsoleLogMock = jest.Mock; - beforeEach(() => { jest.spyOn(console, 'warn').mockImplementation(() => {}); }); @@ -229,13 +227,6 @@ test('returns composite UNSAFE_root', () => { test('returns container', () => { const { container } = render(); - const mockCalls = (console.warn as any as ConsoleLogMock).mock.calls; - expect(mockCalls[0][0]).toMatchInlineSnapshot(` - "'container' property is deprecated and has been renamed to 'UNSAFE_root'. - - Consider using 'root' property which returns root host element." - `); - expect(container).toBeDefined(); // `View` composite component is returned. This behavior will break if we // start returning only host components. @@ -264,3 +255,11 @@ test('RenderAPI type', () => { render() as RenderAPI; expect(true).toBeTruthy(); }); + +test('returned output can be spread using rest operator', () => { + // Next line should not throw + // eslint-disable-next-line @typescript-eslint/no-unused-vars + const { rerender, ...rest } = render(); + expect(rest).toBeTruthy(); + expect(console.warn).not.toHaveBeenCalled(); +}); diff --git a/src/helpers/__tests__/getTextContent.test.tsx b/src/helpers/__tests__/getTextContent.test.tsx index c7e4bb243..e2f937c7d 100644 --- a/src/helpers/__tests__/getTextContent.test.tsx +++ b/src/helpers/__tests__/getTextContent.test.tsx @@ -5,7 +5,7 @@ import { getTextContent } from '../getTextContent'; test('getTextContent with simple content', () => { const view = render(Hello world); - expect(getTextContent(view.container)).toBe('Hello world'); + expect(getTextContent(view.root)).toBe('Hello world'); }); test('getTextContent with null element', () => { @@ -18,7 +18,7 @@ test('getTextContent with single nested content', () => { Hello world ); - expect(getTextContent(view.container)).toBe('Hello world'); + expect(getTextContent(view.root)).toBe('Hello world'); }); test('getTextContent with multiple nested content', () => { @@ -27,7 +27,7 @@ test('getTextContent with multiple nested content', () => { Hello world ); - expect(getTextContent(view.container)).toBe('Hello world'); + expect(getTextContent(view.root)).toBe('Hello world'); }); test('getTextContent with multiple number content', () => { @@ -36,7 +36,7 @@ test('getTextContent with multiple number content', () => { Hello world {100} ); - expect(getTextContent(view.container)).toBe('Hello world 100'); + expect(getTextContent(view.root)).toBe('Hello world 100'); }); test('getTextContent with multiple boolean content', () => { @@ -45,5 +45,5 @@ test('getTextContent with multiple boolean content', () => { Hello{false} {true}world ); - expect(getTextContent(view.container)).toBe('Hello world'); + expect(getTextContent(view.root)).toBe('Hello world'); }); diff --git a/src/render.tsx b/src/render.tsx index 9fac09ee3..045038109 100644 --- a/src/render.tsx +++ b/src/render.tsx @@ -113,11 +113,6 @@ function buildRenderResult( UNSAFE_root: instance, get container() { if (!getConfig().useBreakingChanges) { - // eslint-disable-next-line no-console - console.warn( - "'container' property is deprecated and has been renamed to 'UNSAFE_root'.\n\n" + - "Consider using 'root' property which returns root host element." - ); return instance; }