2016-08-23 20:50:05 +00:00
|
|
|
|
using System;
|
|
|
|
|
using System.Collections.Generic;
|
|
|
|
|
using System.IO;
|
|
|
|
|
using System.Linq;
|
|
|
|
|
using FluentAssertions;
|
|
|
|
|
using Microsoft.Build.Construction;
|
|
|
|
|
using Microsoft.DotNet.ProjectJsonMigration.Rules;
|
2016-10-28 01:46:43 +00:00
|
|
|
|
using Microsoft.DotNet.Internal.ProjectModel;
|
2016-08-23 20:50:05 +00:00
|
|
|
|
using Microsoft.DotNet.Tools.Common;
|
|
|
|
|
using Microsoft.DotNet.Tools.Test.Utilities;
|
|
|
|
|
using NuGet.Frameworks;
|
|
|
|
|
using Xunit;
|
|
|
|
|
|
|
|
|
|
namespace Microsoft.DotNet.ProjectJsonMigration.Tests
|
|
|
|
|
{
|
|
|
|
|
public class GivenAProjectMigrator : TestBase
|
|
|
|
|
{
|
|
|
|
|
[Fact]
|
2016-12-09 03:53:35 +00:00
|
|
|
|
public void ItCopiesProjectDirectoryContentsToOutputDirectoryWhenTheDirectoriesAreDifferent()
|
2016-08-23 20:50:05 +00:00
|
|
|
|
{
|
2016-10-28 01:46:43 +00:00
|
|
|
|
var testProjectDirectory = TestAssetsManager
|
|
|
|
|
.CreateTestInstance("PJTestAppSimple", callingMethod: "z")
|
2016-09-22 04:23:50 +00:00
|
|
|
|
.Path;
|
2016-10-28 01:46:43 +00:00
|
|
|
|
|
2016-08-23 20:50:05 +00:00
|
|
|
|
var outputDirectory = Temp.CreateDirectory().Path;
|
|
|
|
|
|
|
|
|
|
var projectDirectoryRelativeFilePaths = EnumerateFilesWithRelativePath(testProjectDirectory);
|
|
|
|
|
|
|
|
|
|
var mockProj = ProjectRootElement.Create();
|
2016-12-07 21:49:15 +00:00
|
|
|
|
var testSettings = MigrationSettings.CreateMigrationSettingsTestHook(testProjectDirectory, outputDirectory, mockProj);
|
2016-08-23 20:50:05 +00:00
|
|
|
|
|
|
|
|
|
var projectMigrator = new ProjectMigrator(new FakeEmptyMigrationRule());
|
|
|
|
|
projectMigrator.Migrate(testSettings);
|
|
|
|
|
|
|
|
|
|
foreach (var projectDirectoryRelativeFilePath in projectDirectoryRelativeFilePaths)
|
|
|
|
|
{
|
|
|
|
|
File.Exists(Path.Combine(outputDirectory, projectDirectoryRelativeFilePath)).Should().BeTrue();
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
[Fact]
|
2016-12-09 03:53:35 +00:00
|
|
|
|
public void ItHasErrorWhenMigratingADeprecatedProjectJson()
|
2016-08-23 20:50:05 +00:00
|
|
|
|
{
|
|
|
|
|
var testProjectDirectory =
|
|
|
|
|
TestAssetsManager.CreateTestInstance("TestLibraryWithDeprecatedProjectFile", callingMethod: "z")
|
2016-09-22 04:23:50 +00:00
|
|
|
|
.Path;
|
2016-08-23 20:50:05 +00:00
|
|
|
|
|
|
|
|
|
var mockProj = ProjectRootElement.Create();
|
2016-12-07 21:49:15 +00:00
|
|
|
|
var testSettings = MigrationSettings.CreateMigrationSettingsTestHook(testProjectDirectory, testProjectDirectory, mockProj);
|
2016-08-23 20:50:05 +00:00
|
|
|
|
|
|
|
|
|
var projectMigrator = new ProjectMigrator(new FakeEmptyMigrationRule());
|
2016-10-04 21:59:04 +00:00
|
|
|
|
var report = projectMigrator.Migrate(testSettings);
|
2016-08-23 20:50:05 +00:00
|
|
|
|
|
2016-10-04 21:59:04 +00:00
|
|
|
|
var projectReport = report.ProjectMigrationReports.First();
|
|
|
|
|
|
|
|
|
|
var errorMessage = projectReport.Errors.First().GetFormattedErrorMessage();
|
|
|
|
|
errorMessage.Should().Contain("MIGRATE1011::Deprecated Project:");
|
|
|
|
|
errorMessage.Should().Contain("The 'packInclude' option is deprecated. Use 'files' in 'packOptions' instead. (line: 6, file:");
|
|
|
|
|
errorMessage.Should().Contain("The 'compilationOptions' option is deprecated. Use 'buildOptions' instead. (line: 3, file:");
|
2016-08-23 20:50:05 +00:00
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
[Fact]
|
2016-12-09 03:53:35 +00:00
|
|
|
|
public void ItHasErrorWhenMigratingANonCsharpApp()
|
2016-08-23 20:50:05 +00:00
|
|
|
|
{
|
|
|
|
|
var testProjectDirectory =
|
|
|
|
|
TestAssetsManager.CreateTestInstance("FSharpTestProjects/TestApp", callingMethod: "z")
|
2016-09-22 04:23:50 +00:00
|
|
|
|
.Path;
|
2016-08-23 20:50:05 +00:00
|
|
|
|
|
|
|
|
|
var mockProj = ProjectRootElement.Create();
|
2016-12-07 21:49:15 +00:00
|
|
|
|
var testSettings = MigrationSettings.CreateMigrationSettingsTestHook(testProjectDirectory, testProjectDirectory, mockProj);
|
2016-08-23 20:50:05 +00:00
|
|
|
|
|
|
|
|
|
var projectMigrator = new ProjectMigrator(new FakeEmptyMigrationRule());
|
2016-10-04 21:59:04 +00:00
|
|
|
|
var report = projectMigrator.Migrate(testSettings);
|
|
|
|
|
var projectReport = report.ProjectMigrationReports.First();
|
2016-08-23 20:50:05 +00:00
|
|
|
|
|
2016-10-04 21:59:04 +00:00
|
|
|
|
var errorMessage = projectReport.Errors.First().GetFormattedErrorMessage();
|
|
|
|
|
errorMessage.Should().Contain("MIGRATE20013::Non-Csharp App: Cannot migrate project");
|
2016-08-23 20:50:05 +00:00
|
|
|
|
}
|
|
|
|
|
|
2016-12-09 03:53:35 +00:00
|
|
|
|
[Fact]
|
|
|
|
|
public void ItHasErrorWhenMigratingAProjectJsonWithoutAFrameworks()
|
|
|
|
|
{
|
2016-12-09 04:06:55 +00:00
|
|
|
|
var testInstance = TestAssets.Get(
|
|
|
|
|
"NonRestoredTestProjects",
|
|
|
|
|
"TestLibraryWithProjectFileWithoutFrameworks")
|
|
|
|
|
.CreateInstance()
|
2016-12-09 03:53:35 +00:00
|
|
|
|
.WithSourceFiles();
|
|
|
|
|
|
|
|
|
|
var testProjectDirectory = testInstance.Root.FullName;
|
|
|
|
|
|
|
|
|
|
var mockProj = ProjectRootElement.Create();
|
|
|
|
|
var testSettings = MigrationSettings.CreateMigrationSettingsTestHook(
|
|
|
|
|
testProjectDirectory,
|
|
|
|
|
testProjectDirectory,
|
|
|
|
|
mockProj);
|
|
|
|
|
|
|
|
|
|
var projectMigrator = new ProjectMigrator(new FakeEmptyMigrationRule());
|
|
|
|
|
var report = projectMigrator.Migrate(testSettings);
|
|
|
|
|
|
|
|
|
|
var projectReport = report.ProjectMigrationReports.First();
|
|
|
|
|
|
2016-12-09 04:06:55 +00:00
|
|
|
|
projectReport.Errors.First().GetFormattedErrorMessage()
|
|
|
|
|
.Should().Contain("MIGRATE1013::No Project:")
|
|
|
|
|
.And.Contain($"The project.json specifies no target frameworks in {testProjectDirectory}");
|
2016-12-09 03:53:35 +00:00
|
|
|
|
}
|
|
|
|
|
|
2016-08-23 20:50:05 +00:00
|
|
|
|
private IEnumerable<string> EnumerateFilesWithRelativePath(string testProjectDirectory)
|
|
|
|
|
{
|
|
|
|
|
return
|
|
|
|
|
Directory.EnumerateFiles(testProjectDirectory, "*", SearchOption.AllDirectories)
|
|
|
|
|
.Select(file => PathUtility.GetRelativePath(testProjectDirectory, file));
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
private class FakeEmptyMigrationRule : IMigrationRule
|
|
|
|
|
{
|
|
|
|
|
public void Apply(MigrationSettings migrationSettings, MigrationRuleInputs migrationRuleInputs)
|
|
|
|
|
{
|
|
|
|
|
return;
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
}
|