React Native 0.78 support

.buildkite/jobs/pipeline.android_rn_77.yml

@@ -0,0 +1,12 @@
+  - label: ":android: Android (RN 0.77.3)"
+    env:
+      JAVA_HOME: /opt/openjdk/jdk-17.0.9.jdk/Contents/Home/
+      REACT_NATIVE_VERSION: 0.77.3
+    command:
+    - "nvm install"
+    - "./scripts/ci.android.sh"
+    key: "android_rn_77"
+    timeout_in_minutes: 90
+    artifact_paths: "/Users/builder/uibuilder/work/artifacts/**/*"
+
+

.buildkite/jobs/pipeline.android_rn_78.yml

@@ -0,0 +1,12 @@
+  - label: ":android: Android (RN 0.78.3)"
+    env:
+      JAVA_HOME: /opt/openjdk/jdk-17.0.9.jdk/Contents/Home/
+      REACT_NATIVE_VERSION: 0.78.3
+    command:
+    - "nvm install"
+    - "./scripts/ci.android.sh"
+    key: "android"
+    timeout_in_minutes: 90
+    artifact_paths: "/Users/builder/uibuilder/work/artifacts/**/*"
+
+

.buildkite/jobs/pipeline.ios_rn_77.yml

@@ -0,0 +1,11 @@
+  - label: ":ios: iOS (RN 0.77.3)"
+    env:
+      REACT_NATIVE_VERSION: 0.77.3
+    command:
+    - "nvm install"
+    - "./scripts/ci.ios.sh"
+    key: "ios_rn_77"
+    timeout_in_minutes: 90
+    artifact_paths: "/Users/builder/uibuilder/work/artifacts/**/*"
+
+

.buildkite/jobs/pipeline.ios_rn_78.yml

@@ -0,0 +1,11 @@
+  - label: ":ios: iOS (RN 0.78.3)"
+    env:
+      REACT_NATIVE_VERSION: 0.78.3
+    command:
+    - "nvm install"
+    - "./scripts/ci.ios.sh"
+    key: "ios"
+    timeout_in_minutes: 90
+    artifact_paths: "/Users/builder/uibuilder/work/artifacts/**/*"
+
+

.buildkite/jobs/pipeline.publish.yml

@@ -0,0 +1,15 @@
+  - wait
+  - label: ":package: Publish"
+    env:
+      JAVA_HOME: /opt/openjdk/jdk-17.0.9.jdk/Contents/Home/
+    if: "build.pull_request.id == null"
+    command:
+    - "nvm install"
+    - "npm install"
+    - "npm run release"    
+    depends_on:
+    - ios
+    - android
+    timeout_in_minutes: 90
+
+

.buildkite/jobs/pipeline.release.yml

@@ -0,0 +1,20 @@
+  - block: ":rocket: Release!"
+    prompt: "Fill out the details for release"
+    if: 'build.message =~ /^release\$/i'
+    fields:
+      - text: "VERSION"
+        key: "version"
+      - text: "NPM_TAG"
+        key: "npm-tag"
+        default: 'null'
+        required: false
+      - text: "BUILD_DOCUMENTATION_VERSION"
+        key: "build-documentation-version"
+        default: 'null'
+        hint: 'Leave NULL if no version is specified'
+      - text: "REMOVE_DOCUMENTATION_VERSION"
+        key: "remove-documentation-version"
+        default: 'null'
+        hint: 'Leave NULL if no version is specified'
+
+

.buildkite/pipeline.sh

@@ -0,0 +1,12 @@
+#!/bin/bash -e
+
+echo "steps:"
+
+cat .buildkite/jobs/pipeline.release.yml
+cat .buildkite/jobs/pipeline.android_rn_77.yml
+cat .buildkite/jobs/pipeline.android_rn_78.yml
+cat .buildkite/jobs/pipeline.ios_rn_77.yml
+cat .buildkite/jobs/pipeline.ios_rn_78.yml
+cat .buildkite/jobs/pipeline.publish.yml
+
+

integration/redux/Redux.test.js

@@ -1,6 +1,6 @@
 const React = require('react');
 require('react-native');
-const renderer = require('react-test-renderer');
+const { render } = require('@testing-library/react-native');
 const { Provider } = require('react-redux');
 const { Navigation } = require('../../lib/src/index');
 
@@ -30,8 +30,8 @@ describe('redux support', () => {
       store.reduxStore
     );
 
-    const tree = renderer.create(<HOC />);
-    expect(tree.toJSON().children).toEqual(['no name']);
+    const { getByText } = render(<HOC />);
+    expect(getByText('no name')).toBeTruthy();
   });
 
   it('passes props into wrapped components', () => {
@@ -50,45 +50,60 @@ describe('redux support', () => {
       <HOC {...props} />
     ))();
 
-    const tree = renderer.create(
+    const { getByText } = render(
       <CompFromNavigation componentId="componentId" renderCountIncrement={renderCountIncrement} />
     );
-    expect(tree.toJSON().children).toEqual(['no name']);
+    expect(getByText('no name')).toBeTruthy();
     expect(renderCountIncrement).toHaveBeenCalledTimes(1);
   });
 
   it('rerenders as a result of an underlying state change (by selector)', () => {
     const renderCountIncrement = jest.fn();
-    const tree = renderer.create(
+    const { getByText, rerender } = render(
       <Provider store={store.reduxStore}>
         <MyConnectedComponent renderCountIncrement={renderCountIncrement} />
       </Provider>
     );
 
-    expect(tree.toJSON().children).toEqual(['no name']);
+    expect(getByText('no name')).toBeTruthy();
     expect(renderCountIncrement).toHaveBeenCalledTimes(1);
 
     store.reduxStore.dispatch({ type: 'redux.MyStore.setName', name: 'Bob' });
     expect(store.selectors.getName(store.reduxStore.getState())).toEqual('Bob');
-    expect(tree.toJSON().children).toEqual(['Bob']);
+
+    // Re-render to see the updated state
+    rerender(
+      <Provider store={store.reduxStore}>
+        <MyConnectedComponent renderCountIncrement={renderCountIncrement} />
+      </Provider>
+    );
+    expect(getByText('Bob')).toBeTruthy();
 
     expect(renderCountIncrement).toHaveBeenCalledTimes(2);
   });
 
   it('rerenders as a result of an underlying state change with a new key', () => {
     const renderCountIncrement = jest.fn();
-    const tree = renderer.create(
+    const { queryByText, rerender } = render(
       <Provider store={store.reduxStore}>
         <MyConnectedComponent printAge={true} renderCountIncrement={renderCountIncrement} />
       </Provider>
     );
 
-    expect(tree.toJSON().children).toEqual(null);
+    // Initially should show nothing (null children means no text)
+    expect(queryByText('30')).toBeNull();
     expect(renderCountIncrement).toHaveBeenCalledTimes(1);
 
     store.reduxStore.dispatch({ type: 'redux.MyStore.setAge', age: 30 });
     expect(store.selectors.getAge(store.reduxStore.getState())).toEqual(30);
-    expect(tree.toJSON().children).toEqual(['30']);
+
+    // Re-render to see the updated state
+    rerender(
+      <Provider store={store.reduxStore}>
+        <MyConnectedComponent printAge={true} renderCountIncrement={renderCountIncrement} />
+      </Provider>
+    );
+    expect(queryByText('30')).toBeTruthy();
 
     expect(renderCountIncrement).toHaveBeenCalledTimes(2);
   });

integration/remx/remx.test.js

@@ -1,6 +1,6 @@
 const React = require('react');
 require('react-native');
-const renderer = require('react-test-renderer');
+const { render } = require('@testing-library/react-native');
 const { Navigation } = require('../../lib/src/index');
 
 describe('remx support', () => {
@@ -13,39 +13,47 @@ describe('remx support', () => {
   });
 
   it('renders normally', () => {
-    const tree = renderer.create(<MyConnectedComponent />);
-    const tree2 = renderer.create(<MyConnectedComponent />);
-    expect(tree.toJSON().children).toEqual(['no name']);
+    const { getByText } = render(<MyConnectedComponent />);
+    const { getByText: getByText2 } = render(<MyConnectedComponent />);
+    expect(getByText('no name')).toBeTruthy();
+    expect(getByText2('no name')).toBeTruthy();
   });
 
   it('rerenders as a result of an underlying state change (by selector)', () => {
     const renderCountIncrement = jest.fn();
-    const tree = renderer.create(
+    const { getByText, rerender } = render(
       <MyConnectedComponent renderCountIncrement={renderCountIncrement} />
     );
 
-    expect(tree.toJSON().children).toEqual(['no name']);
+    expect(getByText('no name')).toBeTruthy();
     expect(renderCountIncrement).toHaveBeenCalledTimes(1);
 
     store.setters.setName('Bob');
     expect(store.getters.getName()).toEqual('Bob');
-    expect(tree.toJSON().children).toEqual(['Bob']);
+
+    // Re-render to get updated content
+    rerender(<MyConnectedComponent renderCountIncrement={renderCountIncrement} />);
+    expect(getByText('Bob')).toBeTruthy();
 
     expect(renderCountIncrement).toHaveBeenCalledTimes(2);
   });
 
   it('rerenders as a result of an underlying state change with a new key', () => {
     const renderCountIncrement = jest.fn();
-    const tree = renderer.create(
+    const { queryByText, rerender } = render(
       <MyConnectedComponent printAge={true} renderCountIncrement={renderCountIncrement} />
     );
 
-    expect(tree.toJSON().children).toEqual(null);
+    // Initially should show nothing (null children means no text)
+    expect(queryByText('30')).toBeNull();
     expect(renderCountIncrement).toHaveBeenCalledTimes(1);
 
     store.setters.setAge(30);
     expect(store.getters.getAge()).toEqual(30);
-    expect(tree.toJSON().children).toEqual(['30']);
+
+    // Re-render to get updated content
+    rerender(<MyConnectedComponent printAge={true} renderCountIncrement={renderCountIncrement} />);
+    expect(queryByText('30')).toBeTruthy();
 
     expect(renderCountIncrement).toHaveBeenCalledTimes(2);
   });

lib/android/app/src/main/java/com/reactnativenavigation/react/modal/ModalContentLayout.kt

@@ -16,7 +16,7 @@ class ModalContentLayout(context: Context?) : ReactViewGroup(context), RootView{
 
     private val mJSTouchDispatcher = JSTouchDispatcher(this)
 
-    override fun onChildStartedNativeGesture(child: View, androidEvent: MotionEvent) {
+    override fun onChildStartedNativeGesture(child: View?, androidEvent: MotionEvent) {
         mJSTouchDispatcher.onChildStartedNativeGesture(androidEvent, this.getEventDispatcher())
     }
     override fun onChildStartedNativeGesture(androidEvent: MotionEvent) {
@@ -31,7 +31,7 @@ class ModalContentLayout(context: Context?) : ReactViewGroup(context), RootView{
         return UIManagerHelper.getEventDispatcher(reactContext, UIManagerType.FABRIC) ?: throw IllegalStateException("EventDispatcher for Fabric UI Manager is not found")
     }
 
-    override fun handleException(t: Throwable?) {
+    override fun handleException(t: Throwable) {
         getReactContext().handleException(RuntimeException(t))
     }
 

lib/ios/RNNAppDelegate.mm

@@ -13,7 +13,21 @@
 #import <React/RCTSurfacePresenterStub.h>
 #import <React/RCTSurfacePresenterBridgeAdapter.h>
 #import <ReactCommon/RCTTurboModuleManager.h>
+
+
+
+#if __has_include(<React-RCTAppDelegate/RCTReactNativeFactory.h>)
+#import <React-RCTAppDelegate/RCTAppDelegate.h>
+#import <React-RCTAppDelegate/RCTReactNativeFactory.h>
+#elif __has_include(<React_RCTAppDelegate/RCTReactNativeFactory.h>)
+#import <React_RCTAppDelegate/RCTAppDelegate.h>
+#import <React_RCTAppDelegate/RCTReactNativeFactory.h>
+#else
+// RN 0.77 support
+#define RN077
 #import <react/config/ReactNativeConfig.h>
+#endif
+
 #import <react/renderer/runtimescheduler/RuntimeScheduler.h>
 #import <react/renderer/runtimescheduler/RuntimeSchedulerCallInvoker.h>
 #import <React/RCTSurfacePresenter.h>
@@ -37,15 +51,17 @@ @implementation RNNAppDelegate
 - (BOOL)application:(UIApplication *)application
     didFinishLaunchingWithOptions:(NSDictionary *)launchOptions {
 
+#ifdef RN077
     [self _setUpFeatureFlags];
-
-    // Copied from RCTAppDelegate, it private inside it
     self.rootViewFactory = [self createRCTRootViewFactory];
-
     [RCTComponentViewFactory currentComponentViewFactory].thirdPartyFabricComponentsProvider = self;
-
     RCTAppSetupPrepareApp(application, self.newArchEnabled);
     RCTSetNewArchEnabled(TRUE);
+#else
+    self.reactNativeFactory = [RCTReactNativeFactory new];
+    self.reactNativeFactory = [self.reactNativeFactory initWithDelegate:self];
+#endif
+
     RCTEnableTurboModuleInterop(YES);
     RCTEnableTurboModuleInteropBridgeProxy(YES);
 
@@ -56,34 +72,37 @@ - (BOOL)application:(UIApplication *)application
     return YES;
 }
 
+- (NSURL *)sourceURLForBridge:(RCTBridge *)bridge {
+	[NSException raise:@"RCTBridgeDelegate::sourceURLForBridge not implemented"
+				format:@"Subclasses must implement a valid sourceURLForBridge method"];
+	return nil;
+}
+
+- (BOOL)concurrentRootEnabled {
+	return true;
+}
+
+
+
+#ifdef RN077
 - (RCTRootViewFactory *)createRCTRootViewFactory
 {
   __weak __typeof(self) weakSelf = self;
   RCTBundleURLBlock bundleUrlBlock = ^{
-	RCTAppDelegate *strongSelf = weakSelf;
-	return strongSelf.bundleURL;
+    RCTAppDelegate *strongSelf = weakSelf;
+    return strongSelf.bundleURL;
   };
 
   RCTRootViewFactoryConfiguration *configuration =
-	  [[RCTRootViewFactoryConfiguration alloc] initWithBundleURLBlock:bundleUrlBlock
-													   newArchEnabled:self.newArchEnabled];
+      [[RCTRootViewFactoryConfiguration alloc] initWithBundleURLBlock:bundleUrlBlock
+                                                       newArchEnabled:self.newArchEnabled];
 
 
   return [[RCTRootViewFactory alloc] initWithConfiguration:configuration andTurboModuleManagerDelegate:self];
 }
 
-- (NSURL *)sourceURLForBridge:(RCTBridge *)bridge {
-	[NSException raise:@"RCTBridgeDelegate::sourceURLForBridge not implemented"
-				format:@"Subclasses must implement a valid sourceURLForBridge method"];
-	return nil;
-}
-
-- (BOOL)concurrentRootEnabled {
-	return true;
-}
 
 #pragma mark - Feature Flags
-
 class RCTAppDelegateBridgelessFeatureFlags : public facebook::react::ReactNativeFeatureFlagsDefaults {
  public:
   bool enableBridgelessArchitecture() override
@@ -113,6 +132,7 @@ - (void)_setUpFeatureFlags
     facebook::react::ReactNativeFeatureFlags::override(
         std::make_unique<RCTAppDelegateBridgelessFeatureFlags>());
 }
+#endif
 
 @end