Add ? to _activeConnection references in trace logging - fixes the broken unit test

Author: benrr101

src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/SqlCommand.Reader.cs

@@ -70,7 +70,7 @@ public SqlDataReader EndExecuteReader(IAsyncResult asyncResult)
                     "SqlCommand.EndExecuteReader | API | Correlation | " +
                     $"Object Id {ObjectID}, " +
                     $"Activity Id {ActivityCorrelator.Current}, " +
-                    $"Client Connection Id {_activeConnection.ClientConnectionId}, " +
+                    $"Client Connection Id {_activeConnection?.ClientConnectionId}, " +
                     $"Command Text '{CommandText}'");
             }
         }
@@ -82,7 +82,7 @@ public SqlDataReader EndExecuteReader(IAsyncResult asyncResult)
                 "SqlCommand.ExecuteReader | API | Correlation | " +
                 $"Object Id {ObjectID}, " +
                 $"Activity Id {ActivityCorrelator.Current}, " +
-                $"Client Connection Id {_activeConnection.ClientConnectionId}, " +
+                $"Client Connection Id {_activeConnection?.ClientConnectionId}, " +
                 $"Command Text '{CommandText}'");
 
             SqlStatistics statistics = null;
@@ -1018,7 +1018,7 @@ private Task<SqlDataReader> InternalExecuteReaderAsync(
                 #if NET
                 if (!_parentOperationStarted)
                 {
-                    s_diagnosticListener.WriteCommandBefore(operationId, this, _transaction, e);
+                    s_diagnosticListener.WriteCommandError(operationId, this, _transaction, e);
                 }
                 #endif
 
@@ -1459,7 +1459,7 @@ private SqlDataReader RunExecuteReaderTds(
                             "SqlCommand.RunExecuteReaderTds | Info | " +
                             $"Object Id {ObjectID}, " +
                             $"Activity Id {ActivityCorrelator.Current}, " +
-                            $"Client Connection Id {_activeConnection.ClientConnectionId}, " +
+                            $"Client Connection Id {_activeConnection?.ClientConnectionId}, " +
                             $"Command executed as RPC, " +
                             $"RPC Name '{rpc.rpcName}'");
                     }