Skip to content

Commit 19392ab

Browse files
author
John Salem
authored
Add RemoteInvokeOptions to EtwManifestGeneration test (#58478)
* increase test timeout * reduce timeout and add local copy to prevent need for marshalling
1 parent 32cbe96 commit 19392ab

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/libraries/System.Diagnostics.Tracing/tests/BasicEventSourceTest/TestsManifestGeneration.Etw.cs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,10 @@ public partial class TestsManifestGeneration
3434
[ConditionalFact(nameof(IsProcessElevatedAndNotWindowsNanoServerAndRemoteExecutorSupported))]
3535
public void Test_EventSource_EtwManifestGeneration()
3636
{
37+
RemoteInvokeOptions options = new RemoteInvokeOptions { TimeOut = 300_000 /* ms */ };
3738
RemoteExecutor.Invoke(() =>
3839
{
40+
RemoteInvokeOptions localOptions = new RemoteInvokeOptions { TimeOut = 300_000 /* ms */ };
3941
using (RemoteInvokeHandle handle = RemoteExecutor.Invoke(() =>
4042
{
4143
var es = new SimpleEventSource();
@@ -44,7 +46,7 @@ public void Test_EventSource_EtwManifestGeneration()
4446
es.WriteSimpleInt(i);
4547
Thread.Sleep(100);
4648
}
47-
}))
49+
}, localOptions))
4850
{
4951
var etlFileName = @"file.etl";
5052
var tracesession = new TraceEventSession("testname", etlFileName);
@@ -72,7 +74,7 @@ public void Test_EventSource_EtwManifestGeneration()
7274
}
7375
Assert.True(manifestExists);
7476
}
75-
}).Dispose();
77+
}, options).Dispose();
7678
}
7779

7880
[ConditionalFact(nameof(IsProcessElevatedAndNotWindowsNanoServerAndRemoteExecutorSupported))]

0 commit comments

Comments
 (0)