Skip to content

Commit

Permalink
Remove some automocks from tests
Browse files Browse the repository at this point in the history
Summary:
They are bad :D

And they affect us when trying to migrate to the babel runtime helpers

Reviewed By: pvdz

Differential Revision: D9829462

fbshipit-source-id: 15240a56e707e13775d57714646e4960cfe202df
  • Loading branch information
rafeca authored and facebook-github-bot committed Sep 18, 2018
1 parent ade04d0 commit c58fb3b
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 8 deletions.
7 changes: 1 addition & 6 deletions Libraries/BatchedBridge/__tests__/NativeModules-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,12 +10,7 @@

'use strict';

jest
.enableAutomock()
.unmock('BatchedBridge')
.unmock('defineLazyObjectProperty')
.unmock('MessageQueue')
.unmock('NativeModules');
jest.unmock('NativeModules');

let BatchedBridge;
let NativeModules;
Expand Down
2 changes: 1 addition & 1 deletion Libraries/Interaction/__tests__/InteractionMixin-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@

'use strict';

jest.enableAutomock().unmock('InteractionMixin');
jest.mock('InteractionManager');

describe('InteractionMixin', () => {
let InteractionManager;
Expand Down
2 changes: 1 addition & 1 deletion Libraries/WebSocket/__tests__/WebSocket-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@

'use strict';

jest.enableAutomock().unmock('WebSocket');
jest.mock('NativeEventEmitter');
jest.setMock('NativeModules', {
WebSocketModule: {
connect: () => {},
Expand Down

0 comments on commit c58fb3b

Please sign in to comment.