diff options
| author | Eirik Tsarpalis <eirik.tsarpalis@gmail.com> | 2020-02-27 20:32:39 +0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-02-27 20:32:39 +0300 |
| commit | 4f84429fb44801b841c052505935a3cd14da18e2 (patch) | |
| tree | f253fdfbc5ed4168e610f1c76a9c1a1e09c00e56 /src/libraries/System.Net.Security/tests/UnitTests | |
| parent | d76ef042f8ead9d06a447ab2b1004ae626185ca2 (diff) | |
Annotate System.Net.Security for nullable reference types (#32541)
* annotate System.Net.Security for nullable reference types
* address task nullability issue
* add nullable directives to common files
* address feedback
* add missing nullable directive
* make ProcessAuthentication method explicitly nullable
* fix build after rebase
* add missing nullable directives to common source files
* fix build
* fix build
* make APM method annotations consistent
* address feedback
* address feedback
* address feedback
* address feedback
* address feedback
* fix rebase build issues
Diffstat (limited to 'src/libraries/System.Net.Security/tests/UnitTests')
| -rw-r--r-- | src/libraries/System.Net.Security/tests/UnitTests/Fakes/FakeSslStream.Implementation.cs | 2 | ||||
| -rw-r--r-- | src/libraries/System.Net.Security/tests/UnitTests/System.Net.Security.Unit.Tests.csproj | 1 |
2 files changed, 2 insertions, 1 deletions
diff --git a/src/libraries/System.Net.Security/tests/UnitTests/Fakes/FakeSslStream.Implementation.cs b/src/libraries/System.Net.Security/tests/UnitTests/Fakes/FakeSslStream.Implementation.cs index 85eb337a3e1..3dffb86a4da 100644 --- a/src/libraries/System.Net.Security/tests/UnitTests/Fakes/FakeSslStream.Implementation.cs +++ b/src/libraries/System.Net.Security/tests/UnitTests/Fakes/FakeSslStream.Implementation.cs @@ -11,7 +11,7 @@ namespace System.Net.Security { public partial class SslStream { - private void ValidateCreateContext(SslClientAuthenticationOptions sslClientAuthenticationOptions, RemoteCertValidationCallback remoteCallback, LocalCertSelectionCallback localCallback) + private void ValidateCreateContext(SslClientAuthenticationOptions sslClientAuthenticationOptions, RemoteCertValidationCallback remoteCallback, LocalCertSelectionCallback? localCallback) { // Without setting (or using) these members you will get a build exception in the unit test project. // The code that normally uses these in the main solution is in the implementation of SslStream. diff --git a/src/libraries/System.Net.Security/tests/UnitTests/System.Net.Security.Unit.Tests.csproj b/src/libraries/System.Net.Security/tests/UnitTests/System.Net.Security.Unit.Tests.csproj index be0e5019e21..036fba9049b 100644 --- a/src/libraries/System.Net.Security/tests/UnitTests/System.Net.Security.Unit.Tests.csproj +++ b/src/libraries/System.Net.Security/tests/UnitTests/System.Net.Security.Unit.Tests.csproj @@ -11,6 +11,7 @@ <!-- Disable: CLSCompliant attribute is not needed --> <NoWarn>$(NoWarn);3021</NoWarn> <TargetFrameworks>$(NetCoreAppCurrent)-Windows_NT;$(NetCoreAppCurrent)-Unix;$(NetCoreAppCurrent)-OSX</TargetFrameworks> + <Nullable>annotations</Nullable> </PropertyGroup> <ItemGroup> <Compile Include="SslApplicationProtocolTests.cs" /> |