Move use_flipper logic inside use_react_native and simplify the Flipper dependencies logic

packages/rn-tester/Podfile

@@ -32,8 +32,10 @@ def pods(options = {})
     path: @prefix_path,
     fabric_enabled: fabric_enabled,
     hermes_enabled: hermes_enabled,
+    flipper_configuration: FlipperConfiguation.enabled,
     app_path: "#{Dir.pwd}",
     config_file_dir: "#{Dir.pwd}/node_modules",
+    production: !ENV['PRODUCTION'].nil?
   )
   pod 'ReactCommon/turbomodule/samples', :path => "#{@prefix_path}/ReactCommon"
 
@@ -48,9 +50,6 @@ end
 
 target 'RNTester' do
   pods()
-  if !USE_FRAMEWORKS
-    use_flipper!
-  end
 end
 
 target 'RNTesterUnitTests' do

scripts/cocoapods/__tests__/flipper-test.rb

@@ -16,24 +16,15 @@ def setup
     # =========================== #
     # TEST - Install Dependencies #
     # =========================== #
-    def test_installFlipperDependencies_whenProductionIsFalse_installDependencies
+    def test_installFlipperDependencies_installDependencies
         # Act
-        install_flipper_dependencies(false, '../..')
+        install_flipper_dependencies('../..')
 
         # Assert
         assert_equal($podInvocationCount, 1)
         assert_equal($podInvocation['React-Core/DevSupport'][:path], "../../" )
     end
 
-    def test_installFlipperDependencies_whenProductionIsTrue_skipDependencies
-        # Act
-        install_flipper_dependencies(true, '../..')
-
-        # Assert
-        assert_equal($podInvocationCount, 0)
-        assert_true($podInvocation.empty?)
-    end
-
     # ======================= #
     # TEST - Use Flipper Pods #
     # ======================= #

scripts/cocoapods/flipper.rb

@@ -22,10 +22,8 @@
 #
 # @parameter production: a boolean that indicates whether we are in production or not.
 # @parameter pathToReactNative: the path to the React Native installation
-def install_flipper_dependencies(production, pathToReactNative)
-    unless production
-        pod 'React-Core/DevSupport', :path => "#{pathToReactNative}/"
-    end
+def install_flipper_dependencies(pathToReactNative)
+    pod 'React-Core/DevSupport', :path => "#{pathToReactNative}/"
 end
 
 
@@ -92,3 +90,23 @@ def flipper_post_install(installer)
         end
     end
 end
+
+class FlipperConfiguation
+    attr_reader :flipper_enabled
+    attr_reader :configurations
+    attr_reader :versions
+
+    def initialize(flipper_enabled, configurations, versions)
+        @flipper_enabled = flipper_enabled
+        @configurations = configurations
+        @versions = versions
+    end
+
+    def self.enabled(configurations = ["Debug"], versions = {})
+        return FlipperConfiguation.new(true, configurations, versions)
+    end
+
+    def self.disabled
+        return FlipperConfiguation.new(false, [], {})
+    end
+end

scripts/react_native_pods.rb

@@ -33,6 +33,8 @@ def use_react_native! (options={})
   # Include Hermes dependencies
   hermes_enabled = options[:hermes_enabled] ||= false
 
+  flipper_configuration = options[:flipper_configuration] ||= FlipperConfiguation.disabled
+
   if `/usr/sbin/sysctl -n hw.optional.arm64 2>&1`.to_i == 1 && !RUBY_PLATFORM.include?('arm64')
     Pod::UI.warn 'Do not use "pod install" from inside Rosetta2 (x86_64 emulation on arm64).'
     Pod::UI.warn ' - Emulated x86_64 is slower than native arm64'
@@ -59,8 +61,6 @@ def use_react_native! (options={})
   pod 'React-RCTVibration', :path => "#{prefix}/Libraries/Vibration"
   pod 'React-Core/RCTWebSocket', :path => "#{prefix}/"
 
-  install_flipper_dependencies(production, prefix)
-
   pod 'React-bridging', :path => "#{prefix}/ReactCommon/react/bridging"
   pod 'React-cxxreact', :path => "#{prefix}/ReactCommon/cxxreact"
   pod 'React-jsi', :path => "#{prefix}/ReactCommon/jsi"
@@ -108,6 +108,11 @@ def use_react_native! (options={})
     pod 'libevent', '~> 2.1.12'
   end
 
+  if flipper_configuration.flipper_enabled
+    install_flipper_dependencies(prefix)
+    use_flipper_pods(flipper_configuration.versions, :configurations => flipper_configuration.configurations)
+  end
+
   pods_to_update = LocalPodspecPatch.pods_to_update(options)
   if !pods_to_update.empty?
     if Pod::Lockfile.public_instance_methods.include?(:detect_changes_with_podfile)
@@ -122,6 +127,7 @@ def get_default_flags()
   flags = {
     :fabric_enabled => false,
     :hermes_enabled => false,
+    :flipper_configuration => FlipperConfiguation.disabled
   }
 
   if ENV['RCT_NEW_ARCH_ENABLED'] == '1'