From 2483c63017b0d47c377bf67cb7c597e2d7e57897 Mon Sep 17 00:00:00 2001 From: Levi Buzolic Date: Sat, 8 Jun 2024 10:59:04 -0700 Subject: [PATCH] Add missing `remove` method to `addEventListener` Jest mocks (#44270) Summary: While writing some Jest tests, I noticed some instances of the following error: ``` Cannot read properties of undefined (reading 'remove') ``` Looks like there were two cases where the `{remove: () => {}}` return result was missing in the provided Jest mocks: - `AccessibilityInfo.addEventListener` - `Linking.addEventListener` ## Changelog: [GENERAL] [FIXED] - Added missing `remove` methods for `Linking.addEventListener` and `AccessibilityInfo.addEventListener` Jest mocks Pull Request resolved: https://github.com/facebook/react-native/pull/44270 Test Plan: N/A Reviewed By: christophpurrer Differential Revision: D58324784 Pulled By: robhogan fbshipit-source-id: f46bd55db2517413f14182ae1bb81068d8d1e9f6 --- packages/react-native/jest/setup.js | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/packages/react-native/jest/setup.js b/packages/react-native/jest/setup.js index 096623e349d338..4b3958f84b0083 100644 --- a/packages/react-native/jest/setup.js +++ b/packages/react-native/jest/setup.js @@ -137,7 +137,9 @@ jest .mock('../Libraries/Components/AccessibilityInfo/AccessibilityInfo', () => ({ __esModule: true, default: { - addEventListener: jest.fn(), + addEventListener: jest.fn(() => ({ + remove: jest.fn(), + })), announceForAccessibility: jest.fn(), isAccessibilityServiceEnabled: jest.fn(), isBoldTextEnabled: jest.fn(), @@ -200,7 +202,9 @@ jest openURL: jest.fn(), canOpenURL: jest.fn(() => Promise.resolve(true)), openSettings: jest.fn(), - addEventListener: jest.fn(), + addEventListener: jest.fn(() => ({ + remove: jest.fn(), + })), getInitialURL: jest.fn(() => Promise.resolve()), sendIntent: jest.fn(), }))