Skip to content

Commit 7fb3a92

Browse files
timwangdevfacebook-github-bot
authored andcommitted
Fix Jest mocks for NetInfo module.
Summary: Jest mock for `NetInfo.getConnectionInfo()` is missing. Tests are fail with error message: ``` TypeError: _reactNative.NetInfo.getConnectionInfo is not a function ``` And cleaned code in this files for unified code styles. Run test with usage of `NetInfo.getConnectionInfo()` [GENERAL] [BUGFIX] [jest/setup.js] - Fix Jest mocks for NetInfo module Closes #16722 Differential Revision: D6298133 Pulled By: hramos fbshipit-source-id: 589cf6fed93965e7a09823015f2793f5c68a9c3b
1 parent 58c3bc4 commit 7fb3a92

File tree

1 file changed

+13
-5
lines changed

1 file changed

+13
-5
lines changed

jest/setup.js

+13-5
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,7 @@ const mockNativeModules = {
162162
},
163163
ImageLoader: {
164164
getSize: jest.fn(
165-
(url) => new Promise(() => ({width: 320, height: 240}))
165+
(url) => Promise.resolve({width: 320, height: 240})
166166
),
167167
prefetchImage: jest.fn(),
168168
},
@@ -179,11 +179,11 @@ const mockNativeModules = {
179179
Linking: {
180180
openURL: jest.fn(),
181181
canOpenURL: jest.fn(
182-
() => new Promise((resolve) => resolve(true))
182+
() => Promise.resolve(true)
183183
),
184184
addEventListener: jest.fn(),
185185
getInitialURL: jest.fn(
186-
() => new Promise((resolve) => resolve())
186+
() => Promise.resolve()
187187
),
188188
removeEventListener: jest.fn(),
189189
},
@@ -195,15 +195,23 @@ const mockNativeModules = {
195195
ModalFullscreenViewManager: {},
196196
NetInfo: {
197197
fetch: jest.fn(
198-
() => new Promise((resolve) => resolve())
198+
() => Promise.resolve()
199+
),
200+
getConnectionInfo: jest.fn(
201+
() => Promise.resolve()
199202
),
200203
addEventListener: jest.fn(),
204+
removeEventListener: jest.fn(),
201205
isConnected: {
202206
fetch: jest.fn(
203-
() => new Promise((resolve) => resolve())
207+
() => Promise.resolve()
204208
),
205209
addEventListener: jest.fn(),
210+
removeEventListener: jest.fn(),
206211
},
212+
isConnectionExpensive: jest.fn(
213+
() => Promise.resolve()
214+
),
207215
},
208216
Networking: {
209217
sendRequest: jest.fn(),

0 commit comments

Comments
 (0)