fix(xcode): Only parse Plist when required during RN source maps upload

src/commands/react_native/xcode.rs

@@ -201,12 +201,6 @@ pub fn execute(matches: &ArgMatches) -> Result<()> {
         return Ok(());
     }
 
-    info!("Parsing Info.plist");
-    let plist = match InfoPlist::discover_from_env()? {
-        Some(plist) => plist,
-        None => bail!("Could not find info.plist"),
-    };
-    info!("Parse result from Info.plist: {:?}", &plist);
     let report_file = TempFile::create()?;
     let node = find_node();
     info!("Using node interpreter '{}'", &node);
@@ -371,21 +365,43 @@ pub fn execute(matches: &ArgMatches) -> Result<()> {
                 chunk_upload_options: chunk_upload_options.as_ref(),
             })?;
         } else {
-            let dist = dist_from_env.unwrap_or_else(|_| plist.build().to_string());
-            let release_name = release_from_env.unwrap_or(format!(
-                "{}@{}+{}",
-                plist.bundle_id(),
-                plist.version(),
-                dist
-            ));
+            let mut dist: Option<String> = None;
+            let mut release_name: Option<String> = None;
+
+            if dist_from_env.is_err() && release_from_env.is_err() {
+              info!("Parsing Info.plist");
+              let plist = match InfoPlist::discover_from_env()? {
+                  Some(plist) => plist,
+                  None => bail!("Could not find info.plist"),
+              };
+              info!("Parse result from Info.plist: {:?}", &plist);
+
+              let dist_string = plist.build().to_string();
+              dist = Some(dist_string.clone());
+              release_name = Some(format!(
+                  "{}@{}+{}",
+                  plist.bundle_id(),
+                  plist.version(),
+                  dist_string
+              ));
+            }
+
+            if let Ok(_dist) = dist_from_env {
+              info!("Using dist from `SENTRY_DIST` env");
+              dist = Some(_dist);
+            }
+            if let Ok(_release) = release_from_env {
+              info!("Using release from `SENTRY_RELEASE` env");
+              release_name = Some(_release);
+            }
 
             match matches.get_many::<String>("dist") {
                 None => {
                     processor.upload(&UploadContext {
                         org: &org,
                         project: Some(&project),
-                        release: Some(&release_name),
-                        dist: Some(&dist),
+                        release: release_name.as_deref(),
+                        dist: dist.as_deref(),
                         note: None,
                         wait,
                         max_wait,
@@ -398,7 +414,7 @@ pub fn execute(matches: &ArgMatches) -> Result<()> {
                         processor.upload(&UploadContext {
                             org: &org,
                             project: Some(&project),
-                            release: Some(&release_name),
+                            release: release_name.as_deref(),
                             dist: Some(dist),
                             note: None,
                             wait,

tests/integration/_cases/react_native/xcode-upload-source-maps-invalid-plist.trycmd

@@ -0,0 +1,15 @@
+```
+$ CONFIGURATION=Release sentry-cli react-native xcode tests/integration/_fixtures/react_native/react-native-xcode.sh --force-foreground
+? failed
+react-native-xcode.sh called with args: 
+Using React Native Packager bundle and source map.
+Processing react-native sourcemaps for Sentry upload.
+> Analyzing 2 sources
+> Rewriting sources
+> Adding source map references
+error: Could not find info.plist
+
+Add --log-level=[info|debug] or export SENTRY_LOG_LEVEL=[info|debug] to see more output.
+Please attach the full debug log to all bug reports.
+
+```

tests/integration/_cases/react_native/xcode-upload-source-maps-release_and_dist_from_env.trycmd

@@ -0,0 +1,26 @@
+```
+$ CONFIGURATION=Release SENTRY_RELEASE=test-release SENTRY_DIST=test-dist sentry-cli react-native xcode tests/integration/_fixtures/react_native/react-native-xcode.sh --force-foreground
+? success
+react-native-xcode.sh called with args: 
+Using React Native Packager bundle and source map.
+Processing react-native sourcemaps for Sentry upload.
+> Analyzing 2 sources
+> Rewriting sources
+> Adding source map references
+> Bundled 2 files for upload
+> Bundle ID: [..]-[..]-[..]-[..]-[..]
+> Uploaded files to Sentry
+> File upload complete (processing pending on server)
+> Organization: wat-org
+> Project: wat-project
+> Release: test-release
+> Dist: test-dist
+> Upload type: artifact bundle
+
+Source Map Upload Report
+  Scripts
+    ~/react-native-xcode-bundle.js.map
+  Minified Scripts
+    ~/react-native-xcode-bundle.js (sourcemap at react-native-xcode-bundle.map)
+
+```

tests/integration/_fixtures/react_native/react-native-xcode.sh

@@ -0,0 +1,8 @@
+#!/bin/sh
+
+echo "react-native-xcode.sh called with args: $@"
+
+echo '{
+  "packager_bundle_path": "tests/integration/_fixtures/react_native/react-native-xcode-bundle.js",
+  "packager_sourcemap_path": "tests/integration/_fixtures/react_native/react-native-xcode-bundle.js.map"
+}' > "$SENTRY_RN_SOURCEMAP_REPORT"

tests/integration/react_native/mod.rs

@@ -1,6 +1,8 @@
 #[cfg(target_os = "macos")]
 use crate::integration::register_test;
 
+mod xcode;
+
 #[test]
 #[cfg(target_os = "macos")]
 fn xcode_wrap_call_minimum() {

tests/integration/react_native/xcode.rs

@@ -0,0 +1,30 @@
+#[cfg(target_os = "macos")]
+use mockito::Mock;
+#[cfg(target_os = "macos")]
+use crate::integration::register_test;
+#[cfg(target_os = "macos")]
+use crate::integration::{mock_common_upload_endpoints, ChunkOptions, ServerBehavior};
+
+#[test]
+#[cfg(target_os = "macos")]
+fn xcode_upload_source_maps_missing_plist() {
+    let _upload_endpoints =
+        mock_common_upload_endpoints(ServerBehavior::Modern, ChunkOptions::default());
+    register_test("react_native/xcode-upload-source-maps-invalid-plist.trycmd");
+}
+
+#[test]
+#[cfg(target_os = "macos")]
+fn xcode_upload_source_maps_release_and_dist_from_env() {
+    let upload_endpoints =
+        mock_common_upload_endpoints(ServerBehavior::Modern, ChunkOptions::default());
+    register_test("react_native/xcode-upload-source-maps-release_and_dist_from_env.trycmd");
+    assert_endpoints(&upload_endpoints);
+}
+
+#[cfg(target_os = "macos")]
+pub fn assert_endpoints(mocks: &[Mock]) {
+    for mock in mocks {
+        mock.assert();
+    }
+}