diff options
| author | Russ Cox <rsc@golang.org> | 2023-06-01 17:06:44 -0400 |
|---|---|---|
| committer | Russ Cox <rsc@golang.org> | 2023-06-02 02:03:04 +0000 |
| commit | d285cae025f6e6848090ddb748aade98677328db (patch) | |
| tree | 9b7e0e82accdbfc64111bedd76f12a1154bfb184 /git-codereview/api_test.go | |
| parent | 197006278d6bfe1d37a22f19be41ed573d66796d (diff) | |
| download | go-x-review-d285cae025f6e6848090ddb748aade98677328db.tar.xz | |
git-codereview: recognize sso://go/ and rpc://go/v1.3.0
Google engineers are now required to use these access methods
instead of https://go.googlesource.com/, so recognize them as
aliases.
(It is possible for every engineer to configure their Git clients with
insteadOf clauses to hide these from the codereview plugin, but
it's far less error-prone to just handle it here.)
Change-Id: Ic1c9a6b45aa61b11ff25ce21bca6e49344974b04
Reviewed-on: https://go-review.googlesource.com/c/review/+/499923
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Peter Weinberger <pjw@google.com>
Run-TryBot: Russ Cox <rsc@golang.org>
Diffstat (limited to 'git-codereview/api_test.go')
| -rw-r--r-- | git-codereview/api_test.go | 24 |
1 files changed, 20 insertions, 4 deletions
diff --git a/git-codereview/api_test.go b/git-codereview/api_test.go index 8494e90..319815a 100644 --- a/git-codereview/api_test.go +++ b/git-codereview/api_test.go @@ -36,12 +36,12 @@ var authTests = []struct { password: "pw", }, { - cookiefile: "go.googlesource.com TRUE / TRUE 2147483647 o2 git-u2=pw\n", + cookiefile: "go.googlesource.com TRUE / TRUE 2147483647 o2 git-u2=pw\n", cookieName: "o2", cookieValue: "git-u2=pw", }, { - cookiefile: ".googlesource.com TRUE / TRUE 2147483647 o3 git-u3=pw\n", + cookiefile: ".googlesource.com TRUE / TRUE 2147483647 o3 git-u3=pw\n", cookieName: "o3", cookieValue: "git-u3=pw", }, @@ -64,8 +64,8 @@ var authTests = []struct { password: "pw", }, { - netrc: "BOGUS", - cookiefile: "go.googlesource.com TRUE / TRUE 2147483647 o7 git-u7=pw\n", + netrc: "BOGUS", + cookiefile: "go.googlesource.com TRUE / TRUE 2147483647 o7 git-u7=pw\n", cookieName: "o7", cookieValue: "git-u7=pw", }, @@ -146,6 +146,22 @@ func TestLoadGerritOrigin(t *testing.T) { project: "crypto", }, { + // Clone with sso://go/ (Google-internal but common with Go developers) + origin: "", + originUrl: "sso://go/tools", + host: "go.googlesource.com", + url: "https://go-review.googlesource.com", + project: "tools", + }, + { + // Clone with rpc://go/ (Google-internal but common with Go developers) + origin: "", + originUrl: "rpc://go/tools", + host: "go.googlesource.com", + url: "https://go-review.googlesource.com", + project: "tools", + }, + { // Gerrit origin is set. // Gerrit is hosted on a sub-domain. origin: "https://gerrit.mysite.com", |