Default Environment setting if none provided.

src/Sentry/SentryClient.cs

@@ -146,6 +146,15 @@ private SentryId DoSendEvent(SentryEvent @event, Scope? scope)
                 }
             }
 
+            // Each event should have an 'Environment' set. If none was set, then should default
+            // to 'production'.
+            if (string.IsNullOrWhiteSpace(@event.Environment))
+            {
+                @event.Environment = string.IsNullOrWhiteSpace(_options.Environment)
+                    ? "production"
+                    : _options.Environment;
+            }
+
             SentryEvent? processedEvent = @event;
 
             foreach (var processor in scope.GetAllEventProcessors())

test/Sentry.AspNetCore.Tests/IntegrationMockedBackgroundWorker.cs

@@ -234,6 +234,7 @@ public void Environment_NotOnOptions_ValueFromEnvVar()
                 });
         }
 
+        // NOTE: Options take priority over EnvVar's.
         [Fact]
         public void Environment_BothOnOptionsAndEnvVar_ValueFromOption()
         {

test/Sentry.Tests/SentryClientTests.cs

@@ -289,6 +289,47 @@ public void CaptureEvent_DisposedClient_ThrowsObjectDisposedException()
             _ = Assert.Throws<ObjectDisposedException>(() => sut.CaptureEvent(null));
         }
 
+        [Theory]
+        [InlineData(null, "production")] // Missing: will get default value.
+        [InlineData("", "production")] // Missing: will get default value.
+        [InlineData(" ", "production")] // Missing: will get default value.
+        [InlineData("a", "a")] // Provided: nothing will change.
+        [InlineData("production", "production")] // Provided: nothing will change. (value happens to be the default)
+        [InlineData("aBcDe F !@#$ gHi", "aBcDe F !@#$ gHi")] // Provided: nothing will change. (Case check)
+        public void CaptureEvent_Environment_HasCorrectEnvironmentValue(string environment, string expectedEnvironment)
+        {
+            //_fixture.SentryOptions.Environment = environment;
+
+            var @event = new SentryEvent
+            {
+                Environment = environment
+            };
+
+            var sut = _fixture.GetSut();
+            _ = sut.CaptureEvent(@event);
+
+            Assert.Equal(expectedEnvironment, @event.Environment);
+        }
+
+        [Theory]
+        [InlineData(null, "production")] // Missing: will get default value.
+        [InlineData("", "production")] // Missing: will get default value.
+        [InlineData(" ", "production")] // Missing: will get default value.
+        [InlineData("a", "a")] // Provided: nothing will change.
+        [InlineData("production", "production")] // Provided: nothing will change. (value happens to be the default)
+        [InlineData("aBcDe F !@#$ gHi", "aBcDe F !@#$ gHi")] // Provided: nothing will change. (Case check)
+        public void CaptureEvent_OptionsEnvironment_HasCorrectEnvironmentValue(string environment, string expectedEnvironment)
+        {
+            _fixture.SentryOptions.Environment = environment;
+
+            var @event = new SentryEvent();
+
+            var sut = _fixture.GetSut();
+            _ = sut.CaptureEvent(@event);
+
+            Assert.Equal(expectedEnvironment, @event.Environment);
+        }
+
         [Fact]
         public void Dispose_Worker_DisposeCalled()
         {