minor review comments and func tests fix

microsoft · May 3, 2017 · e29dc2a · e29dc2a
1 parent 7e0a386
commit e29dc2a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/Src/DependencyCollector/Shared/DependencyTrackingTelemetryModule.cs b/Src/DependencyCollector/Shared/DependencyTrackingTelemetryModule.cs
@@ -219,7 +219,7 @@ protected virtual void Dispose(bool disposing)
         /// <summary>
         /// Initialize for framework event source (not supported for Net40).
         /// </summary>
-        private void InitializeForFrameworkEventSource()
+        private void InitializeForDiagnosticAndFrameworkEventSource()
         {
 #if NET45
             DesktopDiagnosticSourceHttpProcessing desktopHttpProcessing = new DesktopDiagnosticSourceHttpProcessing(
@@ -267,21 +267,21 @@ private void InitializeForRuntimeInstrumentationOrFramework()
                     }
                     catch (Exception exp)
                     {
-                        this.InitializeForFrameworkEventSource();
+                        this.InitializeForDiagnosticAndFrameworkEventSource();
                         DependencyCollectorEventSource.Log.ProfilerFailedToAttachError(exp.ToInvariantString());
                     }
                 }
                 else
                 {
                     // if config is set to disable runtime instrumentation then default to framework event source
-                    this.InitializeForFrameworkEventSource();
+                    this.InitializeForDiagnosticAndFrameworkEventSource();
                     DependencyCollectorEventSource.Log.RemoteDependencyModuleVerbose("Runtime instrumentation is set to disabled. Initialize with framework event source instead.");
                 }
             }
             else
             {
                 // if profiler is not attached then default to framework event source
-                this.InitializeForFrameworkEventSource();
+                this.InitializeForDiagnosticAndFrameworkEventSource();
 
                 // Log a message to indicate the profiler is not attached
                 DependencyCollectorEventSource.Log.RemoteDependencyModuleProfilerNotAttached();

diff --git a/Test/DependencyCollector/FunctionalTests/FuncTest/AspNetCoreHttpTests.cs b/Test/DependencyCollector/FunctionalTests/FuncTest/AspNetCoreHttpTests.cs
@@ -99,7 +99,7 @@ private static IDisposable DotNetCoreTestSetup()
         {
             EnsureDotNetCoreInstalled();
 
-            return new ExpectedSDKPrefixChanger("rddd");
+            return new ExpectedSDKPrefixChanger("rdddsc");
         }
 
         private const string AspxCoreTestAppFolder = "..\\TestApps\\AspxCore\\";