Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

introduce RCTTestUtils #37810

Closed
wants to merge 3 commits into from
Closed

Conversation

philIip
Copy link
Contributor

@philIip philIip commented Jun 9, 2023

Summary:
Changelog: [Internal]

in this change, i introduce a helper file for swizzling selectors that can be used for building custom shims and fakes. i tried putting this in the cxx target, but i struggled a lot with managing the dependencies and understanding the extra behavior that was happening from being in a rn_xplat_cxx_library. this file is also not obj-c++, so i feel like we should decouple if possible.

it does seem like this is a new practice i'm introducing, so let me know if there's something that i'm missing.

Reviewed By: cipolleschi

Differential Revision: D46507803

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner fb-exported labels Jun 9, 2023
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D46507803

philIip added a commit to philIip/react-native that referenced this pull request Jun 9, 2023
Summary:
Pull Request resolved: facebook#37810

Changelog: [Internal]

in this change, i introduce a helper file for swizzling selectors that can be used for building custom shims and fakes. i tried putting this in the cxx target, but i struggled a lot with managing the dependencies and understanding the extra behavior that was happening from being in a `rn_xplat_cxx_library`. this file is also not obj-c++, so i feel like we should decouple if possible.

it does seem like this is a new practice i'm introducing, so let me know if there's something that i'm missing.

Reviewed By: cipolleschi

Differential Revision: D46507803

fbshipit-source-id: afff16d12093805856ae8ded8919e382d58094e9
@philIip philIip force-pushed the export-D46507803 branch from 78ad3dd to f793715 Compare June 9, 2023 22:40
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D46507803

@analysis-bot
Copy link

analysis-bot commented Jun 9, 2023

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 8,747,927 -1
android hermes armeabi-v7a 8,060,535 -2
android hermes x86 9,239,529 +0
android hermes x86_64 9,089,330 -4
android jsc arm64-v8a 9,310,391 +0
android jsc armeabi-v7a 8,500,328 -1
android jsc x86 9,372,891 -2
android jsc x86_64 9,626,786 +0

Base commit: 7d1f7f3
Branch: main

philIip added a commit to philIip/react-native that referenced this pull request Jun 9, 2023
Summary:
Pull Request resolved: facebook#37810

Changelog: [Internal]

in this change, i introduce a helper file for swizzling selectors that can be used for building custom shims and fakes. i tried putting this in the cxx target, but i struggled a lot with managing the dependencies and understanding the extra behavior that was happening from being in a `rn_xplat_cxx_library`. this file is also not obj-c++, so i feel like we should decouple if possible.

it does seem like this is a new practice i'm introducing, so let me know if there's something that i'm missing.

Reviewed By: cipolleschi

Differential Revision: D46507803

fbshipit-source-id: 70fa17e29499c6c00526c23ed5e583c3e2c7aa1e
@philIip philIip force-pushed the export-D46507803 branch from f793715 to bf83622 Compare June 9, 2023 23:20
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D46507803

philIip added a commit to philIip/react-native that referenced this pull request Jun 11, 2023
Summary:
Pull Request resolved: facebook#37810

Changelog: [Internal]

in this change, i introduce a helper file for swizzling selectors that can be used for building custom shims and fakes. i tried putting this in the cxx target, but i struggled a lot with managing the dependencies and understanding the extra behavior that was happening from being in a `rn_xplat_cxx_library`. this file is also not obj-c++, so i feel like we should decouple if possible.

it does seem like this is a new practice i'm introducing, so let me know if there's something that i'm missing.

Reviewed By: cipolleschi

Differential Revision: D46507803

fbshipit-source-id: a9657a636ce6f97b0f84ec29f0b8944bc0181409
@philIip philIip force-pushed the export-D46507803 branch from bf83622 to 01ab2c0 Compare June 11, 2023 04:35
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D46507803

philIip added a commit to philIip/react-native that referenced this pull request Jun 11, 2023
Summary:
Pull Request resolved: facebook#37810

Changelog: [Internal]

in this change, i introduce a helper file for swizzling selectors that can be used for building custom shims and fakes. i tried putting this in the cxx target, but i struggled a lot with managing the dependencies and understanding the extra behavior that was happening from being in a `rn_xplat_cxx_library`. this file is also not obj-c++, so i feel like we should decouple if possible.

it does seem like this is a new practice i'm introducing, so let me know if there's something that i'm missing.

Reviewed By: cipolleschi

Differential Revision: D46507803

fbshipit-source-id: cef443c56bb705c5ef5f17a4cdaff75e46d34d54
@philIip philIip force-pushed the export-D46507803 branch from 01ab2c0 to e68b621 Compare June 11, 2023 04:46
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D46507803

philIip added a commit to philIip/react-native that referenced this pull request Jun 11, 2023
Summary:
Pull Request resolved: facebook#37810

Changelog: [Internal]

in this change, i introduce a helper file for swizzling selectors that can be used for building custom shims and fakes. i tried putting this in the cxx target, but i struggled a lot with managing the dependencies and understanding the extra behavior that was happening from being in a `rn_xplat_cxx_library`. this file is also not obj-c++, so i feel like we should decouple if possible.

it does seem like this is a new practice i'm introducing, so let me know if there's something that i'm missing.

Reviewed By: cipolleschi

Differential Revision: D46507803

fbshipit-source-id: 79bca0ab853ce060ff90d633d2f6e070f72b6179
@philIip philIip force-pushed the export-D46507803 branch from e68b621 to 3c96787 Compare June 11, 2023 04:55
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D46507803

philIip added a commit to philIip/react-native that referenced this pull request Jun 12, 2023
Summary:
Pull Request resolved: facebook#37810

Changelog: [Internal]

in this change, i introduce a helper file for swizzling selectors that can be used for building custom shims and fakes. i tried putting this in the cxx target, but i struggled a lot with managing the dependencies and understanding the extra behavior that was happening from being in a `rn_xplat_cxx_library`. this file is also not obj-c++, so i feel like we should decouple if possible.

it does seem like this is a new practice i'm introducing, so let me know if there's something that i'm missing.

Reviewed By: cipolleschi

Differential Revision: D46507803

fbshipit-source-id: b042feed2592fcf00bd5338ce3ae4a9c2ca2d191
@philIip philIip force-pushed the export-D46507803 branch from 3c96787 to 1da8b0a Compare June 12, 2023 17:41
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D46507803

philIip added a commit to philIip/react-native that referenced this pull request Jun 12, 2023
Summary:
Pull Request resolved: facebook#37810

Changelog: [Internal]

in this change, i introduce a helper file for swizzling selectors that can be used for building custom shims and fakes. i tried putting this in the cxx target, but i struggled a lot with managing the dependencies and understanding the extra behavior that was happening from being in a `rn_xplat_cxx_library`. this file is also not obj-c++, so i feel like we should decouple if possible.

it does seem like this is a new practice i'm introducing, so let me know if there's something that i'm missing.

Reviewed By: cipolleschi

Differential Revision: D46507803

fbshipit-source-id: 9e3e56c412f1c7725c2855e5b8ded7119044e0c9
@philIip philIip force-pushed the export-D46507803 branch from 1da8b0a to 4bd3941 Compare June 12, 2023 17:49
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D46507803

philIip and others added 3 commits June 12, 2023 10:55
Differential Revision: D46467041

fbshipit-source-id: 27ce5c24fd541b1380ed2fadc7d748bf64bd1fed
Differential Revision: D46467229

fbshipit-source-id: 84fd40219855b1dbe0a6bdb021b9b41a5b9efa7f
Summary:
Pull Request resolved: facebook#37810

Changelog: [Internal]

in this change, i introduce a helper file for swizzling selectors that can be used for building custom shims and fakes. i tried putting this in the cxx target, but i struggled a lot with managing the dependencies and understanding the extra behavior that was happening from being in a `rn_xplat_cxx_library`. this file is also not obj-c++, so i feel like we should decouple if possible.

it does seem like this is a new practice i'm introducing, so let me know if there's something that i'm missing.

Reviewed By: cipolleschi

Differential Revision: D46507803

fbshipit-source-id: 16e8dad7abf30963aa61642b0f2764bcc5204003
@philIip philIip force-pushed the export-D46507803 branch from 4bd3941 to af4b710 Compare June 12, 2023 17:58
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D46507803

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Jun 12, 2023
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in a300a35.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants