removed GetRepository (#57)

pkg/git/provider.go

@@ -14,8 +14,6 @@ type (
 		// CreateRepository creates the repository in the remote provider and returns a
 		// clone url
 		CreateRepository(ctx context.Context, opts *CreateRepoOptions) (string, error)
-
-		GetRepository(ctx context.Context, opts *GetRepoOptions) (string, error)
 	}
 
 	Auth struct {

pkg/git/provider_github.go

@@ -49,19 +49,6 @@ func newGithub(opts *ProviderOptions) (Provider, error) {
 	return g, nil
 }
 
-func (g *github) GetRepository(ctx context.Context, opts *GetRepoOptions) (string, error) {
-	r, res, err := g.Repositories.Get(ctx, opts.Owner, opts.Name)
-	if res != nil && res.StatusCode == http.StatusNotFound {
-		return "", ErrRepoNotFound
-	}
-
-	if err != nil {
-		return "", err
-	}
-
-	return *r.CloneURL, nil
-}
-
 func (g *github) CreateRepository(ctx context.Context, opts *CreateRepoOptions) (string, error) {
 	authUser, res, err := g.Users.Get(ctx, "") // get authenticated user details
 	if err != nil {

pkg/git/provider_github_test.go

@@ -11,71 +11,6 @@ import (
 	"github.com/stretchr/testify/assert"
 )
 
-func Test_github_GetRepository(t *testing.T) {
-	tests := map[string]struct {
-		repo    *gh.Repository
-		res     *gh.Response
-		err     error
-		opts    *GetRepoOptions
-		want    string
-		wantErr string
-	}{
-		"No repo": {
-			res: &gh.Response{
-				Response: &http.Response{
-					StatusCode: http.StatusNotFound,
-				},
-			},
-			opts: &GetRepoOptions{
-				Owner: "owner",
-				Name:  "repo",
-			},
-			want:    "",
-			wantErr: "git repository not found",
-		},
-		"Has repo": {
-			repo: &gh.Repository{
-				CloneURL: gh.String("https://github.com/owner/repo"),
-			},
-			opts: &GetRepoOptions{
-				Owner: "owner",
-				Name:  "repo",
-			},
-			want:    "https://github.com/owner/repo",
-			wantErr: "",
-		},
-		"Error getting repo": {
-			err: errors.New("Some error"),
-			opts: &GetRepoOptions{
-				Owner: "owner",
-				Name:  "repo",
-			},
-			want:    "",
-			wantErr: "Some error",
-		},
-	}
-
-	for name, tt := range tests {
-		t.Run(name, func(t *testing.T) {
-			mockRepo := new(mocks.Repositories)
-			ctx := context.Background()
-			mockRepo.On("Get", ctx, tt.opts.Owner, tt.opts.Name).Return(tt.repo, tt.res, tt.err)
-			g := &github{
-				Repositories: mockRepo,
-			}
-			got, err := g.GetRepository(ctx, tt.opts)
-			if tt.wantErr != "" {
-				assert.EqualError(t, err, tt.wantErr)
-				return
-			}
-
-			if got != tt.want {
-				t.Errorf("github.GetRepository() = %v, want %v", got, tt.want)
-			}
-		})
-	}
-}
-
 func Test_github_CreateRepository(t *testing.T) {
 	tests := map[string]struct {
 		user    *gh.User