Update condition

Require administrator if on Windows, rather than require administrator and Windows.

Author: martincostello

test/IntegrationTests/WcfTestsBase.cs

@@ -62,7 +62,6 @@ protected virtual void Dispose(bool disposing)
 
     protected async Task SubmitsTracesInternal(string clientPackageVersion, WcfServerTestHelperBase wcfServerTestHelperBase)
     {
-        Assert.True(!EnvironmentTools.IsWindows() || EnvironmentTools.IsWindowsAdministrator(), "This test requires Windows Administrator privileges.");
 #if NET
         Assert.NotNull(wcfServerTestHelperBase);
 #else

test/IntegrationTests/WindowsAdministratorFactAttribute.cs

@@ -11,6 +11,9 @@ public sealed class WindowsAdministratorFactAttribute : FactAttribute
     public WindowsAdministratorFactAttribute()
         : base()
     {
-        Skip = EnvironmentTools.IsWindowsAdministrator() ? null : "This test requires Windows Administrator privileges.";
+        Skip = GetSkipReason();
     }
+
+    internal static string? GetSkipReason() =>
+        !EnvironmentTools.IsWindows() || EnvironmentTools.IsWindowsAdministrator() ? null : "This test requires administrative privileges on Windows.";
 }

test/IntegrationTests/WindowsAdministratorTheoryAttribute.cs

@@ -1,8 +1,6 @@
 // Copyright The OpenTelemetry Authors
 // SPDX-License-Identifier: Apache-2.0
 
-using IntegrationTests.Helpers;
-
 namespace IntegrationTests;
 
 [AttributeUsage(AttributeTargets.Method, AllowMultiple = false)]
@@ -11,6 +9,6 @@ public sealed class WindowsAdministratorTheoryAttribute : TheoryAttribute
     public WindowsAdministratorTheoryAttribute()
         : base()
     {
-        Skip = EnvironmentTools.IsWindowsAdministrator() ? null : "This test requires Windows Administrator privileges.";
+        Skip = WindowsAdministratorFactAttribute.GetSkipReason();
     }
 }