feat: Apply release and environment from options

sentry-core/src/main/java/io/sentry/core/SentryClient.java

@@ -35,6 +35,14 @@ public SentryClient(SentryOptions options, @Nullable AsyncConnection connection)
   public SentryId captureEvent(SentryEvent event, @Nullable Scope scope) {
     log(options.getLogger(), SentryLevel.DEBUG, "Capturing event: %s", event.getEventId());
 
+    // TODO: To be done in the main processor
+    if (event.getRelease() == null) {
+      event.setRelease(options.getRelease());
+    }
+    if (event.getEnvironment() == null) {
+      event.setEnvironment(options.getEnvironment());
+    }
+
     for (EventProcessor processor : options.getEventProcessors()) {
       processor.process(event);
     }

sentry-core/src/main/java/io/sentry/core/SentryOptions.java

@@ -20,6 +20,8 @@ public class SentryOptions {
   private String sentryClientName;
   private BeforeSecondCallback beforeSend;
   private String cacheDirPath;
+  private String release;
+  private String environment;
 
   public void addEventProcessor(EventProcessor eventProcessor) {
     eventProcessors.add(eventProcessor);
@@ -120,6 +122,22 @@ public void setCacheDirPath(String cacheDirPath) {
     this.cacheDirPath = cacheDirPath;
   }
 
+  public String getRelease() {
+    return release;
+  }
+
+  public void setRelease(String release) {
+    this.release = release;
+  }
+
+  public String getEnvironment() {
+    return environment;
+  }
+
+  public void setEnvironment(String environment) {
+    this.environment = environment;
+  }
+
   public interface BeforeSecondCallback {
     SentryEvent execute(SentryEvent event);
   }

sentry-core/src/test/java/io/sentry/core/SentryClientTest.kt

@@ -111,4 +111,49 @@ class SentryClientTest {
         sut.captureMessage(actual)
         assertEquals(actual, sentEvent!!.message.formatted)
     }
+
+    @Test
+    fun `when event has release, value from options not applied`() {
+        val event = SentryEvent()
+        val expected = "original"
+        fixture.sentryOptions.environment = "not to be applied"
+        event.release = expected
+        val sut = fixture.getSut()
+        sut.captureEvent(event)
+        verify(fixture.connection).send(event)
+        assertEquals(expected, event.release)
+    }
+
+    @Test
+    fun `when event doesn't have release, value from options applied`() {
+        val event = SentryEvent()
+        val expected = "original"
+        fixture.sentryOptions.release = expected
+        val sut = fixture.getSut()
+        sut.captureEvent(event)
+        verify(fixture.connection).send(event)
+        assertEquals(expected, event.release)
+    }
+
+    @Test
+    fun `when event has environment, value from options not applied`() {
+        val event = SentryEvent()
+        val expected = "original"
+        fixture.sentryOptions.environment = "not to be applied"
+        event.environment = expected
+        val sut = fixture.getSut()
+        sut.captureEvent(event)
+        verify(fixture.connection).send(event)
+        assertEquals(expected, event.environment)
+    }
+
+    @Test
+    fun `when event doesn't have environment, value from options applied`() {
+        val event = SentryEvent()
+        val expected = "original"
+        fixture.sentryOptions.environment = expected
+        val sut = fixture.getSut()
+        sut.captureEvent(event)
+        assertEquals(expected, event.environment)
+    }
 }