Skip to content

Commit 39d63ed

Browse files
authored
Remove Test Workaround (#2328)
This workaround is for a test failure that sometimes occurs in the CI. Since this is from 2018, let's start by removing it and see if the failure shows up again.
1 parent fccab76 commit 39d63ed

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

src/TraceEvent/TraceEvent.Tests/Parsing/GeneralParsing.cs

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -193,13 +193,6 @@ public void ETW_GeneralParsing_Basic(string etlFileName)
193193
var expectedistogramLine = baselineFile.ReadLine();
194194
lineNum++;
195195

196-
// This is a hack. These seem to have different counts on different machines.
197-
// Need to figure out why, but for now it is tracked by issue https://github.com/Microsoft/perfview/issues/643
198-
if (keyValue.Key.Contains("GC/AllocationTick") || keyValue.Key.Contains("Kernel/DiskIO/Read"))
199-
{
200-
continue;
201-
}
202-
203196
if (etlFileName.Contains("net.4.0") && (keyValue.Key.Contains("HeapStats") || keyValue.Key.Contains("Kernel/PerfInfo/Sample")))
204197
{
205198
continue;

0 commit comments

Comments
 (0)