fix(forge): use canonical org repo creation route
Co-Authored-By: Virgil <virgil@lethean.io>
This commit is contained in:
parent
49daff2cb6
commit
c7596dfea3
2 changed files with 29 additions and 1 deletions
2
repos.go
2
repos.go
|
|
@ -98,7 +98,7 @@ func (s *RepoService) Migrate(ctx context.Context, opts *types.MigrateRepoOption
|
|||
|
||||
// CreateOrgRepo creates a repository in an organisation.
|
||||
func (s *RepoService) CreateOrgRepo(ctx context.Context, org string, opts *types.CreateRepoOption) (*types.Repository, error) {
|
||||
path := ResolvePath("/api/v1/org/{org}/repos", pathParams("org", org))
|
||||
path := ResolvePath("/api/v1/orgs/{org}/repos", pathParams("org", org))
|
||||
var out types.Repository
|
||||
if err := s.client.Post(ctx, path, opts, &out); err != nil {
|
||||
return nil, err
|
||||
|
|
|
|||
|
|
@ -2127,6 +2127,34 @@ func TestRepoService_PathParamsAreEscaped_Good(t *testing.T) {
|
|||
}
|
||||
})
|
||||
|
||||
t.Run("CreateOrgRepo", func(t *testing.T) {
|
||||
srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
||||
want := "/api/v1/orgs/team%20alpha/repos"
|
||||
if r.URL.EscapedPath() != want {
|
||||
t.Errorf("got path %q, want %q", r.URL.EscapedPath(), want)
|
||||
http.NotFound(w, r)
|
||||
return
|
||||
}
|
||||
var opts types.CreateRepoOption
|
||||
if err := json.NewDecoder(r.Body).Decode(&opts); err != nil {
|
||||
t.Fatalf("decode body: %v", err)
|
||||
}
|
||||
if opts.Name != "go-forge" || !opts.Private {
|
||||
t.Fatalf("got %#v", opts)
|
||||
}
|
||||
json.NewEncoder(w).Encode(types.Repository{Name: opts.Name})
|
||||
}))
|
||||
defer srv.Close()
|
||||
|
||||
f := NewForge(srv.URL, "tok")
|
||||
if _, err := f.Repos.CreateOrgRepo(context.Background(), org, &types.CreateRepoOption{
|
||||
Name: "go-forge",
|
||||
Private: true,
|
||||
}); err != nil {
|
||||
t.Fatal(err)
|
||||
}
|
||||
})
|
||||
|
||||
t.Run("Fork", func(t *testing.T) {
|
||||
srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
||||
want := "/api/v1/repos/acme%20org/my%2Frepo/forks"
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue