diff --git a/Libraries/LogBox/LogBox.js b/Libraries/LogBox/LogBox.js index 6c489e3e4e..e2754504c1 100644 --- a/Libraries/LogBox/LogBox.js +++ b/Libraries/LogBox/LogBox.js @@ -43,7 +43,7 @@ if (__DEV__) { }, ignoreAllLogs: (value?: ?boolean): void => { - LogBoxData.setDisabled(!!value); + LogBoxData.setDisabled(value == null ? true : value); }, uninstall: (): void => { diff --git a/Libraries/LogBox/__tests__/LogBox-test.js b/Libraries/LogBox/__tests__/LogBox-test.js index 974f1cca30..56857f8ba2 100644 --- a/Libraries/LogBox/__tests__/LogBox-test.js +++ b/Libraries/LogBox/__tests__/LogBox-test.js @@ -69,6 +69,30 @@ describe('LogBox', () => { expect(LogBoxData.isDisabled()).toBe(true); }); + it('will not ignore logs for `ignoreAllLogs(false)`', () => { + expect(LogBoxData.isDisabled()).toBe(false); + + LogBox.install(); + + expect(LogBoxData.isDisabled()).toBe(false); + + LogBox.ignoreAllLogs(false); + + expect(LogBoxData.isDisabled()).toBe(false); + }); + + it('will ignore logs for `ignoreAllLogs()`', () => { + expect(LogBoxData.isDisabled()).toBe(false); + + LogBox.install(); + + expect(LogBoxData.isDisabled()).toBe(false); + + LogBox.ignoreAllLogs(); + + expect(LogBoxData.isDisabled()).toBe(true); + }); + it('registers warnings', () => { jest.mock('../Data/LogBoxData');