Merge pull request #113 from nwinkler/bb-server-context

Bitbucket Server Root Context

Author: derimagia

git-open

@@ -126,10 +126,15 @@ fi
 if [[ "$domain" == 'bitbucket.org' ]]; then
   # Bitbucket, see https://github.com/paulirish/git-open/issues/80 for why ?at is needed.
   providerBranchRef="/src?at=$branch"
-elif [[ ${pathargs[0]} == 'scm' ]]; then
-  # Bitbucket server, which starts with 'scm'
-  # Replace the first element, 'scm', with 'projects'. Keep the first argument, the string 'repos', and finally the rest of the arguments.
-  pathargs=('projects' ${pathargs[1]} 'repos' "${pathargs[@]:2}")
+elif [[ "${#pathargs[@]}" -ge 3 && ${pathargs[${#pathargs[@]} - 3]} == 'scm' ]]; then
+  # Bitbucket server always has /scm/ as the third to last segment in the url path, e.g. /scm/ppp/test-repo.git
+  # Anything before the 'scm' is part of the server's root context
+
+  # If there are other context parts, add them, up to (but not including) the found 'scm'
+  pathPref=("${pathargs[*]:0:${#pathargs[@]} - 3}")
+
+  # Replace the 'scm' element, with 'projects'. Keep the first argument, the string 'repos', and finally the rest of the arguments.
+  pathargs=(${pathPref[@]} 'projects' ${pathargs[${#pathargs[@]} - 2]} 'repos' "${pathargs[@]:${#pathargs[@]} - 1}")
   IFS='/' urlpath="${pathargs[*]}"
   providerBranchRef="/browse?at=$branch"
 elif [[ "${#pathargs[@]}" -ge '2' && ${pathargs[${#pathargs[@]} - 2]} == '_git' ]]; then

test/git-open.bats

@@ -214,22 +214,6 @@ setup() {
   refute_output --partial "@"
 }
 
-@test "bitbucket server" {
-  # https://github.com/paulirish/git-open/pull/15
-  git remote set-url origin "https://[email protected]/scm/ppp/test-repo.git"
-  run ../git-open
-  assert_output "https://bitbucket.example.com/projects/ppp/repos/test-repo"
-}
-
-@test "bitbucket server branch" {
-  # https://github.com/paulirish/git-open/pull/15
-  git remote set-url origin "https://[email protected]/scm/ppp/test-repo.git"
-  git checkout -B "bb-server"
-  run ../git-open
-  assert_output "https://bitbucket.example.com/projects/ppp/repos/test-repo/browse?at=bb-server"
-}
-
-
 @test "bitbucket: Bitbucket Server" {
   # https://github.com/paulirish/git-open/issues/77#issuecomment-309044010
   git remote set-url origin "https://[email protected]/scm/ppp/rrr.git"
@@ -269,6 +253,38 @@ setup() {
 
 }
 
+
+@test "bitbucket: Bitbucket Server with different root context" {
+  # https://github.com/paulirish/git-open/pull/15
+  git remote set-url origin "https://[email protected]/git/scm/ppp/test-repo.git"
+  run ../git-open
+  assert_output "https://bitbucket.example.com/git/projects/ppp/repos/test-repo" ||
+    assert_output "https://bitbucket.example.com/git/projects/ppp/repos/test-repo/?at=master" ||
+    assert_output "https://bitbucket.example.com/git/projects/ppp/repos/test-repo/?at=refs%2Fheads%2Fmaster"
+}
+
+
+@test "bitbucket: Bitbucket Server with different root context with multiple parts" {
+  # https://github.com/paulirish/git-open/pull/15
+  git remote set-url origin "https://[email protected]/really/long/root/context/scm/ppp/test-repo.git"
+  run ../git-open
+  assert_output "https://bitbucket.example.com/really/long/root/context/projects/ppp/repos/test-repo" ||
+    assert_output "https://bitbucket.example.com/really/long/root/context/projects/ppp/repos/test-repo/?at=master" ||
+    assert_output "https://bitbucket.example.com/really/long/root/context/projects/ppp/repos/test-repo/?at=refs%2Fheads%2Fmaster"
+}
+
+
+@test "bitbucket: Bitbucket Server private user repos with different root context" {
+  # https://github.com/paulirish/git-open/pull/83#issuecomment-309968538
+  git remote set-url origin "https://mybb.domain.com/root/context/scm/~first.last/rrr.git"
+  git checkout -B "develop"
+  run ../git-open
+  assert_output "https://mybb.domain.com/root/context/projects/~first.last/repos/rrr/browse?at=develop" ||
+    assert_output "https://mybb.domain.com/root/context/projects/~first.last/repos/rrr/browse?at=refs%2Fheads%2Fdevelop" ||
+    assert_output "https://mybb.domain.com/root/context/projects/~first.last/repos/rrr/browse?at=refs/heads/develop"
+}
+
+
 ##
 ## GitLab
 ##