Skip to content

Commit

Permalink
Remove timeout in a test for GCStress modes (dotnet/coreclr#26258)
Browse files Browse the repository at this point in the history
Fixes https://github.com/dotnet/coreclr/issues/25510
- Timeout will be handled by the harness

Commit migrated from dotnet/coreclr@34f545d
  • Loading branch information
kouvel authored Aug 23, 2019
1 parent abd0671 commit ca17ab9
Showing 1 changed file with 2 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,6 @@

public static class MethodImplOptionsTests
{
private const int UnexpectedTimeoutMs = 30_000;

public static int Main()
{
const int Pass = 100;
Expand Down Expand Up @@ -75,23 +73,15 @@ private static bool RunMethodImplOptionsTest(
GetIlasmProcessStartInfos(ilasmFile, ildasmFile, ilFileName, out disasmIlFileName, out ilasmPsi, out ildasmPsi);

Process ilasmProcess = Process.Start(ilasmPsi);
if (!ilasmProcess.WaitForExit(UnexpectedTimeoutMs))
{
Console.WriteLine("Timed out waiting for ilasm to exit");
return false;
}
ilasmProcess.WaitForExit();
if (ilasmProcess.ExitCode != 0)
{
Console.WriteLine($"ilasm failed with exit code: {ilasmProcess.ExitCode}");
return false;
}

Process ildasmProcess = Process.Start(ildasmPsi);
if (!ildasmProcess.WaitForExit(UnexpectedTimeoutMs))
{
Console.WriteLine("Timed out waiting for ildasm to exit");
return false;
}
ildasmProcess.WaitForExit();
if (ildasmProcess.ExitCode != 0)
{
Console.WriteLine($"ildasm failed with exit code: {ildasmProcess.ExitCode}");
Expand Down

0 comments on commit ca17ab9

Please sign in to comment.