Skip to content

Commit

Permalink
Applied comments
Browse files Browse the repository at this point in the history
  • Loading branch information
e-n-0 committed May 6, 2024
1 parent 6927c7a commit aecb079
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1051,7 +1051,6 @@ public virtual async Task TryStartApp(AspNetCoreTestFixture fixture)
EnableIast(IastEnabled);
EnableEvidenceRedaction(RedactionEnabled);
EnableIastTelemetry(IastTelemetryLevel);
SetEnvironmentVariable(ConfigurationKeys.DebugEnabled, "1");
DisableObfuscationQueryString();
SetEnvironmentVariable(ConfigurationKeys.Iast.IsIastDeduplicationEnabled, IsIastDeduplicationEnabled?.ToString() ?? string.Empty);
SetEnvironmentVariable(ConfigurationKeys.Iast.VulnerabilitiesPerRequest, VulnerabilitiesPerRequest?.ToString() ?? string.Empty);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ public GrpcDotNetTests(ITestOutputHelper output)
public async Task SubmitsTraces()
{
GuardAlpine();
GuardLinux();
SkipOn.Platform(SkipOn.PlatformValue.Linux);

const int expectedSpanCount = 24;
const string filename = "Iast.GrpcDotNetTests.BodyPropagation.SubmitsTraces";
Expand Down Expand Up @@ -70,13 +70,5 @@ private static void GuardAlpine()
throw new SkipException("GRPC.Tools does not support Alpine");
}
}

private static void GuardLinux()
{
if (RuntimeInformation.IsOSPlatform(System.Runtime.InteropServices.OSPlatform.Linux))
{
throw new SkipException("Can't run https tests on Linux");
}
}
}
#endif

0 comments on commit aecb079

Please sign in to comment.