Skip to content

Commit

Permalink
Merge pull request #373 from lipkau/fix/#372-RemoveTrailingSlashFromJ…
Browse files Browse the repository at this point in the history
…iraServerUrl

Added check for trailing `/` to be removed from `JiraServerUrl`
  • Loading branch information
lipkau committed Aug 20, 2019
2 parents ce55dcd + acf739e commit d6ecd39
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion JiraPS/Public/Get-JiraConfigServer.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ function Get-JiraConfigServer {
Write-DebugMessage "[$($MyInvocation.MyCommand.Name)] ParameterSetName: $($PsCmdlet.ParameterSetName)"
Write-DebugMessage "[$($MyInvocation.MyCommand.Name)] PSBoundParameters: $($PSBoundParameters | Out-String)"

return $script:JiraServerUrl
return ($script:JiraServerUrl -replace "\/$", "")
}

end {
Expand Down
2 changes: 1 addition & 1 deletion Tests/JiraPS.Tests.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ Describe "General project validation" -Tag Unit {

Import-Module $env:BHManifestToTest -Force

Get-JiraConfigServer | Should -Be "https://example.com/"
Get-JiraConfigServer | Should -Be "https://example.com"
}

# It "module is imported with default prefix" {
Expand Down

0 comments on commit d6ecd39

Please sign in to comment.