fix(ios): propagate error from BootedUDID instead of silently swallowing

internal/ios/ios.go

@@ -55,12 +55,15 @@ func EnsureSimulator(ctx context.Context, deviceName string, stdout io.Writer) e
 }
 
 // BootedUDID returns the UDID of the currently booted iOS simulator, or "" if none is booted.
-func BootedUDID(ctx context.Context) string {
+func BootedUDID(ctx context.Context) (string, error) {
-	d, _ := bootedSimulator(ctx)
+	d, err := bootedSimulator(ctx)
-	if d == nil {
+	if err != nil {
-		return ""
+		return "", err
 	}
-	return d.UDID
+	if d == nil {
+		return "", nil
+	}
+	return d.UDID, nil
 }
 
 func bootedSimulator(ctx context.Context) (*simDevice, error) {

internal/ios/ios_test.go

@@ -82,7 +82,10 @@ func TestPickSimulator_EmptyList(t *testing.T) {
 func TestBootedUDID_CanceledContext(t *testing.T) {
 	ctx, cancel := context.WithCancel(context.Background())
 	cancel()
-	udid := BootedUDID(ctx)
+	udid, err := BootedUDID(ctx)
+	if err == nil {
+		t.Error("expected error for canceled context, got nil")
+	}
 	if udid != "" {
 		t.Errorf("expected empty UDID on canceled context, got %q", udid)
 	}

internal/testrun/driver.go

@@ -41,7 +41,11 @@ func buildDriver(ctx context.Context, options Options, stdout io.Writer) (driver
 		"--platform", options.Platform,
 	}
 	if options.Platform == "ios" {
-		if udid := ios.BootedUDID(ctx); udid != "" {
+		udid, err := ios.BootedUDID(ctx)
+		if err != nil {
+			return nil, nil, fmt.Errorf("find booted simulator: %w", err)
+		}
+		if udid != "" {
 			sidecarArgs = append(sidecarArgs, "--udid", udid)
 		}
 	}