From bcc04664982f78fe919654f06a348e1ac25e364a Mon Sep 17 00:00:00 2001 From: TheBlob42 Date: Sun, 10 Apr 2022 15:26:31 +0200 Subject: [PATCH] feat: pull request support for bitbucket server --- pkg/commands/hosting_service/definitions.go | 13 +++++ .../hosting_service/hosting_service_test.go | 56 ++++++++++++++++++- 2 files changed, 68 insertions(+), 1 deletion(-) diff --git a/pkg/commands/hosting_service/definitions.go b/pkg/commands/hosting_service/definitions.go index 3e7f144da..04d496aa8 100644 --- a/pkg/commands/hosting_service/definitions.go +++ b/pkg/commands/hosting_service/definitions.go @@ -49,11 +49,24 @@ var azdoServiceDef = ServiceDefinition{ repoURLTemplate: "https://{{.webDomain}}/{{.org}}/{{.project}}/_git/{{.repo}}", } +var bitbucketServerServiceDef = ServiceDefinition{ + provider: "bitbucketServer", + pullRequestURLIntoDefaultBranch: "/pull-requests?create&sourceBranch={{.From}}", + pullRequestURLIntoTargetBranch: "/pull-requests?create&targetBranch={{.To}}&sourceBranch={{.From}}", + commitURL: "/commits/{{.CommitSha}}", + regexStrings: []string{ + `^ssh://git@.*/(?P.*)/(?P.*?)(?:\.git)?$`, + `^https://.*/scm/(?P.*)/(?P.*?)(?:\.git)?$`, + }, + repoURLTemplate: "https://{{.webDomain}}/projects/{{.project}}/repos/{{.repo}}", +} + var serviceDefinitions = []ServiceDefinition{ githubServiceDef, bitbucketServiceDef, gitLabServiceDef, azdoServiceDef, + bitbucketServerServiceDef, } var defaultServiceDomains = []ServiceDomain{ diff --git a/pkg/commands/hosting_service/hosting_service_test.go b/pkg/commands/hosting_service/hosting_service_test.go index df326c4ba..0c926117d 100644 --- a/pkg/commands/hosting_service/hosting_service_test.go +++ b/pkg/commands/hosting_service/hosting_service_test.go @@ -153,6 +153,60 @@ func TestGetPullRequestURL(t *testing.T) { assert.Equal(t, "https://dev.azure.com/myorg/myproject/_git/myrepo/pullrequestcreate?sourceRef=feature%2Fnew&targetRef=dev", url) }, }, + { + testName: "Opens a link to new pull request on Bitbucket Server (SSH)", + from: "feature/new", + remoteUrl: "ssh://git@mycompany.bitbucket.com/myproject/myrepo.git", + configServiceDomains: map[string]string{ + // valid configuration for a bitbucket server URL + "mycompany.bitbucket.com": "bitbucketServer:mycompany.bitbucket.com", + }, + test: func(url string, err error) { + assert.NoError(t, err) + assert.Equal(t, "https://mycompany.bitbucket.com/projects/myproject/repos/myrepo/pull-requests?create&sourceBranch=feature%2Fnew", url) + }, + }, + { + testName: "Opens a link to new pull request on Bitbucket Server (SSH) with specific target", + from: "feature/new", + to: "dev", + remoteUrl: "ssh://git@mycompany.bitbucket.com/myproject/myrepo.git", + configServiceDomains: map[string]string{ + // valid configuration for a bitbucket server URL + "mycompany.bitbucket.com": "bitbucketServer:mycompany.bitbucket.com", + }, + test: func(url string, err error) { + assert.NoError(t, err) + assert.Equal(t, "https://mycompany.bitbucket.com/projects/myproject/repos/myrepo/pull-requests?create&targetBranch=dev&sourceBranch=feature%2Fnew", url) + }, + }, + { + testName: "Opens a link to new pull request on Bitbucket Server (HTTP)", + from: "feature/new", + remoteUrl: "https://mycompany.bitbucket.com/scm/myproject/myrepo.git", + configServiceDomains: map[string]string{ + // valid configuration for a bitbucket server URL + "mycompany.bitbucket.com": "bitbucketServer:mycompany.bitbucket.com", + }, + test: func(url string, err error) { + assert.NoError(t, err) + assert.Equal(t, "https://mycompany.bitbucket.com/projects/myproject/repos/myrepo/pull-requests?create&sourceBranch=feature%2Fnew", url) + }, + }, + { + testName: "Opens a link to new pull request on Bitbucket Server (HTTP) with specific target", + from: "feature/new", + to: "dev", + remoteUrl: "https://mycompany.bitbucket.com/scm/myproject/myrepo.git", + configServiceDomains: map[string]string{ + // valid configuration for a bitbucket server URL + "mycompany.bitbucket.com": "bitbucketServer:mycompany.bitbucket.com", + }, + test: func(url string, err error) { + assert.NoError(t, err) + assert.Equal(t, "https://mycompany.bitbucket.com/projects/myproject/repos/myrepo/pull-requests?create&targetBranch=dev&sourceBranch=feature%2Fnew", url) + }, + }, { testName: "Throws an error if git service is unsupported", from: "feature/divide-operation", @@ -199,7 +253,7 @@ func TestGetPullRequestURL(t *testing.T) { assert.NoError(t, err) assert.Equal(t, "https://bitbucket.org/johndoe/social_network/pull-requests/new?source=feature%2Fprofile-page&t=1", url) }, - expectedLoggedErrors: []string{"Unknown git service type: 'noservice'. Expected one of github, bitbucket, gitlab, azuredevops"}, + expectedLoggedErrors: []string{"Unknown git service type: 'noservice'. Expected one of github, bitbucket, gitlab, azuredevops, bitbucketServer"}, }, { testName: "Escapes reserved URL characters in from branch name",