Ref: EfCore integration. (#1258)

* Add else if condition on EFCore integration.

Author: lucas-zimerman

src/Sentry.DiagnosticSource/Internals/DiagnosticSource/SentryEFCoreListener.cs

@@ -128,7 +128,7 @@ public void OnNext(KeyValuePair<string, object?> value)
 
                 //Connection Span
                 //A transaction may or may not show a connection with it.
-                if (_logConnectionEnabled && value.Key == EFConnectionOpening)
+                else if (_logConnectionEnabled && value.Key == EFConnectionOpening)
                 {
                     AddSpan(SentryEFSpanType.Connection, "db.connection", null);
                 }

test/Sentry.DiagnosticSource.Tests/SentryEFCoreListenerTests.cs

@@ -10,6 +10,7 @@ namespace Sentry.DiagnosticSource.Tests
 {
     public class SentryEFCoreListenerTests
     {
+        internal const string EFQueryStartCompiling = SentryEFCoreListener.EFQueryStartCompiling;
         internal const string EFQueryCompiling = SentryEFCoreListener.EFQueryCompiling;
         internal const string EFQueryCompiled = SentryEFCoreListener.EFQueryCompiled;
         internal const string EFConnectionOpening = SentryEFCoreListener.EFConnectionOpening;
@@ -173,6 +174,27 @@ public void OnNext_TakeSpanButNotSampled_LogWarningNotInvoked(string key, string
                 Arg.Any<Exception>(),
                 Arg.Any<object[]>());
         }
+        [Theory]
+        [InlineData(EFQueryStartCompiling)]
+        [InlineData(EFQueryCompiling)]
+        [InlineData(EFQueryCompiled)]
+        [InlineData(EFConnectionOpening)]
+        [InlineData(EFConnectionClosed)]
+        [InlineData(EFCommandExecuting)]
+        [InlineData(EFCommandFailed)]
+        [InlineData(EFCommandExecuted)]
+        public void OnNext_ConfigureScopeInvokedOnce(string key)
+        {
+            // Arrange
+            var hub = _fixture.Hub;
+            var interceptor = new SentryEFCoreListener(hub, _fixture.Options);
+
+            // Act
+            interceptor.OnNext(new(key, null));
+
+            // Assert
+            hub.Received(1).ConfigureScope(Arg.Any<Action<Scope>>());
+        }
 
         [Fact]
         public void OnNext_HappyPath_IsValid()