mirror of
https://github.com/hanzoai/deploy.git
synced 2026-10-07 08:44:50 +00:00
41536738c8
* feat(api): hard-fork CRD API group argoproj.io -> deploy.hanzo.ai
Hanzo CD hard-fork, phase 1.1. Single source of truth is
pkg/apis/application/register.go's Group constant (Application/AppProject/
ApplicationSet all derive their FullName from it); every other Go call-site
that hardcoded the literal group string is updated to match: the 3 generated
informer GVRs, the app-of-apps nested-Application detector in types.go, the
resource-tree Application-kind cache populator, the 3 NotFound error
constructors in server/application, and the applicationset-controller's
leader-election Lease name/ID (kept in lockstep across Go source + RBAC
resourceNames, or leader election would get RBAC-denied on startup).
CRD YAMLs (application/appproject/applicationset) and every RBAC apiGroups
rule that actually governs our 3 CRDs are repointed to deploy.hanzo.ai, across
both manifests/base/* (namespaced Roles) and manifests/cluster-rbac/* (cluster
scope). Two adjacent things that share the "argoproj.io" string but are NOT
ours are deliberately left untouched: gitops-engine's health-check dispatch
for Argo Workflows' `Workflow` kind, and argocd-server-clusterrole's second
apiGroups block granting read on Argo Workflows' `workflows` resource — both
belong to a separate, unrelated Argoproj-family project we are not forking.
Also untouched, per explicit scope: annotation/label keys (argocd.argoproj.io/*)
and finalizers (*.argoproj.io) that track already-applied live resources, and
argocd-cm.yaml's resource.customizations.ignoreResourceUpdates keys (a narrow
app-of-apps diffing nicety, not build- or RBAC-relevant).
CGO_ENABLED=0 go build ./... verified green.
* ci: wire canonical hanzo.yml + cicd.yml (Hanzo CD image build)
Phase 1.3. Follows the same one-way pattern every hanzo/lux/zoo repo uses:
hanzo.yml declares the image (root Dockerfile -> ghcr.io/hanzoai/deploy) and a
fast go-build sanity test (the multi-stage Dockerfile build is the slow, real
gate); cicd.yml is the ~10-line trigger shim importing hanzoai/ci's reusable
build workflow on the arc pool, secrets from KMS only.
No `deploy:` block yet, matching the hanzoai/cms build-only-first-cutover
pattern: this repo's own live hanzo-cd namespace runs the whole Hanzo fleet's
GitOps, so the prod image/CRD swap is a deliberate separate action (see the
[HELD] universe PR), never an automatic rollout on merge. Every PR/push still
builds and pushes an immutable ghcr.io/hanzoai/deploy:sha-<7>-amd64 image.
* feat(ui): rebrand Argo CD -> Hanzo CD (title, favicon, logo, sidebar)
Phase 1.2. User-facing "Argo CD"/"ArgoCD" text replaced with "Hanzo CD"
across the login page, help modal, settings screens, version-info panel,
sync-options warning, fullscreen-logs title, app.tsx page titles, and
index.html's <title>/noscript text (dropped the now-wrong upstream CLI
doc link rather than invent a hanzo.ai URL that doesn't exist).
Logo/favicon assets replaced with the canonical Hanzo mark (never
hand-drawn, copied from ~/work/hanzo/logo/dist): the two real usages of
Argo's logo collapse onto one Hanzo glyph (mono + white variants) at the
same 5 call sites (login icon + footer, login.scss $logoPath/
$logoNegativePath, sidebar text-logo), preserving the existing CSS
invert() filters so both the light login panel and dark sidebar still
render correctly. Old argologo.svg/argo_o.svg/argo.png deleted (no
longer referenced). favicon-{16x16,32x32}.png added at
ui/src/assets/favicon/ (index.html's links to that path were previously
dead - the directory didn't exist in this checkout). package.json name
argo-cd-ui -> hanzo-cd-ui.
Left alone, deliberately: the two Kubernetes resource-kind icon sets
under ui/src/assets/images/resources/ (unrelated to branding), and
upstream doc/project links (argoproj.io, argo-cd.readthedocs.io) since
we have no equivalent hanzo.ai docs to point to yet - except the login
footer logo's outbound link, which now points to hanzo.ai instead of
argoproj.io (a Hanzo-branded mark linking to the upstream project it
was forked from read as an obvious leftover).
* feat(module): rename Go module github.com/argoproj/argo-cd/v3 -> github.com/hanzoai/deploy
Phase 2. No version suffix on the new path (org policy: never publish our
own Go modules above v1.x.x / with a v2+ suffix) — this fork starts a fresh
v0/v1 module identity rather than carrying the v3 the upstream fork point
had reached.
Mechanical, symmetric rewrite: 704 .go files (import statements only; two
disjoint substitutions applied specific-first so the nested gitops-engine
module path never double-matches), plus go.mod (module line, the
gitops-engine require+replace pair, version reset to v0.0.0-... since a
suffix-less path can't carry a v3 semver), gitops-engine/go.mod, Makefile
(ldflags PACKAGE + 2 TEST_MODULE filters), .golangci.yaml, .goreleaser.yaml,
.mockery.yaml (+ gitops-engine's), Procfile, the codegen/proto generation
scripts (hack/generate-proto.sh, update-codegen.sh, update-openapi.sh), and
20 .proto files' go_package options. Also one OpenTelemetry tracer name
string in gitops-engine/pkg/sync/sync_context.go (a cosmestic label, not an
import, but worth keeping consistent).
CGO_ENABLED=0 go build ./... verified green for both the root module and
the nested gitops-engine module; go vet clean on both; go mod tidy run
(network available) to reconcile go.sum, dependency set unchanged.
Left alone, deliberately: doc-only mentions (docs/snyk/*.html scan reports,
gitops-engine/README.md), a comment-only reference in
.github/workflows/release.yaml, and one Renovate bot config line — none are
build- or codegen-load-bearing. Also left alone: comments citing upstream
GitHub issue URLs (github.com/argoproj/argo-cd/issues/NNNNN) and a handful
of test fixtures using "https://github.com/argoproj/argo-cd.git" as a
representative example repo URL — both are accurate historical/test-data
references, not stale self-references.
---------
Co-authored-by: Hanzo Dev <[email protected]>
549 lines
16 KiB
Go
549 lines
16 KiB
Go
package pull_request
|
|
|
|
import (
|
|
"crypto/x509"
|
|
"encoding/pem"
|
|
"io"
|
|
"net/http"
|
|
"net/http/httptest"
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
|
|
"github.com/hanzoai/deploy/pkg/apis/application/v1alpha1"
|
|
)
|
|
|
|
func defaultHandler(t *testing.T) func(http.ResponseWriter, *http.Request) {
|
|
t.Helper()
|
|
return func(w http.ResponseWriter, r *http.Request) {
|
|
w.Header().Set("Content-Type", "application/json")
|
|
var err error
|
|
switch r.RequestURI {
|
|
case "/rest/api/1.0/projects/PROJECT/repos/REPO/pull-requests?limit=100":
|
|
_, err = io.WriteString(w, `{
|
|
"size": 1,
|
|
"limit": 100,
|
|
"isLastPage": true,
|
|
"values": [
|
|
{
|
|
"id": 101,
|
|
"title": "feat(ABC) : 123",
|
|
"toRef": {
|
|
"latestCommit": "5b766e3564a3453808f3cd3dd3f2e5fad8ef0e7a",
|
|
"displayId": "master",
|
|
"id": "refs/heads/master"
|
|
},
|
|
"fromRef": {
|
|
"id": "refs/heads/feature-ABC-123",
|
|
"displayId": "feature-ABC-123",
|
|
"latestCommit": "cb3cf2e4d1517c83e720d2585b9402dbef71f992"
|
|
},
|
|
"author": {
|
|
"user": {
|
|
"name": "testName"
|
|
}
|
|
}
|
|
}
|
|
],
|
|
"start": 0
|
|
}`)
|
|
default:
|
|
t.Fail()
|
|
}
|
|
if err != nil {
|
|
t.Fail()
|
|
}
|
|
}
|
|
}
|
|
|
|
func TestListPullRequestNoAuth(t *testing.T) {
|
|
t.Parallel()
|
|
ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
|
assert.Empty(t, r.Header.Get("Authorization"))
|
|
defaultHandler(t)(w, r)
|
|
}))
|
|
defer ts.Close()
|
|
svc, err := NewBitbucketServiceNoAuth(t.Context(), ts.URL, "PROJECT", "REPO", "", false, nil, "", "")
|
|
require.NoError(t, err)
|
|
pullRequests, err := ListPullRequests(t.Context(), svc, []v1alpha1.PullRequestGeneratorFilter{})
|
|
require.NoError(t, err)
|
|
assert.Len(t, pullRequests, 1)
|
|
assert.Equal(t, int64(101), pullRequests[0].Number)
|
|
assert.Equal(t, "feat(ABC) : 123", pullRequests[0].Title)
|
|
assert.Equal(t, "feature-ABC-123", pullRequests[0].Branch)
|
|
assert.Equal(t, "master", pullRequests[0].TargetBranch)
|
|
assert.Equal(t, "cb3cf2e4d1517c83e720d2585b9402dbef71f992", pullRequests[0].HeadSHA)
|
|
assert.Equal(t, "testName", pullRequests[0].Author)
|
|
}
|
|
|
|
func TestListPullRequestPagination(t *testing.T) {
|
|
t.Parallel()
|
|
ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
|
w.Header().Set("Content-Type", "application/json")
|
|
var err error
|
|
switch r.RequestURI {
|
|
case "/rest/api/1.0/projects/PROJECT/repos/REPO/pull-requests?limit=100":
|
|
_, err = io.WriteString(w, `{
|
|
"size": 2,
|
|
"limit": 2,
|
|
"isLastPage": false,
|
|
"values": [
|
|
{
|
|
"id": 101,
|
|
"title": "feat(101)",
|
|
"toRef": {
|
|
"latestCommit": "5b766e3564a3453808f3cd3dd3f2e5fad8ef0e7a",
|
|
"displayId": "master",
|
|
"id": "refs/heads/master"
|
|
},
|
|
"fromRef": {
|
|
"id": "refs/heads/feature-101",
|
|
"displayId": "feature-101",
|
|
"latestCommit": "ab3cf2e4d1517c83e720d2585b9402dbef71f992"
|
|
},
|
|
"author": {
|
|
"user": {
|
|
"name": "testName"
|
|
}
|
|
}
|
|
},
|
|
{
|
|
"id": 102,
|
|
"title": "feat(102)",
|
|
"toRef": {
|
|
"latestCommit": "5b766e3564a3453808f3cd3dd3f2e5fad8ef0e7a",
|
|
"displayId": "branch",
|
|
"id": "refs/heads/branch"
|
|
},
|
|
"fromRef": {
|
|
"id": "refs/heads/feature-102",
|
|
"displayId": "feature-102",
|
|
"latestCommit": "bb3cf2e4d1517c83e720d2585b9402dbef71f992"
|
|
},
|
|
"author": {
|
|
"user": {
|
|
"name": "testName"
|
|
}
|
|
}
|
|
}
|
|
],
|
|
"nextPageStart": 200
|
|
}`)
|
|
case "/rest/api/1.0/projects/PROJECT/repos/REPO/pull-requests?limit=100&start=200":
|
|
_, err = io.WriteString(w, `{
|
|
"size": 1,
|
|
"limit": 2,
|
|
"isLastPage": true,
|
|
"values": [
|
|
{
|
|
"id": 200,
|
|
"title": "feat(200)",
|
|
"toRef": {
|
|
"latestCommit": "5b766e3564a3453808f3cd3dd3f2e5fad8ef0e7a",
|
|
"displayId": "master",
|
|
"id": "refs/heads/master"
|
|
},
|
|
"fromRef": {
|
|
"id": "refs/heads/feature-200",
|
|
"displayId": "feature-200",
|
|
"latestCommit": "cb3cf2e4d1517c83e720d2585b9402dbef71f992"
|
|
},
|
|
"author": {
|
|
"user": {
|
|
"name": "testName"
|
|
}
|
|
}
|
|
}
|
|
],
|
|
"start": 200
|
|
}`)
|
|
default:
|
|
t.Fail()
|
|
}
|
|
if err != nil {
|
|
t.Fail()
|
|
}
|
|
}))
|
|
defer ts.Close()
|
|
svc, err := NewBitbucketServiceNoAuth(t.Context(), ts.URL, "PROJECT", "REPO", "", false, nil, "", "")
|
|
require.NoError(t, err)
|
|
pullRequests, err := ListPullRequests(t.Context(), svc, []v1alpha1.PullRequestGeneratorFilter{})
|
|
require.NoError(t, err)
|
|
assert.Len(t, pullRequests, 3)
|
|
assert.Equal(t, PullRequest{
|
|
Number: 101,
|
|
Title: "feat(101)",
|
|
Branch: "feature-101",
|
|
TargetBranch: "master",
|
|
HeadSHA: "ab3cf2e4d1517c83e720d2585b9402dbef71f992",
|
|
Labels: []string{},
|
|
Author: "testName",
|
|
}, *pullRequests[0])
|
|
assert.Equal(t, PullRequest{
|
|
Number: 102,
|
|
Title: "feat(102)",
|
|
Branch: "feature-102",
|
|
TargetBranch: "branch",
|
|
HeadSHA: "bb3cf2e4d1517c83e720d2585b9402dbef71f992",
|
|
Labels: []string{},
|
|
Author: "testName",
|
|
}, *pullRequests[1])
|
|
assert.Equal(t, PullRequest{
|
|
Number: 200,
|
|
Title: "feat(200)",
|
|
Branch: "feature-200",
|
|
TargetBranch: "master",
|
|
HeadSHA: "cb3cf2e4d1517c83e720d2585b9402dbef71f992",
|
|
Labels: []string{},
|
|
Author: "testName",
|
|
}, *pullRequests[2])
|
|
}
|
|
|
|
func TestListPullRequestBasicAuth(t *testing.T) {
|
|
t.Parallel()
|
|
ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
|
// base64(user:password)
|
|
assert.Equal(t, "Basic dXNlcjpwYXNzd29yZA==", r.Header.Get("Authorization"))
|
|
assert.Equal(t, "no-check", r.Header.Get("X-Atlassian-Token"))
|
|
defaultHandler(t)(w, r)
|
|
}))
|
|
defer ts.Close()
|
|
svc, err := NewBitbucketServiceBasicAuth(t.Context(), "user", "password", ts.URL, "PROJECT", "REPO", "", false, nil, "", "")
|
|
require.NoError(t, err)
|
|
pullRequests, err := ListPullRequests(t.Context(), svc, []v1alpha1.PullRequestGeneratorFilter{})
|
|
require.NoError(t, err)
|
|
assert.Len(t, pullRequests, 1)
|
|
assert.Equal(t, int64(101), pullRequests[0].Number)
|
|
assert.Equal(t, "feature-ABC-123", pullRequests[0].Branch)
|
|
assert.Equal(t, "cb3cf2e4d1517c83e720d2585b9402dbef71f992", pullRequests[0].HeadSHA)
|
|
}
|
|
|
|
func TestListPullRequestBearerAuth(t *testing.T) {
|
|
t.Parallel()
|
|
ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
|
assert.Equal(t, "Bearer tolkien", r.Header.Get("Authorization"))
|
|
assert.Equal(t, "no-check", r.Header.Get("X-Atlassian-Token"))
|
|
defaultHandler(t)(w, r)
|
|
}))
|
|
defer ts.Close()
|
|
svc, err := NewBitbucketServiceBearerToken(t.Context(), "tolkien", ts.URL, "PROJECT", "REPO", "", false, nil, "", "")
|
|
require.NoError(t, err)
|
|
pullRequests, err := ListPullRequests(t.Context(), svc, []v1alpha1.PullRequestGeneratorFilter{})
|
|
require.NoError(t, err)
|
|
assert.Len(t, pullRequests, 1)
|
|
assert.Equal(t, int64(101), pullRequests[0].Number)
|
|
assert.Equal(t, "feat(ABC) : 123", pullRequests[0].Title)
|
|
assert.Equal(t, "feature-ABC-123", pullRequests[0].Branch)
|
|
assert.Equal(t, "cb3cf2e4d1517c83e720d2585b9402dbef71f992", pullRequests[0].HeadSHA)
|
|
}
|
|
|
|
func TestListPullRequestTLS(t *testing.T) {
|
|
t.Parallel()
|
|
tests := []struct {
|
|
name string
|
|
tlsInsecure bool
|
|
passCerts bool
|
|
requireErr bool
|
|
}{
|
|
{
|
|
name: "TLS Insecure: true, No Certs",
|
|
tlsInsecure: true,
|
|
passCerts: false,
|
|
requireErr: false,
|
|
},
|
|
{
|
|
name: "TLS Insecure: true, With Certs",
|
|
tlsInsecure: true,
|
|
passCerts: true,
|
|
requireErr: false,
|
|
},
|
|
{
|
|
name: "TLS Insecure: false, With Certs",
|
|
tlsInsecure: false,
|
|
passCerts: true,
|
|
requireErr: false,
|
|
},
|
|
{
|
|
name: "TLS Insecure: false, No Certs",
|
|
tlsInsecure: false,
|
|
passCerts: false,
|
|
requireErr: true,
|
|
},
|
|
}
|
|
|
|
for _, test := range tests {
|
|
t.Run(test.name, func(t *testing.T) {
|
|
t.Parallel()
|
|
ts := httptest.NewTLSServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
|
defaultHandler(t)(w, r)
|
|
}))
|
|
defer ts.Close()
|
|
|
|
var certs []byte
|
|
if test.passCerts {
|
|
for _, cert := range ts.TLS.Certificates {
|
|
for _, c := range cert.Certificate {
|
|
parsedCert, err := x509.ParseCertificate(c)
|
|
require.NoError(t, err, "Failed to parse certificate")
|
|
certs = append(certs, pem.EncodeToMemory(&pem.Block{
|
|
Type: "CERTIFICATE",
|
|
Bytes: parsedCert.Raw,
|
|
})...)
|
|
}
|
|
}
|
|
}
|
|
|
|
svc, err := NewBitbucketServiceBasicAuth(t.Context(), "user", "password", ts.URL, "PROJECT", "REPO", "", test.tlsInsecure, certs, "", "")
|
|
require.NoError(t, err)
|
|
_, err = ListPullRequests(t.Context(), svc, []v1alpha1.PullRequestGeneratorFilter{})
|
|
if test.requireErr {
|
|
require.Error(t, err)
|
|
} else {
|
|
require.NoError(t, err)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestListResponseError(t *testing.T) {
|
|
t.Parallel()
|
|
ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
|
|
w.WriteHeader(http.StatusInternalServerError)
|
|
}))
|
|
defer ts.Close()
|
|
svc, _ := NewBitbucketServiceNoAuth(t.Context(), ts.URL, "PROJECT", "REPO", "", false, nil, "", "")
|
|
_, err := ListPullRequests(t.Context(), svc, []v1alpha1.PullRequestGeneratorFilter{})
|
|
require.Error(t, err)
|
|
}
|
|
|
|
func TestListResponseMalformed(t *testing.T) {
|
|
t.Parallel()
|
|
ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
|
w.Header().Set("Content-Type", "application/json")
|
|
switch r.RequestURI {
|
|
case "/rest/api/1.0/projects/PROJECT/repos/REPO/pull-requests?limit=100":
|
|
_, err := io.WriteString(w, `{
|
|
"size": 1,
|
|
"limit": 100,
|
|
"isLastPage": true,
|
|
"values": { "id": 101 },
|
|
"start": 0
|
|
}`)
|
|
if err != nil {
|
|
t.Fail()
|
|
}
|
|
default:
|
|
t.Fail()
|
|
}
|
|
}))
|
|
defer ts.Close()
|
|
svc, _ := NewBitbucketServiceNoAuth(t.Context(), ts.URL, "PROJECT", "REPO", "", false, nil, "", "")
|
|
_, err := ListPullRequests(t.Context(), svc, []v1alpha1.PullRequestGeneratorFilter{})
|
|
require.Error(t, err)
|
|
}
|
|
|
|
func TestListResponseEmpty(t *testing.T) {
|
|
t.Parallel()
|
|
ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
|
w.Header().Set("Content-Type", "application/json")
|
|
switch r.RequestURI {
|
|
case "/rest/api/1.0/projects/PROJECT/repos/REPO/pull-requests?limit=100":
|
|
_, err := io.WriteString(w, `{
|
|
"size": 0,
|
|
"limit": 100,
|
|
"isLastPage": true,
|
|
"values": [],
|
|
"start": 0
|
|
}`)
|
|
if err != nil {
|
|
t.Fail()
|
|
}
|
|
default:
|
|
t.Fail()
|
|
}
|
|
}))
|
|
defer ts.Close()
|
|
svc, err := NewBitbucketServiceNoAuth(t.Context(), ts.URL, "PROJECT", "REPO", "", false, nil, "", "")
|
|
require.NoError(t, err)
|
|
pullRequests, err := ListPullRequests(t.Context(), svc, []v1alpha1.PullRequestGeneratorFilter{})
|
|
require.NoError(t, err)
|
|
assert.Empty(t, pullRequests)
|
|
}
|
|
|
|
func TestListPullRequestBranchMatch(t *testing.T) {
|
|
t.Parallel()
|
|
ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
|
w.Header().Set("Content-Type", "application/json")
|
|
var err error
|
|
switch r.RequestURI {
|
|
case "/rest/api/1.0/projects/PROJECT/repos/REPO/pull-requests?limit=100":
|
|
_, err = io.WriteString(w, `{
|
|
"size": 2,
|
|
"limit": 2,
|
|
"isLastPage": false,
|
|
"values": [
|
|
{
|
|
"id": 101,
|
|
"title": "feat(101)",
|
|
"toRef": {
|
|
"latestCommit": "5b766e3564a3453808f3cd3dd3f2e5fad8ef0e7a",
|
|
"displayId": "master",
|
|
"id": "refs/heads/master"
|
|
},
|
|
"fromRef": {
|
|
"id": "refs/heads/feature-101",
|
|
"displayId": "feature-101",
|
|
"latestCommit": "ab3cf2e4d1517c83e720d2585b9402dbef71f992"
|
|
},
|
|
"author": {
|
|
"user": {
|
|
"name": "testName"
|
|
}
|
|
}
|
|
},
|
|
{
|
|
"id": 102,
|
|
"title": "feat(102)",
|
|
"toRef": {
|
|
"latestCommit": "5b766e3564a3453808f3cd3dd3f2e5fad8ef0e7a",
|
|
"displayId": "branch",
|
|
"id": "refs/heads/branch"
|
|
},
|
|
"fromRef": {
|
|
"id": "refs/heads/feature-102",
|
|
"displayId": "feature-102",
|
|
"latestCommit": "bb3cf2e4d1517c83e720d2585b9402dbef71f992"
|
|
},
|
|
"author": {
|
|
"user": {
|
|
"name": "testName"
|
|
}
|
|
}
|
|
}
|
|
],
|
|
"nextPageStart": 200
|
|
}`)
|
|
case "/rest/api/1.0/projects/PROJECT/repos/REPO/pull-requests?limit=100&start=200":
|
|
_, err = io.WriteString(w, `{
|
|
"size": 1,
|
|
"limit": 2,
|
|
"isLastPage": true,
|
|
"values": [
|
|
{
|
|
"id": 200,
|
|
"title": "feat(200)",
|
|
"toRef": {
|
|
"latestCommit": "5b766e3564a3453808f3cd3dd3f2e5fad8ef0e7a",
|
|
"displayId": "master",
|
|
"id": "refs/heads/master"
|
|
},
|
|
"fromRef": {
|
|
"id": "refs/heads/feature-200",
|
|
"displayId": "feature-200",
|
|
"latestCommit": "cb3cf2e4d1517c83e720d2585b9402dbef71f992"
|
|
},
|
|
"author": {
|
|
"user": {
|
|
"name": "testName"
|
|
}
|
|
}
|
|
}
|
|
],
|
|
"start": 200
|
|
}`)
|
|
default:
|
|
t.Fail()
|
|
}
|
|
if err != nil {
|
|
t.Fail()
|
|
}
|
|
}))
|
|
defer ts.Close()
|
|
regexp := `feature-1[\d]{2}`
|
|
svc, err := NewBitbucketServiceNoAuth(t.Context(), ts.URL, "PROJECT", "REPO", "", false, nil, "", "")
|
|
require.NoError(t, err)
|
|
pullRequests, err := ListPullRequests(t.Context(), svc, []v1alpha1.PullRequestGeneratorFilter{
|
|
{
|
|
BranchMatch: ®exp,
|
|
},
|
|
})
|
|
require.NoError(t, err)
|
|
assert.Len(t, pullRequests, 2)
|
|
assert.Equal(t, PullRequest{
|
|
Number: 101,
|
|
Title: "feat(101)",
|
|
Branch: "feature-101",
|
|
TargetBranch: "master",
|
|
HeadSHA: "ab3cf2e4d1517c83e720d2585b9402dbef71f992",
|
|
Labels: []string{},
|
|
Author: "testName",
|
|
}, *pullRequests[0])
|
|
assert.Equal(t, PullRequest{
|
|
Number: 102,
|
|
Title: "feat(102)",
|
|
Branch: "feature-102",
|
|
TargetBranch: "branch",
|
|
HeadSHA: "bb3cf2e4d1517c83e720d2585b9402dbef71f992",
|
|
Labels: []string{},
|
|
Author: "testName",
|
|
}, *pullRequests[1])
|
|
|
|
regexp = `.*2$`
|
|
svc, err = NewBitbucketServiceNoAuth(t.Context(), ts.URL, "PROJECT", "REPO", "", false, nil, "", "")
|
|
require.NoError(t, err)
|
|
pullRequests, err = ListPullRequests(t.Context(), svc, []v1alpha1.PullRequestGeneratorFilter{
|
|
{
|
|
BranchMatch: ®exp,
|
|
},
|
|
})
|
|
require.NoError(t, err)
|
|
assert.Len(t, pullRequests, 1)
|
|
assert.Equal(t, PullRequest{
|
|
Number: 102,
|
|
Title: "feat(102)",
|
|
Branch: "feature-102",
|
|
TargetBranch: "branch",
|
|
HeadSHA: "bb3cf2e4d1517c83e720d2585b9402dbef71f992",
|
|
Labels: []string{},
|
|
Author: "testName",
|
|
}, *pullRequests[0])
|
|
|
|
regexp = `[\d{2}`
|
|
svc, err = NewBitbucketServiceNoAuth(t.Context(), ts.URL, "PROJECT", "REPO", "", false, nil, "", "")
|
|
require.NoError(t, err)
|
|
_, err = ListPullRequests(t.Context(), svc, []v1alpha1.PullRequestGeneratorFilter{
|
|
{
|
|
BranchMatch: ®exp,
|
|
},
|
|
})
|
|
require.Error(t, err)
|
|
}
|
|
|
|
func TestBitbucketServerListReturnsRepositoryNotFoundError(t *testing.T) {
|
|
t.Parallel()
|
|
mux := http.NewServeMux()
|
|
server := httptest.NewServer(mux)
|
|
defer server.Close()
|
|
|
|
path := "/rest/api/1.0/projects/nonexistent/repos/nonexistent/pull-requests?limit=100"
|
|
|
|
mux.HandleFunc(path, func(w http.ResponseWriter, _ *http.Request) {
|
|
// Return 404 status to simulate repository not found
|
|
w.WriteHeader(http.StatusNotFound)
|
|
_, _ = w.Write([]byte(`{"message": "404 Project Not Found"}`))
|
|
})
|
|
|
|
svc, err := NewBitbucketServiceNoAuth(t.Context(), server.URL, "nonexistent", "nonexistent", "", false, nil, "", "")
|
|
require.NoError(t, err)
|
|
|
|
prs, err := svc.List(t.Context())
|
|
|
|
// Should return empty pull requests list
|
|
assert.Empty(t, prs)
|
|
|
|
// Should return RepositoryNotFoundError
|
|
require.Error(t, err)
|
|
assert.True(t, IsRepositoryNotFoundError(err), "Expected RepositoryNotFoundError but got: %v", err)
|
|
}
|