From 25d20e91c44ddc0b22e1c306d354057c2282b4f1 Mon Sep 17 00:00:00 2001 From: Lukas Lansky Date: Tue, 20 Apr 2021 23:11:10 +0200 Subject: [PATCH] Disable iOS ARM validation (#51481) * Disable iOS ARM validation * skip SslStream_ClientCertificate_SendsChain test if chain is not valid (#48261) * skip SslStream_ClientCertificate_SendsChain test if chain is not valid * make test conditional * Disable iOS ARM validation further * Code review feedback: Disable also iOS/tvOS/Android Co-authored-by: Tomas Weinfurt --- eng/pipelines/runtime-official.yml | 23 ++++++++++-------- eng/pipelines/runtime.yml | 24 +++++++++++-------- .../SslStreamNetworkStreamTest.cs | 7 ++++-- 3 files changed, 32 insertions(+), 22 deletions(-) diff --git a/eng/pipelines/runtime-official.yml b/eng/pipelines/runtime-official.yml index 3deef33444c41..dc068e8b37c3a 100644 --- a/eng/pipelines/runtime-official.yml +++ b/eng/pipelines/runtime-official.yml @@ -90,22 +90,25 @@ stages: buildConfig: release runtimeFlavor: mono platforms: - - Android_x64 - - Android_x86 - - Android_arm - - Android_arm64 - - tvOS_x64 - - tvOS_arm64 - - iOS_x64 - - iOS_x86 - - iOS_arm - - iOS_arm64 - OSX_x64 - Linux_x64 - Linux_arm - Linux_arm64 - Linux_musl_x64 - Browser_wasm + + # https://github.com/dotnet/runtime/pull/50940 + # - Android_x64 + # - Android_x86 + # - Android_arm + # - Android_arm64 + # - tvOS_x64 + # - tvOS_arm64 + # - iOS_x64 + # - iOS_x86 + # - iOS_arm + # - iOS_arm64 + # - Linux_musl_arm # - Linux_musl_arm64 # - Windows_NT_x64 enable once coreclr.dll has a version header: https://github.com/dotnet/runtime/issues/37503 diff --git a/eng/pipelines/runtime.yml b/eng/pipelines/runtime.yml index 5a023d9e4e87a..3fac198b603aa 100644 --- a/eng/pipelines/runtime.yml +++ b/eng/pipelines/runtime.yml @@ -263,12 +263,14 @@ jobs: buildConfig: ${{ variables.debugOnPrReleaseOnRolling }} runtimeFlavor: mono platforms: - - Android_x86 - - Android_arm64 - - tvOS_x64 - - iOS_arm64 - - iOS_x86 - Linux_arm + + # https://github.com/dotnet/runtime/pull/50940 + # - Android_x86 + # - Android_arm64 + # - tvOS_x64 + # - iOS_x86 + # - iOS_arm64 jobParameters: testGroup: innerloop nameSuffix: AllSubsets_Mono @@ -286,12 +288,14 @@ jobs: buildConfig: Release runtimeFlavor: mono platforms: - - Android_x64 - - Android_arm - - tvOS_arm64 - - iOS_arm - - iOS_x64 - Linux_musl_x64 + + # https://github.com/dotnet/runtime/pull/50940 + # - Android_x64 + # - Android_arm + # - tvOS_arm64 + # - iOS_x64 + # - iOS_arm jobParameters: testGroup: innerloop nameSuffix: AllSubsets_Mono diff --git a/src/libraries/System.Net.Security/tests/FunctionalTests/SslStreamNetworkStreamTest.cs b/src/libraries/System.Net.Security/tests/FunctionalTests/SslStreamNetworkStreamTest.cs index f9c0f986f473d..404ca4af84cfd 100644 --- a/src/libraries/System.Net.Security/tests/FunctionalTests/SslStreamNetworkStreamTest.cs +++ b/src/libraries/System.Net.Security/tests/FunctionalTests/SslStreamNetworkStreamTest.cs @@ -402,7 +402,7 @@ public async Task SslStream_UntrustedCaWithCustomCallback_Throws(bool customCall } } - [Fact] + [ConditionalFact] [ActiveIssue("https://github.com/dotnet/runtime/issues/46837", TestPlatforms.OSX)] public async Task SslStream_ClientCertificate_SendsChain() { @@ -434,7 +434,10 @@ public async Task SslStream_ClientCertificate_SendsChain() } // Verify we can construct full chain - Assert.True(chain.ChainElements.Count > clientChain.Count, "chain cannot be built"); + if (chain.ChainElements.Count < clientChain.Count) + { + throw new SkipTestException($"chain cannot be built {chain.ChainElements.Count}"); + } } var clientOptions = new SslClientAuthenticationOptions() { TargetHost = "localhost", };