2016-09-27 04:40:11 +00:00
|
|
|
// Copyright (c) .NET Foundation and contributors. All rights reserved.
|
|
|
|
// Licensed under the MIT license. See LICENSE file in the project root for full license information.
|
|
|
|
|
|
|
|
using Microsoft.Build.Construction;
|
|
|
|
using Microsoft.DotNet.Tools.Test.Utilities;
|
|
|
|
using System.Linq;
|
|
|
|
using Xunit;
|
|
|
|
using FluentAssertions;
|
2016-10-10 21:24:49 +00:00
|
|
|
using Microsoft.DotNet.ProjectJsonMigration;
|
2016-09-27 04:40:11 +00:00
|
|
|
using Microsoft.DotNet.ProjectJsonMigration.Rules;
|
|
|
|
using System;
|
|
|
|
|
|
|
|
namespace Microsoft.DotNet.ProjectJsonMigration.Tests
|
|
|
|
{
|
|
|
|
public class GivenThatIWantToMigratePackageDependencies : TestBase
|
|
|
|
{
|
|
|
|
[Fact]
|
|
|
|
public void It_migrates_basic_PackageReference()
|
|
|
|
{
|
|
|
|
var mockProj = RunPackageDependenciesRuleOnPj(@"
|
|
|
|
{
|
|
|
|
""dependencies"": {
|
|
|
|
""APackage"" : ""1.0.0-preview"",
|
|
|
|
""BPackage"" : ""1.0.0""
|
|
|
|
}
|
|
|
|
}");
|
|
|
|
|
|
|
|
EmitsPackageReferences(mockProj, Tuple.Create("APackage", "1.0.0-preview", ""), Tuple.Create("BPackage", "1.0.0", ""));
|
|
|
|
}
|
|
|
|
|
2016-10-10 21:24:49 +00:00
|
|
|
[Fact]
|
|
|
|
public void It_migrates_type_build_to_PrivateAssets()
|
|
|
|
{
|
|
|
|
var mockProj = RunPackageDependenciesRuleOnPj(@"
|
|
|
|
{
|
|
|
|
""dependencies"": {
|
|
|
|
""APackage"" : {
|
|
|
|
""version"": ""1.0.0-preview"",
|
|
|
|
""type"": ""build""
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}");
|
|
|
|
|
2016-10-10 23:19:57 +00:00
|
|
|
|
2016-10-10 21:24:49 +00:00
|
|
|
var packageRef = mockProj.Items.First(i => i.Include == "APackage" && i.ItemType == "PackageReference");
|
|
|
|
|
|
|
|
var privateAssetsMetadata = packageRef.GetMetadataWithName("PrivateAssets");
|
2016-10-10 23:19:57 +00:00
|
|
|
privateAssetsMetadata.Value.Should().NotBeNull();
|
2016-10-10 21:24:49 +00:00
|
|
|
privateAssetsMetadata.Value.Should().Be("All");
|
|
|
|
}
|
|
|
|
|
2016-10-28 20:39:07 +00:00
|
|
|
[Fact]
|
|
|
|
public void It_migrates_web_projects_to_have_web_sdk_PrivateAssets()
|
|
|
|
{
|
|
|
|
var mockProj = RunPackageDependenciesRuleOnPj(@"
|
|
|
|
{
|
2016-10-28 21:10:34 +00:00
|
|
|
""buildOptions"": {
|
|
|
|
""emitEntryPoint"": true
|
|
|
|
},
|
2016-10-28 20:39:07 +00:00
|
|
|
""dependencies"": {
|
|
|
|
""Microsoft.AspNetCore.Mvc"" : {
|
|
|
|
""version"": ""1.0.0""
|
|
|
|
}
|
2016-10-29 05:11:13 +00:00
|
|
|
},
|
|
|
|
""frameworks"": {
|
|
|
|
""netcoreapp1.0"": {}
|
2016-10-28 20:39:07 +00:00
|
|
|
}
|
|
|
|
}");
|
|
|
|
|
2016-10-29 05:11:13 +00:00
|
|
|
var packageRef = mockProj.Items.FirstOrDefault(i =>
|
|
|
|
i.Include == "Microsoft.NET.Sdk.Web" && i.ItemType == "PackageReference");
|
2016-10-28 20:39:07 +00:00
|
|
|
|
2016-10-29 05:11:13 +00:00
|
|
|
packageRef.Should().NotBeNull();
|
|
|
|
packageRef.GetMetadataWithName("PrivateAssets").Value.Should().NotBeNull().And.Be("All");
|
2016-10-28 20:39:07 +00:00
|
|
|
}
|
|
|
|
|
2016-10-10 21:24:49 +00:00
|
|
|
[Fact]
|
2016-10-10 23:19:57 +00:00
|
|
|
public void It_migrates_suppress_parent_array_to_PrivateAssets()
|
2016-10-10 21:24:49 +00:00
|
|
|
{
|
|
|
|
var mockProj = RunPackageDependenciesRuleOnPj(@"
|
|
|
|
{
|
|
|
|
""dependencies"": {
|
|
|
|
""APackage"" : {
|
|
|
|
""version"": ""1.0.0-preview"",
|
|
|
|
""suppressParent"":[ ""runtime"", ""native"" ]
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}");
|
2016-10-10 23:19:57 +00:00
|
|
|
var packageRef = mockProj.Items.First(i => i.Include == "APackage" && i.ItemType == "PackageReference");
|
|
|
|
|
|
|
|
var privateAssetsMetadata = packageRef.GetMetadataWithName("PrivateAssets");
|
|
|
|
privateAssetsMetadata.Value.Should().NotBeNull();
|
|
|
|
privateAssetsMetadata.Value.Should().Be("Native;Runtime");
|
|
|
|
}
|
2016-10-10 21:24:49 +00:00
|
|
|
|
2016-10-10 23:19:57 +00:00
|
|
|
[Fact]
|
|
|
|
public void It_migrates_suppress_parent_string_to_PrivateAssets()
|
|
|
|
{
|
|
|
|
var mockProj = RunPackageDependenciesRuleOnPj(@"
|
|
|
|
{
|
|
|
|
""dependencies"": {
|
|
|
|
""APackage"" : {
|
|
|
|
""version"": ""1.0.0-preview"",
|
|
|
|
""suppressParent"":""runtime""
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}");
|
2016-10-10 21:24:49 +00:00
|
|
|
var packageRef = mockProj.Items.First(i => i.Include == "APackage" && i.ItemType == "PackageReference");
|
|
|
|
|
|
|
|
var privateAssetsMetadata = packageRef.GetMetadataWithName("PrivateAssets");
|
2016-10-10 23:19:57 +00:00
|
|
|
privateAssetsMetadata.Value.Should().NotBeNull();
|
|
|
|
privateAssetsMetadata.Value.Should().Be("Runtime");
|
2016-10-10 21:24:49 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
[Fact]
|
2016-10-10 23:19:57 +00:00
|
|
|
public void It_migrates_include_exclude_arrays_to_IncludeAssets()
|
2016-10-10 21:24:49 +00:00
|
|
|
{
|
|
|
|
var mockProj = RunPackageDependenciesRuleOnPj(@"
|
|
|
|
{
|
|
|
|
""dependencies"": {
|
|
|
|
""APackage"" : {
|
|
|
|
""version"": ""1.0.0-preview"",
|
|
|
|
""include"": [ ""compile"", ""runtime"", ""native"" ],
|
|
|
|
""exclude"": [ ""native"" ]
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}");
|
2016-10-10 23:19:57 +00:00
|
|
|
var packageRef = mockProj.Items.First(i => i.Include == "APackage" && i.ItemType == "PackageReference");
|
2016-10-10 21:24:49 +00:00
|
|
|
|
2016-10-10 23:19:57 +00:00
|
|
|
var includeAssetsMetadata = packageRef.GetMetadataWithName("IncludeAssets");
|
|
|
|
includeAssetsMetadata.Value.Should().NotBeNull();
|
|
|
|
includeAssetsMetadata.Value.Should().Be("Compile;Runtime");
|
|
|
|
}
|
|
|
|
|
|
|
|
[Fact]
|
|
|
|
public void It_migrates_include_string_to_IncludeAssets()
|
|
|
|
{
|
|
|
|
var mockProj = RunPackageDependenciesRuleOnPj(@"
|
|
|
|
{
|
|
|
|
""dependencies"": {
|
|
|
|
""APackage"" : {
|
|
|
|
""version"": ""1.0.0-preview"",
|
|
|
|
""include"": ""compile"",
|
|
|
|
""exclude"": ""runtime""
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}");
|
2016-10-10 21:24:49 +00:00
|
|
|
var packageRef = mockProj.Items.First(i => i.Include == "APackage" && i.ItemType == "PackageReference");
|
|
|
|
|
2016-10-10 23:19:57 +00:00
|
|
|
var includeAssetsMetadata = packageRef.GetMetadataWithName("IncludeAssets");
|
|
|
|
includeAssetsMetadata.Value.Should().NotBeNull();
|
|
|
|
includeAssetsMetadata.Value.Should().Be("Compile");
|
2016-10-10 21:24:49 +00:00
|
|
|
}
|
|
|
|
|
2016-10-10 23:19:57 +00:00
|
|
|
[Fact]
|
|
|
|
public void It_migrates_include_exclude_overlapping_strings_to_IncludeAssets()
|
|
|
|
{
|
|
|
|
var mockProj = RunPackageDependenciesRuleOnPj(@"
|
|
|
|
{
|
|
|
|
""dependencies"": {
|
|
|
|
""APackage"" : {
|
|
|
|
""version"": ""1.0.0-preview"",
|
|
|
|
""include"": ""compile"",
|
|
|
|
""exclude"": ""compile"",
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}");
|
|
|
|
var packageRef = mockProj.Items.First(i => i.Include == "APackage" && i.ItemType == "PackageReference");
|
|
|
|
|
|
|
|
var includeAssetsMetadata = packageRef.GetMetadataWithName("IncludeAssets");
|
|
|
|
includeAssetsMetadata.Value.Should().NotBeNull();
|
|
|
|
includeAssetsMetadata.Value.Should().Be("None");
|
|
|
|
}
|
|
|
|
|
|
|
|
|
2016-10-05 18:36:23 +00:00
|
|
|
[Fact]
|
|
|
|
public void It_migrates_Tools()
|
|
|
|
{
|
|
|
|
var mockProj = RunPackageDependenciesRuleOnPj(@"
|
|
|
|
{
|
|
|
|
""tools"": {
|
|
|
|
""APackage"" : ""1.0.0-preview"",
|
|
|
|
""BPackage"" : ""1.0.0""
|
|
|
|
}
|
|
|
|
}");
|
|
|
|
|
|
|
|
EmitsToolReferences(mockProj, Tuple.Create("APackage", "1.0.0-preview"), Tuple.Create("BPackage", "1.0.0"));
|
|
|
|
}
|
|
|
|
|
2016-10-11 22:22:18 +00:00
|
|
|
[Fact]
|
|
|
|
public void It_migrates_imports_per_framework()
|
|
|
|
{
|
|
|
|
var importPropertyName = "PackageTargetFallback";
|
|
|
|
|
|
|
|
var mockProj = RunPackageDependenciesRuleOnPj(@"
|
|
|
|
{
|
|
|
|
""frameworks"": {
|
|
|
|
""netcoreapp1.0"" : {
|
|
|
|
""imports"": [""netstandard1.3"", ""net451""]
|
|
|
|
},
|
|
|
|
""netstandard1.3"" : {
|
|
|
|
""imports"": [""net451""]
|
|
|
|
},
|
|
|
|
""net451"" : {
|
|
|
|
""imports"": ""netstandard1.3""
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}");
|
|
|
|
|
|
|
|
var imports = mockProj.Properties.Where(p => p.Name == importPropertyName);
|
|
|
|
imports.Should().HaveCount(3);
|
|
|
|
|
|
|
|
var netcoreappImport = imports.First(p => p.Condition.Contains("netcoreapp1.0"));
|
|
|
|
var netstandardImport = imports.First(p => p.Condition.Contains("netstandard1.3"));
|
|
|
|
var net451Import = imports.First(p => p.Condition.Contains("net451"));
|
|
|
|
|
|
|
|
netcoreappImport.Should().NotBe(netstandardImport);
|
|
|
|
|
|
|
|
netcoreappImport.Condition.Should().Be(" '$(TargetFramework)' == 'netcoreapp1.0' ");
|
|
|
|
netstandardImport.Condition.Should().Be(" '$(TargetFramework)' == 'netstandard1.3' ");
|
|
|
|
net451Import.Condition.Should().Be(" '$(TargetFramework)' == 'net451' ");
|
|
|
|
|
|
|
|
netcoreappImport.Value.Split(';').Should().BeEquivalentTo($"$({importPropertyName})", "netstandard1.3", "net451");
|
|
|
|
netstandardImport.Value.Split(';').Should().BeEquivalentTo($"$({importPropertyName})", "net451");
|
|
|
|
net451Import.Value.Split(';').Should().BeEquivalentTo($"$({importPropertyName})", "netstandard1.3");
|
|
|
|
}
|
|
|
|
|
2016-10-18 01:02:10 +00:00
|
|
|
[Fact]
|
|
|
|
public void It_auto_add_desktop_references_during_migrate()
|
|
|
|
{
|
|
|
|
var mockProj = RunPackageDependenciesRuleOnPj(@"
|
|
|
|
{
|
|
|
|
""frameworks"": {
|
|
|
|
""net35"" : {},
|
|
|
|
""net4"" : {},
|
|
|
|
""net451"" : {}
|
|
|
|
}
|
|
|
|
}");
|
|
|
|
|
|
|
|
var itemGroup = mockProj.ItemGroups.Where(i => i.Condition == " '$(TargetFramework)' == 'net451' ");
|
|
|
|
itemGroup.Should().HaveCount(1);
|
|
|
|
itemGroup.First().Items.Should().HaveCount(2);
|
|
|
|
var items = itemGroup.First().Items.ToArray();
|
|
|
|
items[0].Include.Should().Be("System");
|
|
|
|
items[1].Include.Should().Be("Microsoft.CSharp");
|
|
|
|
|
|
|
|
itemGroup = mockProj.ItemGroups.Where(i => i.Condition == " '$(TargetFramework)' == 'net40' ");
|
|
|
|
itemGroup.Should().HaveCount(1);
|
|
|
|
itemGroup.First().Items.Should().HaveCount(2);
|
|
|
|
items = itemGroup.First().Items.ToArray();
|
|
|
|
items[0].Include.Should().Be("System");
|
|
|
|
items[1].Include.Should().Be("Microsoft.CSharp");
|
|
|
|
|
|
|
|
itemGroup = mockProj.ItemGroups.Where(i => i.Condition == " '$(TargetFramework)' == 'net35' ");
|
|
|
|
itemGroup.Should().HaveCount(1);
|
|
|
|
itemGroup.First().Items.Should().HaveCount(1);
|
|
|
|
items = itemGroup.First().Items.ToArray();
|
|
|
|
items[0].Include.Should().Be("System");
|
|
|
|
}
|
|
|
|
|
2016-10-31 20:59:47 +00:00
|
|
|
[Fact]
|
|
|
|
public void It_migrates_test_projects_to_have_test_sdk_and_xunit_packagedependencies()
|
|
|
|
{
|
|
|
|
var mockProj = RunPackageDependenciesRuleOnPj(@"
|
|
|
|
{
|
|
|
|
""buildOptions"": {
|
|
|
|
""emitEntryPoint"": true
|
|
|
|
},
|
|
|
|
""frameworks"": {
|
|
|
|
""netcoreapp1.0"": {}
|
|
|
|
},
|
|
|
|
""testRunner"": ""xunit""
|
|
|
|
}");
|
|
|
|
|
|
|
|
var items = mockProj.Items
|
|
|
|
.Where(i => (i.Include == "Microsoft.NET.Test.Sdk" && i.ItemType == "PackageReference") ||
|
|
|
|
(i.Include == "xunit" && i.ItemType == "PackageReference") ||
|
|
|
|
(i.Include == "xunit.runner.visualstudio" && i.ItemType == "PackageReference"));
|
|
|
|
|
|
|
|
items.Should().HaveCount(3);
|
|
|
|
}
|
|
|
|
|
2016-09-27 04:40:11 +00:00
|
|
|
private void EmitsPackageReferences(ProjectRootElement mockProj, params Tuple<string, string, string>[] packageSpecs)
|
|
|
|
{
|
|
|
|
foreach (var packageSpec in packageSpecs)
|
|
|
|
{
|
|
|
|
var packageName = packageSpec.Item1;
|
|
|
|
var packageVersion = packageSpec.Item2;
|
|
|
|
var packageTFM = packageSpec.Item3;
|
|
|
|
|
|
|
|
var items = mockProj.Items
|
|
|
|
.Where(i => i.ItemType == "PackageReference")
|
|
|
|
.Where(i => string.IsNullOrEmpty(packageTFM) || i.ConditionChain().Any(c => c.Contains(packageTFM)))
|
|
|
|
.Where(i => i.Include == packageName)
|
|
|
|
.Where(i => i.GetMetadataWithName("Version").Value == packageVersion);
|
|
|
|
|
|
|
|
items.Should().HaveCount(1);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-10-05 18:36:23 +00:00
|
|
|
private void EmitsToolReferences(ProjectRootElement mockProj, params Tuple<string, string>[] toolSpecs)
|
|
|
|
{
|
|
|
|
foreach (var toolSpec in toolSpecs)
|
|
|
|
{
|
|
|
|
var packageName = toolSpec.Item1;
|
|
|
|
var packageVersion = toolSpec.Item2;
|
|
|
|
|
|
|
|
var items = mockProj.Items
|
2016-10-05 22:27:30 +00:00
|
|
|
.Where(i => i.ItemType == "DotNetCliToolReference")
|
2016-10-05 18:36:23 +00:00
|
|
|
.Where(i => i.Include == packageName)
|
|
|
|
.Where(i => i.GetMetadataWithName("Version").Value == packageVersion);
|
|
|
|
|
|
|
|
items.Should().HaveCount(1);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-09-27 04:40:11 +00:00
|
|
|
private ProjectRootElement RunPackageDependenciesRuleOnPj(string s, string testDirectory = null)
|
|
|
|
{
|
|
|
|
testDirectory = testDirectory ?? Temp.CreateDirectory().Path;
|
|
|
|
return TemporaryProjectFileRuleRunner.RunRules(new IMigrationRule[]
|
|
|
|
{
|
2016-10-05 18:36:23 +00:00
|
|
|
new MigratePackageDependenciesAndToolsRule()
|
2016-09-27 04:40:11 +00:00
|
|
|
}, s, testDirectory);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|