Skip to content

Commit 446c880

Browse files
authored
Actually include child processes when requested during a memory dump in tests (#6520)
## Summary of changes Fix "dump child processes" not working ## Reason for change #6401 added the ability to say "dump the child processes too", except we didn't wire up the parameter. ## Implementation details Wire up the parameter
1 parent abb8103 commit 446c880

File tree

1 file changed

+1
-1
lines changed
  • tracer/test/Datadog.Trace.TestHelpers.AutoInstrumentation

1 file changed

+1
-1
lines changed

tracer/test/Datadog.Trace.TestHelpers.AutoInstrumentation/TestHelper.cs

+1-1
Original file line numberDiff line numberDiff line change
@@ -191,7 +191,7 @@ public ProcessResult WaitForProcessResult(ProcessHelper helper, int expectedExit
191191

192192
if (!ranToCompletion && !process.HasExited)
193193
{
194-
var tookMemoryDump = MemoryDumpHelper.CaptureMemoryDump(process);
194+
var tookMemoryDump = MemoryDumpHelper.CaptureMemoryDump(process, includeChildProcesses: dumpChildProcesses);
195195
process.Kill();
196196
throw new Exception($"The sample did not exit in {timeoutMs}ms. Memory dump taken: {tookMemoryDump}. Killing process.");
197197
}

0 commit comments

Comments
 (0)