From ecd6327bb1f0673866d77e211a1d42d0cebc5e97 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?S=C3=B6ren=20Wrede?= Date: Tue, 10 May 2022 10:58:58 +0200 Subject: [PATCH] Docs: Git.io deprecation - Replace links (#40734) --- .github/workflows/rnmobile-android-runner.yml | 2 +- .../commands/test/fixtures/pull-requests.json | 2 +- .../src/components/inserter/index.native.js | 2 +- packages/block-editor/src/hooks/anchor.js | 2 +- .../block-library/src/block/test/edit.native.js | 2 +- .../block-library/src/cover/controls.native.js | 2 +- .../cover/focal-point-settings-button.native.js | 2 +- .../block-library/src/image/test/edit.native.js | 14 +++++++------- .../src/focal-point-picker/index.native.js | 6 +++--- .../navigation-screen.native.js | 2 +- .../test/navigation-container.native.js | 2 +- .../src/mobile/bottom-sheet/index.native.js | 2 +- packages/components/src/sandbox/index.native.js | 2 +- .../components/src/tooltip/test/index.native.js | 2 +- .../components/editor-help/test/index.native.js | 2 +- packages/react-native-editor/ios/Podfile | 2 +- packages/react-native-editor/metro.config.js | 4 ++-- packages/react-native-editor/src/setup.js | 4 ++-- .../react-native-editor/src/test/index.test.js | 10 +++++----- test/native/helpers.js | 2 +- test/native/setup.js | 2 +- 21 files changed, 35 insertions(+), 35 deletions(-) diff --git a/.github/workflows/rnmobile-android-runner.yml b/.github/workflows/rnmobile-android-runner.yml index 4de748dfa42a13..226ed1a1ba864d 100644 --- a/.github/workflows/rnmobile-android-runner.yml +++ b/.github/workflows/rnmobile-android-runner.yml @@ -48,7 +48,7 @@ jobs: - uses: reactivecircus/android-emulator-runner@5de26e4bd23bf523e8a4b7f077df8bfb8e52b50e # v2.19.1 with: api-level: 28 - emulator-build: 7425822 # https://git.io/JE3jX + emulator-build: 7425822 # https://github.com/ReactiveCircus/android-emulator-runner/issues/160#issuecomment-868615730 profile: pixel_xl script: npm run native test:e2e:android:local ${{ matrix.native-test-name }} diff --git a/bin/plugin/commands/test/fixtures/pull-requests.json b/bin/plugin/commands/test/fixtures/pull-requests.json index a6729cb6e62fda..5eeed9ebe31869 100644 --- a/bin/plugin/commands/test/fixtures/pull-requests.json +++ b/bin/plugin/commands/test/fixtures/pull-requests.json @@ -6628,7 +6628,7 @@ "diff_url": "https://github.com/WordPress/gutenberg/pull/33788.diff", "patch_url": "https://github.com/WordPress/gutenberg/pull/33788.patch" }, - "body": "## Description\r\nReact Native added a `PROJECT_ROOT` [configuration to better support monorepos](https://git.io/JBwVr). Prior to setting the `PROJECT_ROOT` configuration, building the Demo app in Xcode threw errors related to the inability to find the entry `index.js` file, as it was searching in the root directory.\r\n\r\n
Error Text\r\n\r\n```\r\nerror The resource `/Users/username/Sites/a8c/gutenberg/index.js` was not found.\r\nError: The resource `/Users/username/Sites/a8c/gutenberg/index.js` was not found.\r\n at /Users/username/Sites/a8c/gutenberg/node_modules/metro/src/IncrementalBundler.js:457:26\r\n at gotStat (node:fs:2627:21)\r\n at FSReqCallback.oncomplete (node:fs:195:21)\r\ninfo Run CLI with --verbose flag for more details.\r\nCommand PhaseScriptExecution failed with a nonzero exit code\r\n```\r\n\r\n
\r\n\r\n
Error Screenshot\r\n\r\n\"xcode-demo-app-error\"\r\n\r\n
\r\n\r\n## How has this been tested?\r\n1. Check out `trunk` branch. \r\n2. `npm run native start:reset`\r\n3. Launch Xcode and build the Demo app to a physical device. \r\n4. ⚠️ **Unexpected:** Notice the build failure. \r\n5. Check out this PR branch. \r\n6. `npm run native start:reset`\r\n7. Launch Xcode and build the Demo app to a physical device. \r\n8. ℹ️ **Expected:** The build succeeds and runs on the device. \r\n\r\n## Screenshots \r\n\r\n## Types of changes\r\nBug fix\r\n\r\n## Checklist:\r\n- [x] My code is tested.\r\n- [x] My code follows the WordPress code style. \r\n- [x] My code follows the accessibility standards. \r\n- [ ] I've tested my changes with keyboard and screen readers. \r\n- [x] My code has proper inline documentation. \r\n- [x] I've included developer documentation if appropriate. \r\n- [x] I've updated all React Native files affected by any refactorings/renamings in this PR (please manually search all `*.native.js` files for terms that need renaming or removal). \r\n", + "body": "## Description\r\nReact Native added a `PROJECT_ROOT` [configuration to better support monorepos](https://github.com/facebook/react-native/commit/9ccde378b6e6379df61f9d968be6346ca6be7ead). Prior to setting the `PROJECT_ROOT` configuration, building the Demo app in Xcode threw errors related to the inability to find the entry `index.js` file, as it was searching in the root directory.\r\n\r\n
Error Text\r\n\r\n```\r\nerror The resource `/Users/username/Sites/a8c/gutenberg/index.js` was not found.\r\nError: The resource `/Users/username/Sites/a8c/gutenberg/index.js` was not found.\r\n at /Users/username/Sites/a8c/gutenberg/node_modules/metro/src/IncrementalBundler.js:457:26\r\n at gotStat (node:fs:2627:21)\r\n at FSReqCallback.oncomplete (node:fs:195:21)\r\ninfo Run CLI with --verbose flag for more details.\r\nCommand PhaseScriptExecution failed with a nonzero exit code\r\n```\r\n\r\n
\r\n\r\n
Error Screenshot\r\n\r\n\"xcode-demo-app-error\"\r\n\r\n
\r\n\r\n## How has this been tested?\r\n1. Check out `trunk` branch. \r\n2. `npm run native start:reset`\r\n3. Launch Xcode and build the Demo app to a physical device. \r\n4. ⚠️ **Unexpected:** Notice the build failure. \r\n5. Check out this PR branch. \r\n6. `npm run native start:reset`\r\n7. Launch Xcode and build the Demo app to a physical device. \r\n8. ℹ️ **Expected:** The build succeeds and runs on the device. \r\n\r\n## Screenshots \r\n\r\n## Types of changes\r\nBug fix\r\n\r\n## Checklist:\r\n- [x] My code is tested.\r\n- [x] My code follows the WordPress code style. \r\n- [x] My code follows the accessibility standards. \r\n- [ ] I've tested my changes with keyboard and screen readers. \r\n- [x] My code has proper inline documentation. \r\n- [x] I've included developer documentation if appropriate. \r\n- [x] I've updated all React Native files affected by any refactorings/renamings in this PR (please manually search all `*.native.js` files for terms that need renaming or removal). \r\n", "performed_via_github_app": null }, { diff --git a/packages/block-editor/src/components/inserter/index.native.js b/packages/block-editor/src/components/inserter/index.native.js index 0d87bf1b58960a..04e0d302e2338a 100644 --- a/packages/block-editor/src/components/inserter/index.native.js +++ b/packages/block-editor/src/components/inserter/index.native.js @@ -67,7 +67,7 @@ const defaultRenderToggle = ( { extraProps={ { hint: __( 'Double tap to add a block' ), // testID is present to disambiguate this element for native UI tests. It's not - // usually required for components. See: https://git.io/JeQ7G. + // usually required for components. See: https://github.com/WordPress/gutenberg/pull/18832#issuecomment-561411389. testID: 'add-block-button', onLongPress, } } diff --git a/packages/block-editor/src/hooks/anchor.js b/packages/block-editor/src/hooks/anchor.js index 27938c2e395d6e..4be1719ec90925 100644 --- a/packages/block-editor/src/hooks/anchor.js +++ b/packages/block-editor/src/hooks/anchor.js @@ -117,7 +117,7 @@ export const withInspectorControl = createHigherOrderComponent( * We plan to remove scoping anchors to 'core/heading' to support * anchors for all eligble blocks. Additionally we plan to explore * leveraging InspectorAdvancedControls instead of a custom - * PanelBody title. https://git.io/Jtcov + * PanelBody title. https://github.com/WordPress/gutenberg/issues/28363 */ } { ! isWeb && props.name === 'core/heading' && ( diff --git a/packages/block-library/src/block/test/edit.native.js b/packages/block-library/src/block/test/edit.native.js index a93bad623c6270..fc869c20651f2a 100644 --- a/packages/block-library/src/block/test/edit.native.js +++ b/packages/block-library/src/block/test/edit.native.js @@ -146,7 +146,7 @@ describe( 'Reusable block', () => { } ); // Skipped until `pointerEvents: 'none'` no longer erroneously prevents - // triggering `onLayout*` on the element: https://git.io/JSHZt. + // triggering `onLayout*` on the element: https://github.com/callstack/react-native-testing-library/issues/897. it.skip( 'renders block content', async () => { // We have to use different ids because entities are cached in memory. const id = 4; diff --git a/packages/block-library/src/cover/controls.native.js b/packages/block-library/src/cover/controls.native.js index 6b967548a23d45..90403c98b62a70 100644 --- a/packages/block-library/src/cover/controls.native.js +++ b/packages/block-library/src/cover/controls.native.js @@ -217,7 +217,7 @@ function Controls( { } ); setDisplayPlaceholder( false ); // Avoid invisible, paused video on Android, presumably - // related to https://git.io/Jt6Dr + // related to https://github.com/react-native-video/react-native-video/issues/1979 videoRef?.current.seek( 0 ); } } ref={ videoRef } diff --git a/packages/block-library/src/cover/focal-point-settings-button.native.js b/packages/block-library/src/cover/focal-point-settings-button.native.js index d9451831a6a2f5..0cf6dd9f3c8a61 100644 --- a/packages/block-library/src/cover/focal-point-settings-button.native.js +++ b/packages/block-library/src/cover/focal-point-settings-button.native.js @@ -41,7 +41,7 @@ function FocalPointSettingsButton( { > { /* * Wrapper View element used around Icon as workaround for SVG opacity - * issue: https://git.io/JtuXD + * issue: https://github.com/react-native-svg/react-native-svg/issues/1345 */ } diff --git a/packages/block-library/src/image/test/edit.native.js b/packages/block-library/src/image/test/edit.native.js index d41cc2ec89207c..214c953be323b2 100644 --- a/packages/block-library/src/image/test/edit.native.js +++ b/packages/block-library/src/image/test/edit.native.js @@ -87,7 +87,7 @@ describe( 'Image Block', () => { fireEvent.press( screen.getByA11yLabel( /Image Block/ ) ); // Awaiting navigation event seemingly required due to React Navigation bug - // https://git.io/Ju35Z + // https://github.com/react-navigation/react-navigation/issues/9701 await act( () => fireEvent.press( screen.getByA11yLabel( 'Open Settings' ) ) ); @@ -113,7 +113,7 @@ describe( 'Image Block', () => { fireEvent.press( screen.getByA11yLabel( /Image Block/ ) ); // Awaiting navigation event seemingly required due to React Navigation bug - // https://git.io/Ju35Z + // https://github.com/react-navigation/react-navigation/issues/9701 await act( () => fireEvent.press( screen.getByA11yLabel( 'Open Settings' ) ) ); @@ -139,7 +139,7 @@ describe( 'Image Block', () => { fireEvent.press( screen.getByA11yLabel( /Image Block/ ) ); // Awaiting navigation event seemingly required due to React Navigation bug - // https://git.io/Ju35Z + // https://github.com/react-navigation/react-navigation/issues/9701 await act( () => fireEvent.press( screen.getByA11yLabel( 'Open Settings' ) ) ); @@ -172,7 +172,7 @@ describe( 'Image Block', () => { fireEvent.press( screen.getByA11yLabel( /Image Block/ ) ); // Awaiting navigation event seemingly required due to React Navigation bug - // https://git.io/Ju35Z + // https://github.com/react-navigation/react-navigation/issues/9701 await act( () => fireEvent.press( screen.getByA11yLabel( 'Open Settings' ) ) ); @@ -212,7 +212,7 @@ describe( 'Image Block', () => { fireEvent.press( screen.getByA11yLabel( /Image Block/ ) ); // Awaiting navigation event seemingly required due to React Navigation bug - // https://git.io/Ju35Z + // https://github.com/react-navigation/react-navigation/issues/9701 await act( () => fireEvent.press( screen.getByA11yLabel( 'Open Settings' ) ) ); @@ -239,7 +239,7 @@ describe( 'Image Block', () => { const settingsButton = screen.getByA11yLabel( 'Open Settings' ); // Awaiting navigation event seemingly required due to React Navigation bug - // https://git.io/Ju35Z + // https://github.com/react-navigation/react-navigation/issues/9701 await act( () => fireEvent.press( settingsButton ) ); const linkTargetButton = screen.getByText( 'Open in new tab' ); @@ -270,7 +270,7 @@ describe( 'Image Block', () => { const settingsButton = screen.getByA11yLabel( 'Open Settings' ); // Awaiting navigation event seemingly required due to React Navigation bug - // https://git.io/Ju35Z + // https://github.com/react-navigation/react-navigation/issues/9701 await act( () => fireEvent.press( settingsButton ) ); const linkTargetButton = screen.getByText( 'Open in new tab' ); diff --git a/packages/components/src/focal-point-picker/index.native.js b/packages/components/src/focal-point-picker/index.native.js index 2b61f9f5300237..61f9c48c218f44 100644 --- a/packages/components/src/focal-point-picker/index.native.js +++ b/packages/components/src/focal-point-picker/index.native.js @@ -105,7 +105,7 @@ function FocalPointPicker( props ) { // coordinates to workaround a bug affecting Android's PanResponder. // Specifically, dragging the handle outside the bounds of the image // results in inaccurate locationX and locationY coordinates to be - // reported. https://git.io/JtWmi + // reported. https://github.com/facebook/react-native/issues/15290#issuecomment-435494944 const x = pageX - locationPageOffsetX; const y = pageY - locationPageOffsetY; onChange( { @@ -116,7 +116,7 @@ function FocalPointPicker( props ) { } ); // Slider (child of RangeCell) is uncontrolled, so we must increment a // key to re-mount and sync the pan gesture values to the sliders - // https://git.io/JTe4A + // https://github.com/callstack/react-native-slider/tree/v3.0.3#value setSliderKey( ( prevState ) => prevState + 1 ); }, } ), @@ -193,7 +193,7 @@ function FocalPointPicker( props ) { setVideoNaturalSize( { height, width } ); setDisplayPlaceholder( false ); // Avoid invisible, paused video on Android, presumably related to - // https://git.io/Jt6Dr + // https://github.com/react-native-video/react-native-video/issues/1979 videoRef?.current.seek( 0 ); }; const onXCoordinateChange = ( x ) => diff --git a/packages/components/src/mobile/bottom-sheet/bottom-sheet-navigation/navigation-screen.native.js b/packages/components/src/mobile/bottom-sheet/bottom-sheet-navigation/navigation-screen.native.js index 0adb3d45854598..0723e25ab2a898 100644 --- a/packages/components/src/mobile/bottom-sheet/bottom-sheet-navigation/navigation-screen.native.js +++ b/packages/components/src/mobile/bottom-sheet/bottom-sheet-navigation/navigation-screen.native.js @@ -69,7 +69,7 @@ const BottomSheetNavigationScreen = ( { * Ideally, we refactor onHandleHardwareButtonPress to manage multiple * callbacks triggered based upon which screen is currently active. * - * Related: https://git.io/JD2no + * Related: https://github.com/WordPress/gutenberg/pull/36328#discussion_r768897546 */ }, [] ) ); diff --git a/packages/components/src/mobile/bottom-sheet/bottom-sheet-navigation/test/navigation-container.native.js b/packages/components/src/mobile/bottom-sheet/bottom-sheet-navigation/test/navigation-container.native.js index f3921c831edc21..2b4d7cc5743f13 100644 --- a/packages/components/src/mobile/bottom-sheet/bottom-sheet-navigation/test/navigation-container.native.js +++ b/packages/components/src/mobile/bottom-sheet/bottom-sheet-navigation/test/navigation-container.native.js @@ -108,7 +108,7 @@ it( 'animates height transitioning from full-screen to non-full-screen', async ( ); // Navigate to screen 1 fireEvent.press( - // Use custom waitFor due to https://git.io/JYYGE + // Use custom waitFor due to https://github.com/callstack/react-native-testing-library/issues/379 await waitFor( () => screen.getByText( /test-screen-2/ ) ) ); // Await navigation screen to allow async state updates to complete diff --git a/packages/components/src/mobile/bottom-sheet/index.native.js b/packages/components/src/mobile/bottom-sheet/index.native.js index d7977ba49bbab1..aff2f6922ddcd8 100644 --- a/packages/components/src/mobile/bottom-sheet/index.native.js +++ b/packages/components/src/mobile/bottom-sheet/index.native.js @@ -148,7 +148,7 @@ class BottomSheet extends Component { // TODO: Reinstate animations, possibly replacing `LayoutAnimation` with // more nuanced `Animated` usage or replacing our custom `BottomSheet` // with `@gorhom/bottom-sheet`. This animation was disabled to avoid a - // preexisting bug: https://git.io/JMPCV + // preexisting bug: https://github.com/WordPress/gutenberg/issues/30562 // this.performRegularLayoutAnimation( { // useLastLayoutAnimation: false, // } );. diff --git a/packages/components/src/sandbox/index.native.js b/packages/components/src/sandbox/index.native.js index 2b3e824197787d..844997779e3e6a 100644 --- a/packages/components/src/sandbox/index.native.js +++ b/packages/components/src/sandbox/index.native.js @@ -110,7 +110,7 @@ const style = ` /** * Add responsiveness to embeds with aspect ratios. * - * These styles have been copied from the web version (https://git.io/JEFcX) and + * These styles have been copied from the web version (https://github.com/WordPress/gutenberg/blob/7901895ca20cf61e402925e31571d659dab64721/packages/block-library/src/embed/style.scss#L42-L89) and * adapted for the native version. */ .wp-has-aspect-ratio.wp-block-embed__wrapper::before { diff --git a/packages/components/src/tooltip/test/index.native.js b/packages/components/src/tooltip/test/index.native.js index 77254fef8e09da..56ff914323d640 100644 --- a/packages/components/src/tooltip/test/index.native.js +++ b/packages/components/src/tooltip/test/index.native.js @@ -55,7 +55,7 @@ it( 'displays the message', () => { } ); // Skipped until `pointerEvents: 'box-none'` no longer erroneously prevents -// triggering `onTouch*` on the element: https://git.io/JSHZt +// triggering `onTouch*` on the element: https://github.com/callstack/react-native-testing-library/issues/897 it.skip( 'dismisses when the screen is tapped', () => { const screen = render( diff --git a/packages/editor/src/components/editor-help/test/index.native.js b/packages/editor/src/components/editor-help/test/index.native.js index 69aab1fccbd47e..332b6d65fa9565 100644 --- a/packages/editor/src/components/editor-help/test/index.native.js +++ b/packages/editor/src/components/editor-help/test/index.native.js @@ -35,7 +35,7 @@ it( 'navigates back from help topic detail screen', async () => { const backButton = screen.getAllByA11yLabel( 'Go back' ); fireEvent.press( backButton[ backButton.length - 1 ] ); - // Currently logs `act` warning due to https://git.io/JYYGE + // Currently logs `act` warning due to https://github.com/callstack/react-native-testing-library/issues/379 await waitForElementToBeRemoved( () => screen.getByText( 'Each block has its own settings. To find them, tap on a block. Its settings will appear on the toolbar at the bottom of the screen.' diff --git a/packages/react-native-editor/ios/Podfile b/packages/react-native-editor/ios/Podfile index 88d0963b5d5235..2785e498e7c5c7 100644 --- a/packages/react-native-editor/ios/Podfile +++ b/packages/react-native-editor/ios/Podfile @@ -30,7 +30,7 @@ target 'GutenbergDemo' do # Let Pods targets inherit deployment target from the app # This solution is suggested here: https://github.com/CocoaPods/CocoaPods/issues/4859 installer.pods_project.targets.each do |target| - # Exclude RCT-Folly as it requires explicit deployment target https://git.io/JPb73 + # Exclude RCT-Folly as it requires explicit deployment target https://github.com/facebook/flipper/issues/834#issuecomment-899725463 if (target.name != 'RCT-Folly') target.build_configurations.each do |configuration| pod_ios_deployment_target = Gem::Version.new(configuration.build_settings['IPHONEOS_DEPLOYMENT_TARGET']) diff --git a/packages/react-native-editor/metro.config.js b/packages/react-native-editor/metro.config.js index 77bc68ec663d52..73bc29c15d1b14 100644 --- a/packages/react-native-editor/metro.config.js +++ b/packages/react-native-editor/metro.config.js @@ -36,8 +36,8 @@ module.exports = { * within this project to include the necessary `/assets/..` that Metro's * server expects to traverse to the correct directory. * - * - https://git.io/JBV4e - * - https://git.io/JBFon + * - https://github.com/facebook/metro/issues/290 + * - https://github.com/expo/expo/issues/7545#issuecomment-712737616 */ const firstUrlSegment = req.url.split( '/' )[ 1 ]; if ( packageNames.includes( firstUrlSegment ) ) { diff --git a/packages/react-native-editor/src/setup.js b/packages/react-native-editor/src/setup.js index b55b9a4253cec3..f31538b70f24b2 100644 --- a/packages/react-native-editor/src/setup.js +++ b/packages/react-native-editor/src/setup.js @@ -26,8 +26,8 @@ const reactNativeSetup = () => { * TODO: Migrate to @gorhom/bottom-sheet or replace usage of * LayoutAnimation to Animated. KeyboardAvoidingView's usage of * LayoutAnimation collides with both BottomSheet and NavigationContainer - * usage of LayoutAnimation simultaneously https://git.io/J1lZv, - * https://git.io/J1lZY + * usage of LayoutAnimation simultaneously https://github.com/facebook/react-native/issues/12663, + * https://github.com/facebook/react-native/issues/10606 */ 'Overriding previous layout animation', ] ); diff --git a/packages/react-native-editor/src/test/index.test.js b/packages/react-native-editor/src/test/index.test.js index 9d4569b114ec32..05412dcfafcdaf 100644 --- a/packages/react-native-editor/src/test/index.test.js +++ b/packages/react-native-editor/src/test/index.test.js @@ -53,7 +53,7 @@ describe( 'Register Gutenberg', () => { jest.isolateModules( () => render( ) ); // "invocationCallOrder" can be used to compare call orders between different mocks. - // Reference: https://git.io/JyBk0 + // Reference: https://github.com/facebook/jest/issues/4402#issuecomment-534516219 const setupLocaleCallOrder = setupLocale.mock.invocationCallOrder[ 0 ]; const onSetupImportedCallOrder = mockOnModuleImported.mock.invocationCallOrder[ 0 ]; @@ -79,7 +79,7 @@ describe( 'Register Gutenberg', () => { jest.isolateModules( () => render( ) ); // "invocationCallOrder" can be used to compare call orders between different mocks. - // Reference: https://git.io/JyBk0 + // Reference: https://github.com/facebook/jest/issues/4402#issuecomment-534516219 const beforeInitCallOrder = beforeInitCallback.mock.invocationCallOrder[ 0 ]; const onSetupImportedCallOrder = @@ -108,7 +108,7 @@ describe( 'Register Gutenberg', () => { const hookCallIndex = 0; // "invocationCallOrder" can be used to compare call orders between different mocks. - // Reference: https://git.io/JyBk0 + // Reference: https://github.com/facebook/jest/issues/4402#issuecomment-534516219 const hookCallOrder = doAction.mock.invocationCallOrder[ hookCallIndex ]; const onRenderEditorCallOrder = @@ -139,7 +139,7 @@ describe( 'Register Gutenberg', () => { const hookCallIndex = 0; // "invocationCallOrder" can be used to compare call orders between different mocks. - // Reference: https://git.io/JyBk0 + // Reference: https://github.com/facebook/jest/issues/4402#issuecomment-534516219 const hookCallOrder = applyFilters.mock.invocationCallOrder[ hookCallIndex ]; const onRenderEditorCallOrder = @@ -170,7 +170,7 @@ describe( 'Register Gutenberg', () => { const hookCallIndex = 1; // "invocationCallOrder" can be used to compare call orders between different mocks. - // Reference: https://git.io/JyBk0 + // Reference: https://github.com/facebook/jest/issues/4402#issuecomment-534516219 const hookCallOrder = doAction.mock.invocationCallOrder[ hookCallIndex ]; const onRenderEditorCallOrder = diff --git a/test/native/helpers.js b/test/native/helpers.js index af17feed5e49ea..e40c95273db74b 100644 --- a/test/native/helpers.js +++ b/test/native/helpers.js @@ -114,7 +114,7 @@ export async function initializeEditor( props, { component } = {} ) { export * from '@testing-library/react-native'; -// Custom implementation of the waitFor utility to prevent the issue: https://git.io/JYYGE. +// Custom implementation of the waitFor utility to prevent the issue: https://github.com/callstack/react-native-testing-library/issues/379. export function waitFor( cb, { timeout, interval } = { timeout: 1000, interval: 50 } diff --git a/test/native/setup.js b/test/native/setup.js index 5fb9927752771d..ad59b671dcc411 100644 --- a/test/native/setup.js +++ b/test/native/setup.js @@ -5,7 +5,7 @@ import 'react-native-gesture-handler/jestSetup'; import { Image, NativeModules as RNNativeModules } from 'react-native'; // React Native sets up a global navigator, but that is not executed in the -// testing environment: https://git.io/JSSBg +// testing environment: https://github.com/facebook/react-native/blob/6c19dc3266b84f47a076b647a1c93b3c3b69d2c5/Libraries/Core/setUpNavigator.js#L17 global.navigator = global.navigator ?? {}; // Set up the app runtime globals for the test environment, which includes