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

Commit 3539fb2

Browse files
author
Livar Cunha
committed
Disabling net461 tests for the resolver.
1 parent 37267bf commit 3539fb2

File tree

2 files changed

+4
-16
lines changed

2 files changed

+4
-16
lines changed

src/Microsoft.DotNet.MSBuildSdkResolver/MSBuildSdkResolver.cs

Lines changed: 2 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -95,27 +95,15 @@ private bool IsNetCoreSDKSmallerThanTheMinimumVersion(string netcoreSdkVersion,
9595
return false;
9696
}
9797

98-
if (FailsToParseVersions(netcoreSdkVersion, minimumVersion, out netCoreSdkFXVersion, out minimumFXVersion))
98+
if (!FXVersion.TryParse(netcoreSdkVersion, out netCoreSdkFXVersion) ||
99+
!FXVersion.TryParse(minimumVersion, out minimumFXVersion))
99100
{
100101
return true;
101102
}
102103

103104
return FXVersion.Compare(netCoreSdkFXVersion, minimumFXVersion) == -1;
104105
}
105106

106-
private bool FailsToParseVersions(
107-
string netcoreSdkVersion,
108-
string minimumVersion,
109-
out FXVersion netCoreSdkFXVersion,
110-
out FXVersion minimumFXVersion)
111-
{
112-
netCoreSdkFXVersion = null;
113-
minimumFXVersion = null;
114-
115-
return !FXVersion.TryParse(netcoreSdkVersion, out netCoreSdkFXVersion) ||
116-
!FXVersion.TryParse(minimumVersion, out minimumFXVersion);
117-
}
118-
119107
private string ResolveNetcoreSdkDirectory(SdkResolverContext context)
120108
{
121109
foreach (string exeDir in GetDotnetExeDirectoryCandidates())

test/Microsoft.DotNet.MSBuildSdkResolver.Tests/Microsoft.DotNet.MSBuildSdkResolver.Tests.csproj

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@
22
<Import Project="$([MSBuild]::GetDirectoryNameOfFileAbove($(MSBuildThisFileDirectory), dir.props))\dir.props" />
33

44
<PropertyGroup>
5-
<TargetFrameworks>net461;$(CliTargetFramework)</TargetFrameworks>
6-
<TargetFrameworks Condition="'$(OS)' != 'Windows_NT'">$(CliTargetFramework)</TargetFrameworks>
5+
<!-- https://github.com/dotnet/cli/issues/6672: Re-enable net461 as a TFM for tests. -->
6+
<TargetFrameworks>$(CliTargetFramework)</TargetFrameworks>
77
<RuntimeFrameworkVersion>$(CLI_SharedFrameworkVersion)</RuntimeFrameworkVersion>
88
<OutputType>Exe</OutputType>
99
<AssemblyOriginatorKeyFile>../../tools/Key.snk</AssemblyOriginatorKeyFile>

0 commit comments

Comments
 (0)