Skip EventSource tests if process is not elevated (#99261)

davmason · web-flow · commit f84d33c7b831 · 2024-03-04T22:33:10.000-08:00
diff --git a/src/libraries/System.Diagnostics.Tracing/tests/BasicEventSourceTest/Harness/EtwListener.cs b/src/libraries/System.Diagnostics.Tracing/tests/BasicEventSourceTest/Harness/EtwListener.cs
@@ -3,6 +3,7 @@
 
 using Microsoft.Diagnostics.Tracing;
 using Microsoft.Diagnostics.Tracing.Session;
+using Microsoft.DotNet.XUnitExtensions;
 using System;
 using System.Collections.Generic;
 using System.Diagnostics;
@@ -39,7 +40,7 @@ public EtwListener(string dataFileName = "EventSourceTestData.etl", string sessi
             // Today you have to be Admin to turn on ETW events (anyone can write ETW events).
             if (TraceEventSession.IsElevated() != true)
             {
-                throw new Exception("Need to be elevated to run. ");
+                throw new SkipTestException("Need to be elevated to run. ");
             }
 
             if (dataFileName == null)

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`
`4`	`4`	`using Microsoft.Diagnostics.Tracing;`
`5`	`5`	`using Microsoft.Diagnostics.Tracing.Session;`
	`6`	`+using Microsoft.DotNet.XUnitExtensions;`
`6`	`7`	`using System;`
`7`	`8`	`using System.Collections.Generic;`
`8`	`9`	`using System.Diagnostics;`
`@@ -39,7 +40,7 @@ public EtwListener(string dataFileName = "EventSourceTestData.etl", string sessi`
`39`	`40`	`// Today you have to be Admin to turn on ETW events (anyone can write ETW events).`
`40`	`41`	`if (TraceEventSession.IsElevated() != true)`
`41`	`42`	`{`
`42`		`- throw new Exception("Need to be elevated to run. ");`
	`43`	`+ throw new SkipTestException("Need to be elevated to run. ");`
`43`	`44`	`}`
`44`	`45`
`45`	`46`	`if (dataFileName == null)`