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

☂️ Help us Kotlin-ify React Native tests - Round 2 #38825

Closed
27 tasks done
mdvacca opened this issue Aug 7, 2023 · 93 comments
Closed
27 tasks done

☂️ Help us Kotlin-ify React Native tests - Round 2 #38825

mdvacca opened this issue Aug 7, 2023 · 93 comments
Labels
Good first issue Interested in collaborating? Take a stab at fixing one of these issues. Help Wanted :octocat: Issues ideal for external contributors. ☂️ Umbrella To label issues that serve as coordination point and drivers for tasks in the react-native repo

Comments

@mdvacca
Copy link
Contributor

mdvacca commented Aug 7, 2023

☂️ Help us Kotlin-ify React Native tests - Round 2

Hey all 👋
After the successful first round, we're back with another round of tasks to help us improve Kotlin support in React Native core. In this first stage we are looking for help to migrate tests and RN Tester to Kotlin.

How to work

If you wish to grab one of the task you should:

  • Verify that the task is not claimed yet.
  • Comment here that you claim a task.
  • Start working on the task. Testing instructions are below.
  • Send the PR for review. Ping @cortinico and @mdvacca for a review.
  • You can’t claim more than one task at a time.
  • If you’re not completing the task within a week, we’ll release it for the next contributor.

Code formatting

Please use KtFmt to format Kotlin tests.
You can use the web UI to reformat directly from the browser

[Easy] Migrating RN Tester to Kotlin

We’re looking into migrating our Sample App to Kotlin.

Please act as follow:

  • Fork and clone the repo
  • Verify that RN Tester runs with yarn && yarn android from the root folder
  • Migrate the file to Kotlin
  • Verify that RN Tester runs again using the same command.

List of files to migrate

[Hard] Migrating tests to Kotlin

We’re looking into migrating some of our production tests to Kotlin. Please note that some of those tests are @Ignore (so they’re not running). Your task would be to fix them and convert them to Kotlin.

Please act as follow:

  • Fork and clone the repo
  • Install the dependencies with yarn from the root repo.
  • Verify that tests are running green with: ./gradlew :packages:react-native:ReactAndroid:test
  • Remove the @Ignore annotation on the class/tests
  • Fix the test so that it’s green
  • Migrate the file to Kotlin
  • Verify that the tests are still green by using the command above.

List of tests to migrate

@github-actions github-actions bot added Needs: Version Info Needs: Author Feedback Needs: Repro This issue could be improved with a clear list of steps to reproduce the issue. and removed Needs: Triage 🔍 labels Aug 7, 2023
@Vgtnyc123

This comment was marked as off-topic.

@mateusz1913
Copy link
Contributor

👋 I would like to claim react/views/text/ReactTextTest

@okwasniewski
Copy link
Contributor

Hey! I can work on react/views/view/ColorUtilTest

@atlj
Copy link
Contributor

atlj commented Aug 8, 2023

Hey, I can grab react/views/image/ImageResizeModeTest

@retyui
Copy link
Contributor

retyui commented Aug 8, 2023

for me please react/uimanager/BaseViewManagerTest.java

UPD: done #38841

@bufgix
Copy link
Contributor

bufgix commented Aug 8, 2023

Hey 👋, i can work on react/uimanager/ReactPropAnnotationSetterTest

edit: This was not as strightforward as I thought so I've decided to take react/views/image/ReactImagePropertyTest instead. Sorry for inconvenience folks

@fabioh8010
Copy link
Contributor

Hey 👋 I can work on react/uimanager/MatrixMathHelperTest

@tarunrajput
Copy link
Contributor

@mdvacca, I'd like to migrate react/views/text/CustomLineHeightSpanTest

@siddarthkay
Copy link
Contributor

Hi @mdvacca : I'd like to migrate react/uimanager/JSPointerDispatcherTest Please assign it to me :)

@BogiKay
Copy link
Contributor

BogiKay commented Aug 8, 2023

Hey! I can work on component/MyNativeView.java

@yasincidem
Copy link
Contributor

Hi, I'd like to work on react/views/textinput/ReactTextInputPropertyTest

@thiagobrez
Copy link
Contributor

Hey 👋🏻 , I can work on react/uimanager/ReactPropConstantsTest

@sapkotamadhusudan
Copy link
Contributor

Hi, i can work on react/uimanager/SimpleViewPropertyTest

@imalgrab
Copy link
Contributor

imalgrab commented Aug 8, 2023

Hey, I'd like to work on component/MyLegacyViewManager.java

@cortinico cortinico added Good first issue Interested in collaborating? Take a stab at fixing one of these issues. Help Wanted :octocat: Issues ideal for external contributors. ☂️ Umbrella To label issues that serve as coordination point and drivers for tasks in the react-native repo and removed Needs: Author Feedback Needs: Repro This issue could be improved with a clear list of steps to reproduce the issue. Needs: Version Info labels Aug 8, 2023
@facebook facebook deleted a comment from github-actions bot Aug 8, 2023
@adhamhassan99
Copy link

Hey @adhamhassan99 : yeah I've been caught up and will probably be more caught up in coming 2 weeks, you can go ahead and start working on TextInputTest 🙏🏻

Perfect will do my best as its my first ever OSS contribution ! wish me luck

Saadnajmi added a commit to microsoft/react-native-macos that referenced this issue Nov 7, 2023
* Set Hermes release version

* bumped packages versions

#publish-packages-to-npm

* Bump react-native-babel-preset to 0.73.18 #publish-packages-to-npm

* Enable native view configs in bridgeless mode in OSS (facebook#39476)

Summary:
Pull Request resolved: facebook#39476

This diff enables native view config interop layer in bridgeless mode by default for OSS.
It also removes redundant `enableNativeViewConfigsInBridgelessMode` JS feature flag.
Changelog: [General][Added] - Native view config interop layer enabled in bridgeless mode.

Reviewed By: luluwu2032

Differential Revision: D49318325

fbshipit-source-id: ea2e38b2ea10637b578d98e38d97eed923498fb3

* Enable Template with Bridgeless (facebook#39661)

Summary:
Pull Request resolved: facebook#39661

Changelog:
[Android][Changed] - Enable Template with Bridgeless

Reviewed By: cortinico

Differential Revision: D49464580

fbshipit-source-id: 26dd8e2a0b9a66421d58f147eed4d2fd28817dc7

* [Hotfix] Remove node engines constraint for normalize-color (facebook#39698)

* Bump CLI to 12.0.0-alpha.16

* bumped packages versions

#publish-packages-to-npm

* Reduce visibility of functions in ReactHostImpl (facebook#39640)

Summary:
Pull Request resolved: facebook#39640

changelog: [internal] internal

Reducing visibility from public to package only for `ReactHostImpl.getDefaultHardwareBackBtnHandler()` since it's only used within package

Reviewed By: mdvacca

Differential Revision: D49612859

fbshipit-source-id: 3c40888da732f33dc046d9363b08119e707f4ea4

* Enable TurboModule interop in Bridgeless Mode (facebook#39687)

Summary:
Pull Request resolved: facebook#39687

Bridgeless mode is releasing in 0.73.

Bridgeless mode isn't stable inside RNTester (in 0.73) without the TurboModule interop layer: T164853040.

**Changes:** Enable the TurboModule interop layer in RNTester (Bridgeless Mode).

**Urgency:** This will unblock the 0.73.

Changelog: [General][Added] - Enable TurboModule interop in Bridgeless mode

Reviewed By: fkgozali, dmytrorykun, luluwu2032

Differential Revision: D49653919

fbshipit-source-id: a4a09a99544c2ef8713484dbeab5cb04f29256cc

* Export Commands and Constants only if native view config interop is enabled (facebook#39696)

Summary:
Pull Request resolved: facebook#39696

`Commands` and `Constants` should be set in native only if component data is instantiated via native view config interop layer.
Changelog: [Internal]

Reviewed By: RSNara

Differential Revision: D49684166

fbshipit-source-id: ceaa29c2ed3336aa6e21a116a3f5f94e03c225c1

* Bump CLI to 12.0.0-alpha.17

* Bump package versions

#publish-packages-to-npm

* Update Xcode 15 patches to be more robust (facebook#39710)

Summary:
Pull Request resolved: facebook#39710

Last week Apple released Xcode 15, which required us to ship a workaround for the new linker.
Unfortunately, the previous fix was not good enough and there were some edge cases that were not covered.
For example, in some occasions the flags are read as an array and the `-Wl` and the `-ld_classic` flags were separated and not properly removed when moving from Xcode 15 to Xcpde 14.3.1.

This change fixes those edge cases, with a more robust solution where:
- We convert the flags to a string.
- We trim the string and the values properly.
- We add the flags when running `pod install` with Xcode 15 as the default iOS toolchain.
- We remove the flags when running `pod install` with Xcode <15 as the default iOS toolchain.

## Changelog:
[Internal] - Make the Xcode 15 workaround more robust.

Reviewed By: dmytrorykun

Differential Revision: D49748844

fbshipit-source-id: 34976d148f123c5aacba6487a500874bb938fe99

* Sync debugger-frontend

Summary:
Sync latest `debugger-frontend` artifacts, based on motiz88/rn-chrome-devtools-frontend#12.

Changelog: [Internal]

Reviewed By: motiz88

Differential Revision: D49502838

fbshipit-source-id: 25805066670ac35ec976dc820a73a722ee182dd2

* Mark initHybrid as @JvmStatic (facebook#39755)

Summary:
Pull Request resolved: facebook#39755

The goal of this diff is to fix:
```
JNI DETECTED ERROR IN APPLICATION: JNI NewGlobalRef called with pending exception java.lang.NoSuchMethodError: no static or non-static method
"Lcom/facebook/react/jscexecutor/JSCExecutor;.initHybrid(Lcom/facebook/react/bridge/ReadableNativeMap;)Lcom/facebook/jni/HybridData;"

```

changelog: [internal] internal

Reviewed By: luluwu2032

Differential Revision: D49831595

fbshipit-source-id: 9ce22cdccdd02af74edb27be2df72a469d3166c9

* Bump package versions

#publish-packages-to-npm

* Bump package versions (dependant packages)

#publish-packages-to-npm

* [0.73.0-rc.0] Bump version numbers

* Fix typings for ReactNativeVersion.js

This is an eager cherry pick of facebook#39784 for `0.73-stable`.

* Revert "[0.73.0-rc.0] Bump version numbers"

This reverts commit 6b9cf50.

* [RN][CI] Make the Choose CI Job run also on tags (facebook#39774)

* [0.73.0-rc.0] Bump version numbers

* Revert "Fix typings for ReactNativeVersion.js"

This reverts commit b6c0703.

* Fix typings for ReactNativeVersionCheck

Applies correct fix for `_formatVersion` misalignment — based on facebook#36657.

* Fix start command exit behaviour

This is an eager cherry pick of facebook#39788 for `0.73-stable`.

* Bump package versions

#publish-packages-to-npm

* [0.73.0-rc.1] Bump version numbers

* Revert "[0.73.0-rc.1] Bump version numbers"

This reverts commit 725b1c9.

* Make jobs required for hermes to run on tagged builds (facebook#39800)

* [0.73.0-rc.1] Bump version numbers

* Fix broken Loading/Refreshing indicator on Android

Summary:
The Loading.../Refreshing... indicator is currently broken on Android.
The reason is related to D42599220
We used to have a Toast shown to users on Android as a fallback, but as the
DevLoadingView is not always loaded as a module in the core package, this ends up in the banner never beign shown to the user (on RN Tester or template apps).

Changelog:
[Android] [Fixed] - Fix broken Loading/Refreshing indicator on Android

Reviewed By: cipolleschi

Differential Revision: D49876757

fbshipit-source-id: 400e002327ebca908e3e7a7f81c5066888ac4e9b

* Remove unneeded write to .version file (facebook#39807)

Summary:
Pull Request resolved: facebook#39807

We used to need this file so that we could read the react native version when [creating Hermes artifacts](facebook@e4b5d3e#diff-78a8a19706dbd2a4425dd72bdab0502ed7a2cef16365ab7030a5a0588927bf47R1507). Originally, that change was introduced [here](facebook@e4b5d3e#diff-78a8a19706dbd2a4425dd72bdab0502ed7a2cef16365ab7030a5a0588927bf47)

Despite the fact that that approach was wrong, as we already have the right version in the package.json which is guaranteed to be present, a lot has changed since then and we don't need that file anymore.

## Changelog:
[Internal] - Remove lines that write a .version file while releasing on npm

Reviewed By: lunaleaps

Differential Revision: D49909718

fbshipit-source-id: bd23d6d73001d0b58bf6b0321ed6d4ceb3523e7a

* Default to native view configs in bridged mode and to static view configs in bridgeless mode (facebook#39704)

Summary:
Pull Request resolved: facebook#39704

Default to native view configs in bridged mode and to static view configs in bridgeless mode.
Remove `setRuntimeConfigProvider` calls from RNTester and from the Template.
Changelog: [Internal]

Reviewed By: RSNara

Differential Revision: D49687252

fbshipit-source-id: 140e1c510ba3fbc153978b59c8bb4b4e35bc7571

* Fix Gemfile, setting Active support to < 7.1.0 (facebook#39828)

Summary:
Pull Request resolved: facebook#39828

Active Suppert released a new Gem which is incompatible with Cocoapods 1.13.0, the latest release, as they removed a method used by cocoapods.

This fix ensures that we install compatible versions of the Gem.

## Changelog:
[iOS][Fixed] - Set the max version of Active support to 7.0.8

Reviewed By: hoxyq

Differential Revision: D49949782

fbshipit-source-id: 278097502d3a416567cc8c0b90090fee4fb21503

* fix: view flips on RTL with new transform origin changes in Paper arch. (facebook#39803)

Summary:
Fixes - facebook#38626 (comment). Explained the issue [here](facebook#38626 (comment)).

## Changelog:

[IOS] [FIXED] - View flips horizontally in paper arch on RTL

Pull Request resolved: facebook#39803

Test Plan: Run RNTester in Paper mode on iOS

Reviewed By: NickGerleman

Differential Revision: D49952227

Pulled By: lunaleaps

fbshipit-source-id: 7240552e499765859dceea0a0406561cc3a3148f

* [iOS] Deprecate RCTGetMultiplierForContentSizeCategory (facebook#39785)

* Set hermes for Android template project in testing

Summary:
Changelog: [Internal] - Set the hermes value as specified by the test-e2e-local script flag. Right now, the script incorrectly ignores the flag

By default, the template project has `hermesEnabled=true`

Reviewed By: cipolleschi

Differential Revision: D49831355

fbshipit-source-id: 7fb8613fa86f2c6140b7d25b16aeb583e6e26c12

* fix missing sed import

* Update pod for CI

* Update .gitignore for sdks/downloads

* [0.73.0-rc.2] Bump version numbers

* Add --experimental-debugger-frontend flag, restore 0.72 flow as base (facebook#40766)

Summary:
Pull Request resolved: facebook#40766

This changeset allows users to opt into the new debugger frontend experience by passing `--experimental-debugger` to `react-native start`. **We are defaulting this option to `true`** for now, but will continue to evaluate this feature before 0.73 ships. It restores Flipper (via `flipper://`) as the default handling for `/open-debugger` (matching 0.72 behaviour) when this flag is not enabled.

Detailed changes:

- Replaces `enableCustomDebuggerFrontend` experiment in `dev-middleware` with `enableNewDebugger`. The latter now hard-swaps between the Flipper and new launch flows.
    - Removes now-unused switching of `devtoolsFrontendUrl`.
- Implements `deprecated_openFlipperMiddleware` (matching previous RN CLI implementation).
- Disables "`j` to debug" key handler by default.
- Marks "`j` to debug" and `/open-debugger` console logs as experimental.

Changelog:
[Changed][General] Gate new debugger frontend behind `--experimental-debugger` flag, restore Flipper as base launch flow

Reviewed By: motiz88

Differential Revision: D50084590

fbshipit-source-id: 5234634f20110cb7933b1787bd2c86f645411fff

* [0.73] Make sure template is consuming the right buildToolsVersion (facebook#39957)

* Store hermes stable artifacts inside Pods directory (facebook#40733)

Summary:
Pull Request resolved: facebook#40733

Node package managers may purge or recreate `node_modules/react-native` when adding/removenf project dependencies. Storing hermes iOS artifacts inside `node_modules/react-native/sdks` is not reliable.
This diff moves those artifacts to `Pods/hermes-engine-artifacts`.
Should fix facebook#39903
Changelog: [Internal]

Reviewed By: cipolleschi

Differential Revision: D50081559

fbshipit-source-id: a130898e12fb6275cadaef7617bf4b6a09e6487e

* Top-down onLayout events (facebook#39644)

Summary:
Pull Request resolved: facebook#39644

This makes Android Paper/Classic renderer fire `onLayout` events top down, like in Fabric/new Architecture. This gives a much more sane model for using layout events to calculate bottom/right-edge insets.

I was under the impression that Paper in general was bottom-up, but it turns out that is only true for Android and Windows (iOS seems totally deterministic).

This is a behavior change, but to my knowledge was never hit during the Fabric migration, and any JS code already written for both Android and iOS cannot make assumptions here anyways.

Changelog:
[General][Changed] - Make layout events top-down on Android classic renderer

Reviewed By: mdvacca

Differential Revision: D49627996

fbshipit-source-id: 29964b421dd420681d45348c7db16f211a6c087f

* Deterministic onLayout event ordering for iOS Paper (facebook#40748)

Summary:
Pull Request resolved: facebook#40748

The ordering of `onLayout` events is non-deterministic on iOS Paper, due to nodes being added to an `NSHashTable` before iteration, instead of an ordered collection.

We don't do any lookups on the collection, so I think this was chosen over `NSMutableArray` for the sake of `[NSHashTable weakObjectsHashTable]`, to avoid retain/release. Using a collection which does retain/release seems to cause a crash due to double release or similar, so those semantics seem intentional (though I'm not super familiar with the model here).

We can replicate the memory semantics with ordering by using `NSPointerArray` (which is unfortunately not parameterized). This change does that, so we get consistently top-down layout events (matching Fabric, and Android Paper as of D49627996). This lets us use multiple layout events to calculate right/bottom edge insets deterministically.

Changelog:
[iOS][Changed] -  Deterministic onLayout event ordering for iOS Paper

Reviewed By: luluwu2032

Differential Revision: D50093411

fbshipit-source-id: f6a9d5c973b97aede879baa8b952cc1be2447f28

* Remove code to support bottom-up layout events in horizontal RTL (facebook#39646)

Summary:
Pull Request resolved: facebook#39646

We can dramatically simplify this code and remove quirks/hacks, now that we can assume layout events are always fired top down.

Changelog: [Internal]

Reviewed By: yungsters

Differential Revision: D49628669

fbshipit-source-id: 7de5bbc4597eba1c59aaa7672c70e76d2786c7ef

* Fix iOS Paper Scroll Event RTL check (facebook#40751)

Summary:
Pull Request resolved: facebook#40751

In D48379915 I fixed inverted `contentOffset` in `onScroll` events on iOS. I thought I tested on Paper, but I think this was during a period where the Paper route in Catalyst was actually launching Fabric (oops).

In Paper, at least under `forceRTL` and English, `[UIApplication sharedApplication].userInterfaceLayoutDirection` is not set to RTL. We instead have a per-view `reactLayoutDirection` we should be reading.

This sort of thing isn't currently set on Fabric, which checks application-level RTL. This seems... not right with being able to set `direction` in a subtree context, but Android does the same thing, and that would take some greater changes.

Changelog:
[iOS][Fixed] - Fix iOS Paper Scroll Event RTL check

Reviewed By: luluwu2032

Differential Revision: D50098310

fbshipit-source-id: e321fca7b2f7983e903e23237bc2d604c72f98a3

* Disable --experimental-debugger by default (facebook#40795)

Summary:
Pull Request resolved: facebook#40795

Changelog: [Internal]

Reviewed By: hoxyq

Differential Revision: D50178494

fbshipit-source-id: b833b6accadd2902d97a45e030cd48d1b62ac438

* make runtime reference thread safe on tickleJs call

Summary:
The reference to runtime assumes the queue will ensure references to runtime are valid when invoked. This
isn't the case if you create a breakpoint, Hermes hit that breakpoint and your refresh the app. This consistently
will crash the app.

The fix is to not assument this, similar to ReactCommon/react/runtime/hermes/HermesInstance.cpp

Reviewed By: javache

Differential Revision: D50225678

fbshipit-source-id: b45cae1f5f687bc8c699fd74b187376a547012c5

* Run all the tests when not on a PR

Summary:
## Changelog:
[Internal] - Run all the tests in CI when not on a PR

Reviewed By: cortinico

Differential Revision: D50220596

fbshipit-source-id: be1a30d713e9d427858cf22bd3ca9549ad513057

* Add "Hermes only" to open debugger key prompt

Summary: Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D50327180

fbshipit-source-id: 6ee08fadd313828ddbc7702d5b3b1174a91b1ac1

* Refactors InspectorProxy to compare pathname portion of url passed in… (facebook#41005)

Summary:
… request to local pathname comparator variables to fix issue with other rightward elements of url such as query or fragment entering the comparison and causing 404 errors for key debugging routes.

A change in Chromium appended the query "?for_tabs" to the /json/list request made by Chrome DevTools to find remote debugger targets.

The current comparison in InspectorProxy.js compares the entire node IncomingMessage url field to the local pathname constants. The issue arises as url can also contain the query and fragment portions so the original comparison of "/json/list" === "/json/list" which resolved as true would become "/json/list?for_tabs" === "/json/list" and evaluate to false ultimately resulting in a 404 for the request.

In summary, all these changes/issues caused remote debugging of Hermes code in React Native apps to become unavailable, greatly impacting developer experience.

## Changelog:

[GENERAL] [FIXED] JS Debugging: Fix inspector-proxy to allow for DevTools requests with query strings

Pull Request resolved: facebook#41005

Reviewed By: NickGerleman

Differential Revision: D50342265

Pulled By: robhogan

fbshipit-source-id: a65f2908f0bea9fc15e1e3e4e6d31a3b9598e81f

* Make sure buildConfig is turned on for all the 3rd party libraries (facebook#40939)

Summary:
Pull Request resolved: facebook#40939

Currently some libs on RN 0.73 are broken as the default for Build Config generation changed
from true to false since AGP 8.x. This reverts the behavior to the old flag.

Closes facebook#40791
Closes facebook#40559

Changelog:
[Internal] [Changed] - Make sure buildConfig is turned on for all the 3rd party libraries

Reviewed By: mdvacca

Differential Revision: D50270382

fbshipit-source-id: 02dcb031c577f65be2f41d9da0334c1b3d89e4c5

* Configure Java toolchains only when executing App configuration (facebook#40757)

Summary:
Fix facebook#40560

## Changelog:

[ANDROID] [FIXED] - Ensure that `configureJavaToolChains` is only executed once during configuration

Pull Request resolved: facebook#40757

Test Plan:
- Create a fresh `react-native@0.73.0-rc.1` project
- Install `react-native-webview`
- Apply [this patch](https://github.com/react-native-webview/react-native-webview/pull/3175/files) for `react-native-webview` (caused by another issue facebook#40559)
- Edit `android/gradle.properties` and set `newArchEnabled` to true
- Build application
- (Expected) Application fail to build
- Apply this PR
- (Expected) Application build successfully

**Additional explanation:**

According to the implementation of `configureJavaToolChains`, all the subprojects (both the app and the libraries) will have their toolchains setup in one execution of the method. Therefore, it is okay for the method to be invoked only when configuring the plugin for the app.

On the other hand, invoking the method for more than one time will cause the issue stated in facebook#40560.

Reviewed By: cipolleschi

Differential Revision: D50361871

Pulled By: cortinico

fbshipit-source-id: bd5e18df97988122788d0482dba954e517a0cb5c

* Fix instacrash on bridgeless due to calling showMessage on null instance (facebook#39834)

Summary:
Pull Request resolved: facebook#39834

Bridgeless is instacrashing on fast-refresh. This fixes it.

Changelog:
[Android] [Fixed] - Fix instacrash on bridgeless due to calling showMessage on null instance

Reviewed By: cipolleschi

Differential Revision: D49929822

fbshipit-source-id: a2ce65797abd34d6a3e2b7f2c50d38a62ea8bdea

* Re-introduce FabricViewStateManager interface (facebook#40998)

Summary:
Pull Request resolved: facebook#40998

changelog: [Android] Add back interface FabricViewStateManager to unblock 0.73

I incorrectly deleted FabricViewStateManager in D47993140. This is a breaking change even for old architecture. Let's add it back and mark it as deprected so we can remove it later on.
This interface is not used in react-native anymore.

We are removing FabricViewStateManager because it simply wraps StateWrapper and provides no additional anymore.

Reviewed By: cortinico

Differential Revision: D50318633

fbshipit-source-id: aeb1c66c35018e336339616b564dee6f3156b54b

* Add ExceptionsManagerModule for RNTester (facebook#40767)

Summary:
Pull Request resolved: facebook#40767

Fix the following issue:
```Invariant Violation: TurboModuleRegistry.getEnforcing(...):
'ExceptionsManager' could not be found. Verify that a module by this name is registered in the native
binary.Bridgeless mode: true. TurboModule interop: true. Modules loaded: {"NativeModules":[],"TurboModules":
["PlatformConstants","AppState","SourceCode","BlobModule","WebSocketModule","DevSettings","DevToolsSettingsManager","LogBox","Networking","Appearance","DevLoadingView","DeviceInfo","DeviceEventManager",
"SoundManager","ImageLoader","DialogManagerAndroid","NativeAnimatedModule","I18nManager","AccessibilityInfo","StatusBarManager","StatusBarManager","IntentAndroid","ToastAndroid","ShareModule","Vibration"],
"NotFound":["NativePerformanceCxx","NativePerformanceObserverCxx","RedBox","BugReporting","HeadlessJsTaskSupport","FrameRateLogger","KeyboardObserver",
"AccessibilityManager","ModalManager","LinkingManager","ActionSheetManager","ExceptionsManager"]}
```

Changelog:
[Android][Changed] - Add Add ExceptionsManagerModule to CoreReactPackage

Reviewed By: cortinico

Differential Revision: D50017783

fbshipit-source-id: 8642bb23bdae50a1e702f5e0586b0ede80007bb1

# Conflicts:
#	packages/rn-tester/android/app/src/main/java/com/facebook/react/uiapp/RNTesterApplication.kt

* Fix compilation warnings introduced by Gradle 8.4 (facebook#39959)

Summary:
Pull Request resolved: facebook#39959

We're still accessing project.buildDir which will be removed in Gradle 9.0
I'm cleaning it up here.

Changelog:
[Internal] [Changed] - Fix compilation warnings introduced by Gradle 8.4

Reviewed By: yungsters

Differential Revision: D50016573

fbshipit-source-id: de7a725f61b503f08991ebf85b9a002cefab221a

* update: fresco 3.0.0 -> 3.1.0 (facebook#41024)

Summary:
- ref: reactwg/react-native-releases#64 (comment)

With react-native@0.73.0-rc.2 and fresco@3.0.0, `dlopen failed: library "libnative-imagetranscoder.so"` error made crash in android. so, I patched fresco version to 3.1.0, so that resolves the crash error according to facebook/fresco#2722 (comment) this comment.

## Changelog:

<!-- Help reviewers and the release process by writing your own changelog entry.

Pick one each for the category and type tags:

[ANDROID|GENERAL|IOS|INTERNAL] [BREAKING|ADDED|CHANGED|DEPRECATED|REMOVED|FIXED|SECURITY] - Message

For more details, see:
https://reactnative.dev/contributing/changelogs-in-pull-requests
-->

[ANDROID] [FIXED] - b1fccb0 bumped fresco version to 3.1.0.

Pull Request resolved: facebook#41024

Test Plan: after bumping fresco version, error not occurs.

Reviewed By: cipolleschi

Differential Revision: D50359564

Pulled By: cortinico

fbshipit-source-id: d24caecc057bad4d9c94e378422ab18c106a7ee6

# Conflicts:
#	packages/react-native/gradle/libs.versions.toml

* Rename `RCTRootViewFrameDidChangeNotification` as it's not trac… (facebook#39835)

Summary:
…king root view frame changes

Looking through where this was introduced (facebook#37649), it seems the notification went from tracking root view size changes to window size changes. However, it was not renamed. I was using it for root view changes in RN-macOS, which.. I guess I'll refactor. Meanwhile, let's update the name?

## Changelog:

[IOS] [CHANGED] - Rename `RCTRootViewFrameDidChangeNotification` as it's not tracking root view frame changes

Pull Request resolved: facebook#39835

Test Plan: CI should pass

Reviewed By: cipolleschi

Differential Revision: D50173742

Pulled By: javache

fbshipit-source-id: 4651696174c439800984a5e6cf642200bb9c4f3c

* Fix unstable RCTAppDelegate podspec (facebook#41009)

Summary:
Pull Request resolved: facebook#41009

This change should fix [facebook#39971](facebook#39971), computing the relative path from the App path to the pod installation root and using that instead of the absolute path to the `react-native.config.js` file

## Changelog
[Internal] - Stabilize RCTAppDelegate podspec

Reviewed By: cortinico

Differential Revision: D50323710

fbshipit-source-id: e29e62228d08c752e822d7a9ab5b1a2b5dcd6eb4

* Symbolicate unhandled promise rejections (facebook#40914)

Summary:
For a very long time when a promise rejects without an attached catch we get this warning screen without a correct stack trace, only some internal calls to the RN internals.

<img src="https://github.com/facebook/react-native/assets/1634213/75aa7615-ee3e-4229-80d6-1744130de6e5" width="200" />

I created [an issue for discussion](react-native-community/discussions-and-proposals#718) in the react-native-community repo and we figured out it was only a matter of symbolication. While it cannot be done on release without external packages and source maps, at least while developing we can provide a symbolicated stack-trace so developers can better debug the source of rejected promise.

I got the stack trace symbolicated and the correct code frame. I'm missing some help trying to display it in the warning view but at the very least I can now correctly show the line of the error and log the codeframe to the console.
## Changelog:

<!-- Help reviewers and the release process by writing your own changelog entry.

Pick one each for the category and type tags:

[GENERAL] [FIXED] - Show correct stack frame on unhandled promise rejections on development mode.

For more details, see:
https://reactnative.dev/contributing/changelogs-in-pull-requests

Pull Request resolved: facebook#40914

Test Plan:
I simply created a throwing function on a dummy app, and checked the output of the console and the warning view:

```ts
import React from 'react';
import {SafeAreaView, Text} from 'react-native';

async function throwme() {
  throw new Error('UNHANDLED');
}

function App(): JSX.Element {
  throwme();

  return (
    <SafeAreaView>
      <Text>Throw test</Text>
    </SafeAreaView>
  );
}

export default App;
```

Here is the output

<img src="https://github.com/facebook/react-native/assets/1634213/2c100e4d-618e-4143-8d64-4095e8370f4f" width="200" />

Edit: I got the warning window working properly:

<img src="https://github.com/facebook/react-native/assets/1634213/f02a2568-da3e-4daa-8132-e05cbe591737" width="200" />

Reviewed By: yungsters

Differential Revision: D50324344

Pulled By: javache

fbshipit-source-id: 66850312d444cf1ae5333b493222ae0868d47056

* Align all the pods to 13.4 (facebook#41050)

Summary:
Pull Request resolved: facebook#41050

This change aligns all the podspecs to 13.4, including 3rd party libraries. This should fix facebook#39826

## Changelog
[iOS][Changed] - Align all pods to 13.4

Reviewed By: huntie

Differential Revision: D50405114

fbshipit-source-id: 2d1b50ed8433e8cdc2214eb17e15be6ea81107a1

# Conflicts:
#	packages/react-native/scripts/cocoapods/utils.rb

* Bump SocketRocket to 0.6.1 (facebook#40774)

Summary:
The SocketRocket version was upgraded to 0.6.1 on the 0.72-stable branch but for some reason it was not updated in main, causing a downgrade when running `pod install` with 0.73.0 RC1

Original commit bumping SocketRocket -> facebook@8ce471e

## Changelog:

[IOS] [CHANGED] - Bump SocketRocket to 0.6.1

Pull Request resolved: facebook#40774

Test Plan: Run rntester locally

Reviewed By: cipolleschi

Differential Revision: D50137261

Pulled By: arushikesarwani94

fbshipit-source-id: dfc2760f5d5611881126ad114d8f6ada23630a29

# Conflicts:
#	packages/react-native/React-Core.podspec
#	packages/react-native/React/CoreModules/React-CoreModules.podspec
#	packages/rn-tester/Podfile.lock

* Bump RNTester Cocoapods cache keys after SocketRocket 6.1.0 bump (facebook#40789)

Summary:
Commit 8b88883 broke the Cache for RNTester because the cached version of the pods does not know about the exitence of SocketRocket 6.1.0
Bumping the keys should force a redownload of the cocoapods specs repo

## Changelog:
[Internal] - Bump RNTester cache keys

Pull Request resolved: facebook#40789

Test Plan: CircleCI is green

Reviewed By: GijsWeterings

Differential Revision: D50169281

Pulled By: cipolleschi

fbshipit-source-id: 83e251495bfa43d62384470efe97c5505d76684f

# Conflicts:
#	packages/rn-tester/Podfile.lock

* Revert "Add ExceptionsManagerModule for RNTester (facebook#40767)"

This reverts commit 034d815.

* rewrite RNTesterApplication to kotlin (facebook#39557)

Summary:
Rewrite `RNTesterApplication` to Kotlin as per [Help us Kotlin-ify React Native tests - Round 2](facebook#38825)

## Changelog:

[ANDROID] [CHANGED] - Rewrite RNTesterApplication to Kotlin, add AnnotationTarget property.

Pull Request resolved: facebook#39557

Test Plan:
`yarn && yarn android` ✅

The only thing I'm kinda unsure of is whether `AnnotationTarget.PROPERTY` should be added, but it didn't let me annotate `reactHostInterface` without that and didn't compile.
<img width="637" alt="image" src="https://github.com/facebook/react-native/assets/33528752/8bc84870-f3f2-4a46-b076-6ee7e38bd735">

 cortinico mdvacca

Reviewed By: cortinico

Differential Revision: D49598401

Pulled By: mdvacca

fbshipit-source-id: 105ae0c13c93dae0eeb2b6fa9040f03f42d2736a

* Do not guard initializeFlipper for bridgeless for RN Tester

Summary:
We don't need this if-than-else because the initializeFlipper already checks if we're on bridgeless or not

Changelog:
[Internal] [Changed] - Do not guard initializeFlipper for bridgeless for RN Tester

Reviewed By: NickGerleman

Differential Revision: D49881903

fbshipit-source-id: e6bfc941b43382580bd418a5f27ad9426d300c69

* Fix new arch example not render in RNTester (facebook#39810)

Summary:
Pull Request resolved: facebook#39810

Two issues will be fixed:
- Bridgeless has lazy view manager loading by default so the React Package that provides view managers must implement ViewManagerOnDemandReactPackage, we might could refactor the design of package classes later
- ThemedReactContext should **NOT** be used directly to call function ```getJSModule```, since it doesn't overrides ```getJSModule``` for Bridgeless, we can use it's internal variable ```meactApplicationContext``` which should be an instance of BridgelessReactContext

Reviewed By: cortinico

Differential Revision: D49912656

fbshipit-source-id: a0bdd717612398e8d7a6f36d36dba241a3b06bd7

* Move ReactHost to com.facebook.react (facebook#39769)

Summary:
Pull Request resolved: facebook#39769

Move ReactHost to com.facebook.react

bypass-github-export-checks

changelog: [internal] internal

Reviewed By: javache

Differential Revision: D49834870

fbshipit-source-id: 2cfdd34297399115457424f0ba1279fd477da4a1

* Add rn-tester Android BUCK configuration (facebook#39987)

Summary:
Pull Request resolved: facebook#39987

Changelog: [Internal]

Reviewed By: fkgozali

Differential Revision: D49983211

fbshipit-source-id: c362d5b93e159bfb5ca17bea0ec0484adcdb4dca

* Add ExceptionsManagerModule for RNTester (facebook#40767)

Summary:
Pull Request resolved: facebook#40767

Fix the following issue:
```Invariant Violation: TurboModuleRegistry.getEnforcing(...):
'ExceptionsManager' could not be found. Verify that a module by this name is registered in the native
binary.Bridgeless mode: true. TurboModule interop: true. Modules loaded: {"NativeModules":[],"TurboModules":
["PlatformConstants","AppState","SourceCode","BlobModule","WebSocketModule","DevSettings","DevToolsSettingsManager","LogBox","Networking","Appearance","DevLoadingView","DeviceInfo","DeviceEventManager",
"SoundManager","ImageLoader","DialogManagerAndroid","NativeAnimatedModule","I18nManager","AccessibilityInfo","StatusBarManager","StatusBarManager","IntentAndroid","ToastAndroid","ShareModule","Vibration"],
"NotFound":["NativePerformanceCxx","NativePerformanceObserverCxx","RedBox","BugReporting","HeadlessJsTaskSupport","FrameRateLogger","KeyboardObserver",
"AccessibilityManager","ModalManager","LinkingManager","ActionSheetManager","ExceptionsManager"]}
```

Changelog:
[Android][Changed] - Add Add ExceptionsManagerModule to CoreReactPackage

Reviewed By: cortinico

Differential Revision: D50017783

fbshipit-source-id: 8642bb23bdae50a1e702f5e0586b0ede80007bb1

* Make sure onIntArrayChanged is invoked on RN Tester (facebook#40940)

Summary:
Pull Request resolved: facebook#40940

Currently the `onIntArrayChanged` event is invoked
 only once inside RN Tester. I'm changing the logic to make sure it fires whenever we click "Set Opacity"

Changelog:
[Internal] [Changed] - Make sure onIntArrayChanged is invoked on RN Tester

Reviewed By: mdvacca, dmytrorykun

Differential Revision: D50264765

fbshipit-source-id: 93a60fd1b657c3d8b8182cab6bb7cd4368ac9a42

* Fix broken Fabric Interop example (facebook#40942)

Summary:
Pull Request resolved: facebook#40942

The Fabric Interop example for Android is broken. This is due to the Kotlin conversion which moved the `Color.colorToHSV(color, hsv)` statement *after* the HSV array is read so the array is always [0,0,0].

I'm fixing it here.

Changelog:
[Internal] [Changed] - Fix broken Fabric Interop example

Reviewed By: cipolleschi

Differential Revision: D50264766

fbshipit-source-id: 27ae5289408c7c23c667d6d7112437fa7ebe36d5

* Simplify new app template for bridgeless (facebook#40929)

Summary:
Pull Request resolved: facebook#40929

This diff reduces the footprint that bridgeless is imposing on the new app template. Specifically:
- I've created a `.toReactHost` method that converts a DefaultReactNativeHost to a DefaultReactHost
- I've updated RN Tester to use the same setup as the New App template which reduces code duplication.

I also had to remove a couple of `UnstableReactNativeAPI` as those were bleeding in the new app template.
I don't think we should ask users to opt-in in `UnstableReactNativeAPI` in the New App template itself as
this means that all the apps will get this opt-in.
Instead we should keep it only for specific APIs that we want the users to opt into.

Changelog:
[Internal] [Changed] - Simplify new app template for bridgeless

Reviewed By: cipolleschi, luluwu2032

Differential Revision: D50227693

fbshipit-source-id: e86c54d5156cc27f1f898b43ca89c57d5cf148b8

* Make events work for Fabric Interop on Bridgeless (facebook#40941)

Summary:
Pull Request resolved: facebook#40941

Events are currently not working for Fabric Interop on Bridgeless. That's because the `BridgelessReactContext` is not checking for interop modules on `getJsModule` calls, so the `InteropEventEmitter` is never returned.

This extends `BridgelessReactContext` so that  `InteropEventEmitter` is returned if the Interop Layer is turned on.

Changelog:
[Internal] [Changed] - Make events work for Fabric Interop on Bridgeless

Reviewed By: cipolleschi

Differential Revision: D50266484

fbshipit-source-id: 0188d71bdc7acc8c188d886d45f0258914ad7af7

* Make IntArray events work on Bridgeless for RN-Tester (facebook#41047)

Summary:
Pull Request resolved: facebook#41047

Array events are currently broken in the sample for RN Tester. This is because the event name is not registered correctly.

I'm updating the event registration to be correct.

Changelog:
[Internal] [Changed] - Make IntArray events work on Bridgeless for RN-Tester

Reviewed By: cipolleschi

Differential Revision: D50266485

fbshipit-source-id: 13bbce91a41281383d4857048e573b6d9cc5387b

* Bump package versions

#publish-packages-to-npm

* Make hermes build fetching more flavour resilient (facebook#41055)

Summary:
Pull Request resolved: facebook#41055

This should fix the issue highlighted in [this comment](reactwg/react-native-releases#64 (comment)).
Basically, before the fix, we were not supporting flavours correctly, as we assumed that only Debug and Release were available.

With this change, we infer whether we have to fetch Hermes for Debug or Release based on the actual flags that are passed. In this way, the users can customize their app's flavors more freely.

## Changelog:
[Internal] - Support multiple flavors when downloading Hermes

Reviewed By: huntie

Differential Revision: D50408381

fbshipit-source-id: 6990218b286b4dd823323bc63de90279efc9e74e

* Fix Glog for earlier versions of Xcode (facebook#41058)

Summary:
Pull Request resolved: facebook#41058

Glog has a piece of code which looks like this:
```
namespace google {

// They need the definitions of integer types.
#include "glog/log_severity.h"
#include "glog/vlog_is_on.h"
```

This fragment is:
- Always valid when the pod does not define a module
- Valid for Xcode >= 14.3, when the pod do define a module
- Invalid for Xcode < 14.3, when the pod do define a module

Modules are required to support Swift, so, in the long run, we want to have `DEFINES_MODULE` set to `YES` for `Glog`.

This is a temporary workaround to keep supporting older versions of Xcode while Apple keeps allowing to use Xcode 14.1 to submit apps to the store.
Historically, Apple pushes the minimum version of Xcode every April, so we expect to be able to remove this workaround in April 2024.

## Changelog:
[Internal] - Make Glog work with older versions of Xcode

Reviewed By: cortinico

Differential Revision: D50410487

fbshipit-source-id: 96145cdf9ba1bc75622403d3c06454d6d4bfd967

* Make the Fabric Interop layer work in Bridgeless (facebook#40732)

Summary:
Pull Request resolved: facebook#40732

This change allows the Fabric Interop Layer to work in bridgeless mode.

Given that the legacy components requires a Bridge to send events and use commands, this change simulates a bridge when it doesn't exists (i.e.: we are in bridgeless mode).

In order to make it work, we had to simulate a few elements from the Bridge and the UIManager.

## Changelog:
[iOS][Fixed] - Make the Fabric interop layer works in bridgeless mode.

## Facebook:
As an alternative approach, we could have created a `protocol RCTBridging`, have the `RCTBridge` conform to that protocol, and create a new type for it.
Practically this would have been much more cumbersome:
1. The [RCTBridge](https://www.internalfb.com/code/fbsource/[916531b9bf7a9943036807f7563c925b4c3e0101]/xplat/js/react-native-github/packages/react-native/React/Base/RCTBridge.h?lines=87-238) interface is quite big. All the props and method should be part of the protocol.
2. Extensions declared on RCTBridge would not have worked. For example, [`RCTBridge (RCTUIManager)`](https://www.internalfb.com/code/fbsource/[916531b9bf7a9943036807f7563c925b4c3e0101]/xplat/js/react-native-github/packages/react-native/React/Modules/RCTUIManager.h?lines=170-174).
3. It would require a major overhaul of the APIs, returning `id<RCTBridging>` in place of **every** function that takes/return an instance of `RCTBridge *`.

Clearly, not a feasible way to go.

Reviewed By: RSNara

Differential Revision: D50079929

fbshipit-source-id: 77d4370e9043c8e06b676bff3987511c042b0f9c

* [0.73.0-rc.3] Bump version numbers

* Remove deprecated jsinspector files (facebook#41159)

Summary:
Pull Request resolved: facebook#41159

Changelog: [Internal]

Cleanup from D48966244. The deprecated `jsinspector` files are no longer used by any code in either fbsource or `react-native`.

Reviewed By: hoxyq

Differential Revision: D50530796

fbshipit-source-id: b539b097cb6caf6c50a482fa93bf5d7886e76e52

* [LOCAL] Fix e2e script for 0.73

* [LOCAL] Run prettier

* [LOCAL] Update Podfile.lock

* Update Metro to ^0.80.0, don't pin to exact version

cherry-pick 7853b06

* cipolleschi/fix test script (facebook#41267)

* [0.73] Collection of Android picks for RC4 (facebook#41221)

Co-authored-by: Lulu Wu <luluwu@meta.com>
Co-authored-by: Gabriel Donadel <donadeldev@gmail.com>
fix `java.lang.NoSuchMethodError` for Bridgeless (facebook#41081)
resolved: facebook#41081
resolved: facebook#40999
resolved: facebook#41165
Fix RNTester not showing Redbox when Metro is not connected (facebook#41191)
resolved: facebook#41191
resolved: facebook#41190
resolved: facebook#41085
resolved: facebook#41206

* Build fixes

---------

Co-authored-by: Alex Hunt <hello@alexhunt.io>
Co-authored-by: Luna Wei <luwe@fb.com>
Co-authored-by: Dmitry Rykun <dmitryrykun@meta.com>
Co-authored-by: Lulu Wu <luluwu@meta.com>
Co-authored-by: Alex Hunt <alexeh@meta.com>
Co-authored-by: Arushi Kesarwani <arushikesarwani@meta.com>
Co-authored-by: Ramanpreet Nara <ramanpreet@meta.com>
Co-authored-by: Riccardo Cipolleschi <cipolleschi@meta.com>
Co-authored-by: Alex Hunt <huntie@meta.com>
Co-authored-by: David Vacca <dvacca@meta.com>
Co-authored-by: Distiller <distiller@static.198.206.135.134.macminivault.com>
Co-authored-by: Distiller <distiller@static.38.23.39.57.cyberlynk.net>
Co-authored-by: Distiller <distiller@static.38.23.38.177.cyberlynk.net>
Co-authored-by: Thibault Malbranche <malbranche.thibault@gmail.com>
Co-authored-by: Distiller <distiller@static.38.23.40.157.cyberlynk.net>
Co-authored-by: Nicola Corti <ncor@meta.com>
Co-authored-by: Nishan <nishanbende@gmail.com>
Co-authored-by: Luna Wei <luwe@meta.com>
Co-authored-by: Distiller <distiller@static.38.23.40.28.cyberlynk.net>
Co-authored-by: Nick Gerleman <ngerlem@meta.com>
Co-authored-by: Blake Friedman <blakef@meta.com>
Co-authored-by: 116-7 <1167@local>
Co-authored-by: UNIDY2002 <UNIDY2002@outlook.com>
Co-authored-by: Samuel Susla <samuelsusla@meta.com>
Co-authored-by: DeokHyeon Wi <dnl9807@gmail.com>
Co-authored-by: Oscar Franco <ospfranco@gmail.com>
Co-authored-by: Gabriel Donadel <donadeldev@gmail.com>
Co-authored-by: Riccardo Cipolleschi <riccardo.cipolleschi@gmail.com>
Co-authored-by: Oleksii <alexx271298@gmail.com>
Co-authored-by: Christoph Purrer <chpurrer@meta.com>
Co-authored-by: Distiller <distiller@static.38.39.184.83.cyberlynk.net>
Co-authored-by: Moti Zilberman <moti@meta.com>
Co-authored-by: Rob Hogan <rob@rhogan.net>
@arjun1194
Copy link

@adhamhassan99 are you working on react/views/textinput/TextInputTest ? if not, i would like to take it up.

@cortinico
Copy link
Contributor

Ping for @adhamhassan99 @ananta @AntoineDoubovetzky @cu8code @MCanhisares @Frenzyritz13
Please let me know by the end of this week if you're still working on your test or not, otherwise we'll release it

@MCanhisares
Copy link
Contributor

@cortinico I wasn't able to work on these for the past few weeks; I'll have bandwidth this weekend to finish it if there's time for that, otherwise someone can pick it up and finish it by then

@AntoineDoubovetzky
Copy link

@cortinico It's quite busy these days, if anyone who has more free time wants to work on this you can remove me from the task.

@arjun1194
Copy link

@cortinico can i pick this up ? i am free for a few weeks, i will be able to work on it. however, this is my first time working on a n opensource project so i might need some help starting out. thanks.

@cortinico
Copy link
Contributor

@cortinico can i pick this up ? i am free for a few weeks, i will be able to work on it. however, this is my first time working on a n opensource project so i might need some help starting out. thanks.

@arjun1194 yup but you need to specify which file you want to work on.

@arjun1194
Copy link

arjun1194 commented Nov 27, 2023

I will be able to work on any file that no body else is working on right now, as i see all files already marked with "assigned to someone". i am not able to decide

@cortinico
Copy link
Contributor

I will be able to work on any file that no body else is working on right now, as i see all files already marked with "assigned to someone". i am not able to decide

I've assigned you to @UIManagerModuleTest

I've also released your files @adhamhassan99 @ananta @cu8code @Frenzyritz13
Please let me know by the end of this week if you're still working on your test or not, otherwise we'll release it

@giolaq
Copy link
Contributor

giolaq commented Nov 28, 2023

✋ I can take TextInputTest.java. @cortinico feel free to assign it to me if it's available

@arjun1194
Copy link

arjun1194 commented Nov 29, 2023

@cortinico can you help me with some already merged PRs, want to check the code style that you guys follow. since it is my first time contributing to this project

edit - i found this PR

facebook-github-bot pushed a commit that referenced this issue Nov 29, 2023
Summary:
This PR converts to kotlin the java code for LayoutPropertyApplicatorTest, as requested in: #38825

## Changelog:

[INTERNAL][CHANGED]: Convert LayoutPropertyApplicatorTest to Kotlin

Pull Request resolved: #41649

Test Plan: `./gradlew :packages:react-native:ReactAndroid:test `

Reviewed By: rshest

Differential Revision: D51614921

Pulled By: cortinico

fbshipit-source-id: 06ee403a496f34afc9abeabc0c406391e316538a
@arjun1194
Copy link

@cortinico in the file, react/uimanager/UIManagerModuleTest , there is a comment @Ignore // TODO T14964130 can i get some more information about this ? i am not able to run the tests after removing the annotation.

@cortinico
Copy link
Contributor

@cortinico in the file, react/uimanager/UIManagerModuleTest , there is a comment @Ignore // TODO T14964130 can i get some more information about this ? i am not able to run the tests after removing the annotation.

Let me get back to you on this next week

@Aditya8840
Copy link

Aditya8840 commented Dec 2, 2023

Hi, I'd like to claim react/views/text/ReactTextTest.java

@moskalakamil
Copy link
Contributor

Hey! I'd like to work on react/uimanager/ReactPropForShadowNodeSetterTest

@KrzysztofMoch
Copy link
Contributor

Hey👋, I would like to work on NetworkingModuleTest

facebook-github-bot pushed a commit that referenced this issue Dec 5, 2023
Summary:
Migrated `NetworkingModuleTest` to kotlin (part of #38825)

## Changelog:
[Android] [CHANGED] - NetworkingModuleTest.java -> NetworkingModuleTest.kt

Pull Request resolved: #41792

Test Plan: `./gradlew :packages:react-native:ReactAndroid:test`

Reviewed By: cortinico

Differential Revision: D51847402

Pulled By: rshest

fbshipit-source-id: 7a1938c575a7e5b22b6ff578e1f8d26b600f2c27
@cortinico
Copy link
Contributor

Hey @giolaq @arjun1194 @MCanhisares @Aditya8840 I've reviewed the remaining tests that were still suppressed and I've ended up deleting your tests as they were too legacy and they had to be rewritten from scratch (so it was not worth to convert them to Kotlin)

facebook-github-bot pushed a commit that referenced this issue Dec 7, 2023
Summary:
This PR convert `ReactPropForShadowNodeSetterTest` to kotlin as part of #38825

## Changelog:

[INTERNAL] [CHANGED] - Convert ReactPropForShadowNodeSetterTest to kotlin

<!-- Help reviewers and the release process by writing your own changelog entry.

Pick one each for the category and type tags:

[ANDROID|GENERAL|IOS|INTERNAL] [BREAKING|ADDED|CHANGED|DEPRECATED|REMOVED|FIXED|SECURITY] - Message

For more details, see:
https://reactnative.dev/contributing/changelogs-in-pull-requests

Pull Request resolved: #41774

Test Plan:
```
./gradlew :packages:react-native:ReactAndroid:test
```

Reviewed By: NickGerleman

Differential Revision: D51882685

Pulled By: cortinico

fbshipit-source-id: ff1cce824dc342200f1f5ccbb297b955747b10c8
@cortinico
Copy link
Contributor

Thank you all for the support!
We were able to migrate to Kotlin more than 20 tests thanks to the community support!
We'll be back in the future with more files to migrate

@cortinico cortinico unpinned this issue Dec 7, 2023
Othinn pushed a commit to Othinn/react-native that referenced this issue Jan 9, 2024
Summary:
This PR converts to kotlin the java code for LayoutPropertyApplicatorTest, as requested in: facebook#38825

## Changelog:

[INTERNAL][CHANGED]: Convert LayoutPropertyApplicatorTest to Kotlin

Pull Request resolved: facebook#41649

Test Plan: `./gradlew :packages:react-native:ReactAndroid:test `

Reviewed By: rshest

Differential Revision: D51614921

Pulled By: cortinico

fbshipit-source-id: 06ee403a496f34afc9abeabc0c406391e316538a
Othinn pushed a commit to Othinn/react-native that referenced this issue Jan 9, 2024
Summary:
Migrated `NetworkingModuleTest` to kotlin (part of facebook#38825)

## Changelog:
[Android] [CHANGED] - NetworkingModuleTest.java -> NetworkingModuleTest.kt

Pull Request resolved: facebook#41792

Test Plan: `./gradlew :packages:react-native:ReactAndroid:test`

Reviewed By: cortinico

Differential Revision: D51847402

Pulled By: rshest

fbshipit-source-id: 7a1938c575a7e5b22b6ff578e1f8d26b600f2c27
Othinn pushed a commit to Othinn/react-native that referenced this issue Jan 9, 2024
…k#41774)

Summary:
This PR convert `ReactPropForShadowNodeSetterTest` to kotlin as part of facebook#38825

## Changelog:

[INTERNAL] [CHANGED] - Convert ReactPropForShadowNodeSetterTest to kotlin

<!-- Help reviewers and the release process by writing your own changelog entry.

Pick one each for the category and type tags:

[ANDROID|GENERAL|IOS|INTERNAL] [BREAKING|ADDED|CHANGED|DEPRECATED|REMOVED|FIXED|SECURITY] - Message

For more details, see:
https://reactnative.dev/contributing/changelogs-in-pull-requests

Pull Request resolved: facebook#41774

Test Plan:
```
./gradlew :packages:react-native:ReactAndroid:test
```

Reviewed By: NickGerleman

Differential Revision: D51882685

Pulled By: cortinico

fbshipit-source-id: ff1cce824dc342200f1f5ccbb297b955747b10c8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good first issue Interested in collaborating? Take a stab at fixing one of these issues. Help Wanted :octocat: Issues ideal for external contributors. ☂️ Umbrella To label issues that serve as coordination point and drivers for tasks in the react-native repo
Projects
None yet
Development

No branches or pull requests