From ec5b94bf88632d98cce22af3f8bef8dec3a1024a Mon Sep 17 00:00:00 2001
From: Maciej Jastrzebski <mdjastrzebski@gmail.com>
Date: Mon, 18 May 2020 15:36:18 +0200
Subject: [PATCH 1/4] Removed global `debug` functions

---
 src/__tests__/debug.test.js      | 88 --------------------------------
 src/debug.js                     | 42 ---------------
 src/index.js                     |  2 -
 typings/__tests__/index.test.tsx | 12 -----
 typings/index.d.ts               | 14 -----
 5 files changed, 158 deletions(-)
 delete mode 100644 src/__tests__/debug.test.js
 delete mode 100644 src/debug.js

diff --git a/src/__tests__/debug.test.js b/src/__tests__/debug.test.js
deleted file mode 100644
index baf350275..000000000
--- a/src/__tests__/debug.test.js
+++ /dev/null
@@ -1,88 +0,0 @@
-// @flow
-/* eslint-disable no-console */
-import React from 'react';
-import { TouchableOpacity, Text } from 'react-native';
-import stripAnsi from 'strip-ansi';
-import { debug, render, fireEvent, flushMicrotasksQueue } from '..';
-import debugShallow from '../helpers/debugShallow';
-
-type ConsoleLogMock = JestMockFn<Array<string>, void>;
-
-function TextComponent({ text }) {
-  return <Text>{text}</Text>;
-}
-
-class Button extends React.Component<any, any> {
-  state = { counter: 0 };
-
-  onPress = async () => {
-    await Promise.resolve();
-
-    this.setState({ counter: 1 });
-    this.props.onPress();
-  };
-
-  render() {
-    return (
-      <TouchableOpacity onPress={this.onPress}>
-        <TextComponent text={`${this.props.text} ${this.state.counter}`} />
-      </TouchableOpacity>
-    );
-  }
-}
-
-test('debug', () => {
-  jest.spyOn(console, 'log').mockImplementation((x) => x);
-  const component = <Button onPress={jest.fn} text="Press me" />;
-  debug(component);
-
-  const output = ((console.log: any): ConsoleLogMock).mock.calls[0][0];
-
-  expect(stripAnsi(output)).not.toEqual(output);
-  expect(stripAnsi(output)).toMatchSnapshot();
-
-  ((console.log: any): ConsoleLogMock).mockReset();
-
-  debug(component, 'test message');
-
-  expect(console.log).toHaveBeenCalledWith('test message\n\n', output);
-});
-
-test('debug.shallow', () => {
-  expect(debug.shallow).toBe(debugShallow);
-});
-
-test('debug.deep', () => {
-  // $FlowFixMe
-  console.log = jest.fn();
-  const component = <Button onPress={jest.fn} text="Press me" />;
-  debug.deep(component);
-
-  const output = ((console.log: any): ConsoleLogMock).mock.calls[0][0];
-
-  expect(stripAnsi(output)).not.toEqual(output);
-  expect(stripAnsi(output)).toMatchSnapshot();
-
-  ((console.log: any): ConsoleLogMock).mockReset();
-
-  debug.deep(component, 'test message');
-
-  expect(console.log).toHaveBeenCalledWith('test message\n\n', output);
-});
-
-test('debug.deep async test', async () => {
-  // $FlowFixMe
-  console.log = jest.fn();
-  const { toJSON, getByName } = render(
-    <Button onPress={jest.fn} text="Press me" />
-  );
-
-  fireEvent.press(getByName('TouchableOpacity'));
-  await flushMicrotasksQueue();
-
-  debug.deep(toJSON());
-
-  const output = ((console.log: any): ConsoleLogMock).mock.calls[0][0];
-
-  expect(stripAnsi(output)).toMatchSnapshot();
-});
diff --git a/src/debug.js b/src/debug.js
deleted file mode 100644
index 807eaa571..000000000
--- a/src/debug.js
+++ /dev/null
@@ -1,42 +0,0 @@
-// @flow
-/* eslint-disable no-console */
-import * as React from 'react';
-import render from './render';
-import debugShallow from './helpers/debugShallow';
-import debugDeep from './helpers/debugDeep';
-import format from './helpers/format';
-
-/**
- * Log pretty-printed deep test component instance
- */
-function debugDeepElementOrInstance(
-  instance: React.Element<any> | ?ReactTestRendererJSON,
-  message?: any = ''
-) {
-  try {
-    // We're assuming React.Element<any> here and fallback to
-    // rendering ?ReactTestRendererJSON
-    // $FlowFixMe
-    const { toJSON } = render(instance);
-    if (message) {
-      console.log(`${message}\n\n`, format(toJSON()));
-    } else {
-      console.log(format(toJSON()));
-    }
-  } catch (e) {
-    // $FlowFixMe
-    debugDeep(instance);
-  }
-}
-
-function debug(
-  instance: ReactTestInstance | React.Element<any>,
-  message?: any
-) {
-  return debugShallow(instance, message);
-}
-
-debug.shallow = debugShallow;
-debug.deep = debugDeepElementOrInstance;
-
-export default debug;
diff --git a/src/index.js b/src/index.js
index 2e7ac9f20..1d73981f5 100644
--- a/src/index.js
+++ b/src/index.js
@@ -1,7 +1,6 @@
 // @flow
 import act from './act';
 import cleanup from './cleanup';
-import debug from './debug';
 import fireEvent from './fireEvent';
 import flushMicrotasksQueue from './flushMicrotasksQueue';
 import render from './render';
@@ -11,7 +10,6 @@ import within from './within';
 
 export { act };
 export { cleanup };
-export { debug };
 export { fireEvent };
 export { flushMicrotasksQueue };
 export { render };
diff --git a/typings/__tests__/index.test.tsx b/typings/__tests__/index.test.tsx
index 3caf5f299..209102b3e 100644
--- a/typings/__tests__/index.test.tsx
+++ b/typings/__tests__/index.test.tsx
@@ -6,7 +6,6 @@ import {
   fireEvent,
   shallow,
   flushMicrotasksQueue,
-  debug,
   waitForElement,
   act,
   within,
@@ -251,17 +250,6 @@ const shallowTree: { output: React.ReactElement<any> } = shallow(
 
 const waitForFlush: Promise<any> = flushMicrotasksQueue();
 
-// debug API
-debug(<TestComponent />);
-debug(<TestComponent />, 'message');
-debug(getByNameString);
-debug(getByNameString, 'message');
-debug.shallow(<TestComponent />);
-debug.shallow(<TestComponent />, 'message');
-debug.deep(<TestComponent />);
-debug.deep(<TestComponent />, 'message');
-debug.deep(tree.toJSON());
-
 const waitBy: Promise<ReactTestInstance> = waitForElement<ReactTestInstance>(
   () => tree.getByName('View')
 );
diff --git a/typings/index.d.ts b/typings/index.d.ts
index 0a5bb8d2f..ef4895f1a 100644
--- a/typings/index.d.ts
+++ b/typings/index.d.ts
@@ -230,19 +230,6 @@ export type WaitForElementFunction = <T = any>(
   interval?: number
 ) => Promise<T>;
 
-export type DebugFunction = (
-  instance: ReactTestInstance | React.ReactElement<any>,
-  message?: string
-) => void;
-
-export type DebugAPI = DebugFunction & {
-  shallow: DebugFunction;
-  deep: (
-    instance: React.ReactElement<any> | ReactTestRendererJSON | null,
-    message?: string
-  ) => void;
-};
-
 export declare const render: (
   component: React.ReactElement<any>,
   options?: RenderOptions
@@ -252,7 +239,6 @@ export declare const shallow: <P = {}>(
 ) => { output: React.ReactElement<P> };
 export declare const flushMicrotasksQueue: () => Promise<any>;
 export declare const cleanup: () => void;
-export declare const debug: DebugAPI;
 export declare const fireEvent: FireEventAPI;
 export declare const waitForElement: WaitForElementFunction;
 export declare const act: (callback: () => void) => Thenable;

From a7cbddff6ffdcf951132395b97f9972a9fcb4bdf Mon Sep 17 00:00:00 2001
From: Maciej Jastrzebski <mdjastrzebski@gmail.com>
Date: Mon, 18 May 2020 15:39:24 +0200
Subject: [PATCH 2/4] Removed global `debug` from docs

---
 website/docs/API.md | 57 ---------------------------------------------
 1 file changed, 57 deletions(-)

diff --git a/website/docs/API.md b/website/docs/API.md
index 62176f09b..5a7bca6c8 100644
--- a/website/docs/API.md
+++ b/website/docs/API.md
@@ -370,63 +370,6 @@ Use cases for scoped queries include:
 - queries scoped to a single item inside a FlatList containing many items
 - queries scoped to a single screen in tests involving screen transitions (e.g. with react-navigation)
 
-## `debug`
-
-- [`Example code`](https://github.com/callstack/react-native-testing-library/blob/master/src/__tests__/debug.test.js)
-
-Log prettified shallowly rendered component or test instance (just like snapshot) to stdout.
-
-```jsx
-import { debug } from 'react-native-testing-library';
-
-debug(<Component />);
-debug.shallow(<Component />); // an alias for `debug`
-```
-
-logs:
-
-```jsx
-<TouchableOpacity
-  activeOpacity={0.2}
-  onPress={[Function bound fn]}
->
-  <TextComponent
-    text="Press me"
-  />
-</TouchableOpacity>
-```
-
-There's also `debug.deep` that renders deeply to stdout.
-
-```jsx
-import { debug } from 'react-native-testing-library';
-
-debug.deep(<Component />);
-debug.deep(toJSON(), 'actually debug JSON too'); // useful when Component state changes
-```
-
-logs:
-
-```jsx
-<View
-  accessible={true}
-  isTVSelectable={true}
-  onResponderGrant={[Function bound touchableHandleResponderGrant]}
-  // ... more props
-  style={
-    Object {
-      \\"opacity\\": 1,
-    }
-  }
->
-  <Text>
-    Press me
-  </Text>
-</View>
-```
-
-Optionally you can provide a string message as a second argument to `debug`, which will be displayed right after the component.
-
 ## `flushMicrotasksQueue`
 
 Waits for microtasks queue to flush. Useful if you want to wait for some promises with `async/await`.

From 24a9782537904cfa9e2a28d357389023c4ca5c75 Mon Sep 17 00:00:00 2001
From: Maciej Jastrzebski <mdjastrzebski@gmail.com>
Date: Mon, 18 May 2020 15:41:31 +0200
Subject: [PATCH 3/4] Cleanup

---
 .../__snapshots__/debug.test.js.snap          | 57 -------------------
 1 file changed, 57 deletions(-)
 delete mode 100644 src/__tests__/__snapshots__/debug.test.js.snap

diff --git a/src/__tests__/__snapshots__/debug.test.js.snap b/src/__tests__/__snapshots__/debug.test.js.snap
deleted file mode 100644
index b1732ecc6..000000000
--- a/src/__tests__/__snapshots__/debug.test.js.snap
+++ /dev/null
@@ -1,57 +0,0 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
-
-exports[`debug 1`] = `
-"<ForwardRef
-  onPress={[Function _callee]}
->
-  <TextComponent
-    text=\\"Press me 0\\"
-  />
-</ForwardRef>"
-`;
-
-exports[`debug.deep 1`] = `
-"<View
-  accessible={true}
-  focusable={true}
-  onClick={[Function onClick]}
-  onResponderGrant={[Function onResponderGrant]}
-  onResponderMove={[Function onResponderMove]}
-  onResponderRelease={[Function onResponderRelease]}
-  onResponderTerminate={[Function onResponderTerminate]}
-  onResponderTerminationRequest={[Function onResponderTerminationRequest]}
-  onStartShouldSetResponder={[Function onStartShouldSetResponder]}
-  style={
-    Object {
-      \\"opacity\\": 1,
-    }
-  }
->
-  <Text>
-    Press me 0
-  </Text>
-</View>"
-`;
-
-exports[`debug.deep async test 1`] = `
-"<View
-  accessible={true}
-  focusable={true}
-  onClick={[Function onClick]}
-  onResponderGrant={[Function onResponderGrant]}
-  onResponderMove={[Function onResponderMove]}
-  onResponderRelease={[Function onResponderRelease]}
-  onResponderTerminate={[Function onResponderTerminate]}
-  onResponderTerminationRequest={[Function onResponderTerminationRequest]}
-  onStartShouldSetResponder={[Function onStartShouldSetResponder]}
-  style={
-    Object {
-      \\"opacity\\": 1,
-    }
-  }
->
-  <Text>
-    Press me 1
-  </Text>
-</View>"
-`;

From 2b2af083f517e3125bda4872896dc17191f43b8b Mon Sep 17 00:00:00 2001
From: Maciej Jastrzebski <mdjastrzebski@gmail.com>
Date: Mon, 18 May 2020 15:54:29 +0200
Subject: [PATCH 4/4] Added migration section

---
 website/docs/Migration20.md | 12 ++++++++++++
 1 file changed, 12 insertions(+)
 create mode 100644 website/docs/Migration20.md

diff --git a/website/docs/Migration20.md b/website/docs/Migration20.md
new file mode 100644
index 000000000..2570f5547
--- /dev/null
+++ b/website/docs/Migration20.md
@@ -0,0 +1,12 @@
+---
+id: migration20
+title: Migration to 2.0
+---
+
+This guides describes major steps involved in migrating your testing code from using React Native Testing Library version `1.x` to version `2.0`.
+
+## Removed global `debug` function
+
+Global debug function has been removed in favor of `debug()` method returned from `render()` function.
+
+## Some `byTestId` queries behavior changes