Skip to content

Make Resolve-Path -Relative return useful path when $PWD and -Path is on different drive #5740

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
adityapatwardhan merged 11 commits into from
Mar 16, 2018
Merged

Make Resolve-Path -Relative return useful path when $PWD and -Path is on different drive #5740

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
6cc21cb
commands: make rvpa -relative do not return ./absolute_path
kwkam Dec 22, 2017
8ee29ba
[Feature] add test
kwkam Feb 13, 2018
2529663
fixup! [Feature] add test
kwkam Feb 22, 2018
e30fbf5
only return relative path inside current root
kwkam Feb 23, 2018
ca6f977
change test
kwkam Feb 23, 2018
095ccd5
[Feature] update test
kwkam Mar 3, 2018
d7d18d9
[feature] remove drive after test
kwkam Mar 10, 2018
ee4faa0
[feature] use named parameters
kwkam Mar 10, 2018
9d6c156
[feature] fix parameter and test
kwkam Mar 15, 2018
dd76ce8
[Feature] More fixes for test
kwkam Mar 15, 2018
4c5d450
[Feature] Check if Drive is null
kwkam Mar 16, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
[feature] remove drive after test
  • Loading branch information
@kwkam
kwkam committed Mar 10, 2018
commit d7d18d916c2542e8e45a5a8d5accf958a8be21eb
44 changes: 23 additions & 21 deletions test/powershell/Modules/Microsoft.PowerShell.Management/Resolve-Path.Tests.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,23 @@
Describe "Resolve-Path returns proper path" -Tag "CI" {
BeforeAll {
$driveName = "RvpaTest"
$root = Join-Path $TestDrive "fakeroot"
$file = Join-Path $root "file.txt"
$null = New-Item -Path $root -ItemType Directory -Force
$null = New-Item -Path $file -ItemType File -Force
$null = New-PSDrive -Name $driveName -PSProvider FileSystem -Root $root

$testRoot = Join-Path $TestDrive ""
$fakeRoot = Join-Path "$driveName`:" ""

$relCases = @(
@{ wd = $fakeRoot; target = $testRoot; expected = $testRoot }
@{ wd = $testRoot; target = Join-Path $fakeRoot "file.txt"; expected = Join-Path "." "fakeroot" "file.txt" }
)
}
AfterAll {
Remove-PSDrive -Name $driveName -PSProvider FileSystem
Copy link
Copy Markdown
Member

@adityapatwardhan adityapatwardhan Mar 14, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

-PSProvider FileSystem is unnecessary

}
It "Resolve-Path returns resolved paths" {
Resolve-Path $TESTDRIVE | Should be "$TESTDRIVE"
}
Expand All @@ -23,28 +42,11 @@ Describe "Resolve-Path returns proper path" -Tag "CI" {
$result = Resolve-Path -LiteralPath "TestDrive:\\\\\"
($result.Path.TrimEnd('/\')) | Should Be "TestDrive:"
}
It "Resolve-Path -Relative should return correct path on different drive" {
$base = Join-Path $TestDrive "ResolvePath.relative"
$root = Join-Path $base "fakeroot"
$file = Join-Path $root "file.txt"
$expectedFilePath = Join-Path "." "fakeroot" "file.txt"
$driveName = "RvpaTest"
$null = New-Item -Path $base -ItemType Directory -Force
$null = New-Item -Path $root -ItemType Directory -Force
$null = New-Item -Path $file -ItemType File -Force
$null = New-PSDrive -Name $driveName -PSProvider FileSystem -Root $root
$driveRoot = Join-Path "$driveName`:" ""
$driveFile = Join-Path "$driveName`:" "file.txt"
try {
Push-Location -Path $driveRoot
Resolve-Path -Path $base -Relative | Should BeExactly $base
}
finally {
Pop-Location
}
It "Resolve-Path -Relative should return correct path on different drive" -TestCases $relCases {
Copy link
Copy Markdown
Member

@adityapatwardhan adityapatwardhan Mar 14, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a item for test name in $relCases which can be used in the It. Otherwise, all the variations have the same name in the log file.

It "Resolve-Path -Relative should return correct path on different drive - " -TestCases $relCases

param($wd, $target, $expected)
try {
Push-Location -Path $base
Resolve-Path -Path $driveFile -Relative | Should BeExactly $expectedFilePath
Push-Location -Path $wd
Resolve-Path -Path $target -Relative | Should BeExactly $expected
Copy link
Copy Markdown
Collaborator

@iSazonov iSazonov Mar 15, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use new syntax Should -BeExactly.

}
finally {
Pop-Location
Expand Down