Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Disable SslStreamAlertTests by making them netcoreapp1.1 specific#12733

Merged
weshaggard merged 1 commit into
dotnet:masterfrom
weshaggard:FixSslStreamAlertsTests
Oct 17, 2016
Merged

Disable SslStreamAlertTests by making them netcoreapp1.1 specific#12733
weshaggard merged 1 commit into
dotnet:masterfrom
weshaggard:FixSslStreamAlertsTests

Conversation

@weshaggard
Copy link
Copy Markdown
Member

After #11489

This error shows up after trying to consume the new packages.
SslStreamAlertsTest.cs(81,30): error CS1061: 'SslStream' does not contain a definition for 'ShutdownAsync' and no extension method 'ShutdownAsync' accepting a first argument of type 'SslStream' could be found (are you missing a using directive or an assembly reference?) [D:\A_work\55\s\corefx\src\System.Net.Security\tests\FunctionalTests\System.Net.Security.Tests.csproj]

For now disabling these tests by conditioning them on netcoreapp1.1.

@CIPop you will need to fix the issue with your PR to create a netcoreapp1.1 build configuration for the ref and these tests in order to re-enable them.

FYI @sepidehms @joperezr

After dotnet#11489

This error shows up after trying to consume the new packages.
 SslStreamAlertsTest.cs(81,30): error CS1061: 'SslStream' does not contain a definition for 'ShutdownAsync' and no extension method 'ShutdownAsync' accepting a first argument of type 'SslStream' could be found (are you missing a using directive or an assembly reference?) [D:\A\_work\55\s\corefx\src\System.Net.Security\tests\FunctionalTests\System.Net.Security.Tests.csproj]

For now disabling these tests by conditioning them on netcoreapp1.1.
Copy link
Copy Markdown
Member

@joperezr joperezr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Can we also log an issue for this to get tracked?

@weshaggard
Copy link
Copy Markdown
Member Author

@weshaggard weshaggard merged commit bb24814 into dotnet:master Oct 17, 2016
@weshaggard weshaggard deleted the FixSslStreamAlertsTests branch November 9, 2016 17:48
@karelz karelz modified the milestone: 1.2.0 Dec 3, 2016
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
…ertsTests

Disable SslStreamAlertTests by making them netcoreapp1.1 specific

Commit migrated from dotnet/corefx@bb24814
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants