19090744cb
On environments where registry access is disabled, the first run experience fails because it could not add the tools path to the user's environment. This fix properly handles the security exception by printing a warning and continuing. Users will have to manually add the PATH environment variable to their environments to prevent `dotnet tool install` from printing PATH instructions. A new file sentinel is added to track whether or not the PATH has been modified. The first run experience also now correctly skips modifying the PATH if `DOTNET_SKIP_FIRST_TIME_EXPERIENCE` is set. Fixes #8874.
153 lines
5.3 KiB
C#
153 lines
5.3 KiB
C#
// 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 System;
|
|
using FluentAssertions;
|
|
using Microsoft.DotNet.Cli.Utils;
|
|
using Microsoft.DotNet.Configurer;
|
|
using Microsoft.DotNet.Tools;
|
|
using Microsoft.DotNet.Tools.Test.Utilities;
|
|
using Microsoft.Extensions.DependencyModel.Tests;
|
|
using Moq;
|
|
using Xunit;
|
|
|
|
namespace Microsoft.DotNet.ShellShim.Tests
|
|
{
|
|
public class OsxEnvironmentPathTests
|
|
{
|
|
[Fact]
|
|
public void GivenPathNotSetItPrintsManualInstructions()
|
|
{
|
|
var reporter = new BufferedReporter();
|
|
var toolsPath = new BashPathUnderHomeDirectory("/home/user", ".dotnet/tools");
|
|
var pathValue = @"/usr/bin";
|
|
var provider = new Mock<IEnvironmentProvider>(MockBehavior.Strict);
|
|
|
|
provider
|
|
.Setup(p => p.GetEnvironmentVariable("PATH"))
|
|
.Returns(pathValue);
|
|
|
|
var environmentPath = new OSXEnvironmentPath(
|
|
toolsPath,
|
|
reporter,
|
|
provider.Object,
|
|
FileSystemMockBuilder.Empty.File);
|
|
|
|
environmentPath.PrintAddPathInstructionIfPathDoesNotExist();
|
|
|
|
reporter.Lines.Should().Equal(
|
|
string.Format(
|
|
CommonLocalizableStrings.EnvironmentPathOSXManualInstructions,
|
|
toolsPath.Path));
|
|
}
|
|
|
|
[Fact]
|
|
public void GivenPathNotSetAndProfileExistsItPrintsReopenMessage()
|
|
{
|
|
var reporter = new BufferedReporter();
|
|
var toolsPath = new BashPathUnderHomeDirectory("/home/user", ".dotnet/tools");
|
|
var pathValue = @"/usr/bin";
|
|
var provider = new Mock<IEnvironmentProvider>(MockBehavior.Strict);
|
|
|
|
provider
|
|
.Setup(p => p.GetEnvironmentVariable("PATH"))
|
|
.Returns(pathValue);
|
|
|
|
var environmentPath = new OSXEnvironmentPath(
|
|
toolsPath,
|
|
reporter,
|
|
provider.Object,
|
|
new FileSystemMockBuilder()
|
|
.AddFile(OSXEnvironmentPath.DotnetCliToolsPathsDPath, "")
|
|
.Build()
|
|
.File);
|
|
|
|
environmentPath.PrintAddPathInstructionIfPathDoesNotExist();
|
|
|
|
reporter.Lines.Should().Equal(CommonLocalizableStrings.EnvironmentPathOSXNeedReopen);
|
|
}
|
|
|
|
[Theory]
|
|
[InlineData("/home/user/.dotnet/tools")]
|
|
[InlineData("~/.dotnet/tools")]
|
|
public void GivenPathSetItPrintsNothing(string toolsDiretoryOnPath)
|
|
{
|
|
var reporter = new BufferedReporter();
|
|
var toolsPath = new BashPathUnderHomeDirectory("/home/user", ".dotnet/tools");
|
|
var pathValue = @"/usr/bin";
|
|
var provider = new Mock<IEnvironmentProvider>(MockBehavior.Strict);
|
|
|
|
provider
|
|
.Setup(p => p.GetEnvironmentVariable("PATH"))
|
|
.Returns(pathValue + ":" + toolsDiretoryOnPath);
|
|
|
|
var environmentPath = new OSXEnvironmentPath(
|
|
toolsPath,
|
|
reporter,
|
|
provider.Object,
|
|
FileSystemMockBuilder.Empty.File);
|
|
|
|
environmentPath.PrintAddPathInstructionIfPathDoesNotExist();
|
|
|
|
reporter.Lines.Should().BeEmpty();
|
|
}
|
|
|
|
[Fact]
|
|
public void GivenPathSetItDoesNotAddPathToEnvironment()
|
|
{
|
|
var reporter = new BufferedReporter();
|
|
var toolsPath = new BashPathUnderHomeDirectory("/home/user", ".dotnet/tools");
|
|
var pathValue = @"/usr/bin";
|
|
var provider = new Mock<IEnvironmentProvider>(MockBehavior.Strict);
|
|
var fileSystem = new FileSystemMockBuilder().Build().File;
|
|
|
|
provider
|
|
.Setup(p => p.GetEnvironmentVariable("PATH"))
|
|
.Returns(pathValue + ":" + toolsPath.Path);
|
|
|
|
var environmentPath = new OSXEnvironmentPath(
|
|
toolsPath,
|
|
reporter,
|
|
provider.Object,
|
|
fileSystem);
|
|
|
|
environmentPath.AddPackageExecutablePathToUserPath();
|
|
|
|
reporter.Lines.Should().BeEmpty();
|
|
|
|
fileSystem
|
|
.Exists(OSXEnvironmentPath.DotnetCliToolsPathsDPath)
|
|
.Should()
|
|
.Be(false);
|
|
}
|
|
|
|
[Fact]
|
|
public void GivenPathNotSetItAddsToEnvironment()
|
|
{
|
|
var reporter = new BufferedReporter();
|
|
var toolsPath = new BashPathUnderHomeDirectory("/home/user", ".dotnet/tools");
|
|
var pathValue = @"/usr/bin";
|
|
var provider = new Mock<IEnvironmentProvider>(MockBehavior.Strict);
|
|
var fileSystem = new FileSystemMockBuilder().Build().File;
|
|
|
|
provider
|
|
.Setup(p => p.GetEnvironmentVariable("PATH"))
|
|
.Returns(pathValue);
|
|
|
|
var environmentPath = new OSXEnvironmentPath(
|
|
toolsPath,
|
|
reporter,
|
|
provider.Object,
|
|
fileSystem);
|
|
|
|
environmentPath.AddPackageExecutablePathToUserPath();
|
|
|
|
reporter.Lines.Should().BeEmpty();
|
|
|
|
fileSystem
|
|
.ReadAllText(OSXEnvironmentPath.DotnetCliToolsPathsDPath)
|
|
.Should()
|
|
.Be(toolsPath.PathWithTilde);
|
|
}
|
|
}
|
|
}
|