feedback

Author: milldr

internal/exec/pro.go

@@ -216,7 +216,7 @@ func uploadStatus(info *schema.ConfigAndStacksInfo, exitCode int, client pro.Atm
 	}
 
 	// Get the git repository info
-	repoInfo, err := gitRepo.GetLocalRepo()
+	repoInfo, err := gitRepo.GetLocalRepoInfo()
 	if err != nil {
 		return fmt.Errorf(atmosErrors.ErrWrappingFormat, atmosErrors.ErrFailedToGetLocalRepo, err)
 	}

internal/exec/pro_test.go

@@ -31,7 +31,7 @@ type MockGitRepo struct {
 	mock.Mock
 }
 
-func (m *MockGitRepo) GetLocalRepo() (*atmosgit.RepoInfo, error) {
+func (m *MockGitRepo) GetLocalRepoInfo() (*atmosgit.RepoInfo, error) {
 	args := m.Called()
 	if args.Get(0) == nil {
 		return nil, args.Error(1)
@@ -193,7 +193,7 @@ func TestUploadStatus(t *testing.T) {
 			// The function only processes exit codes 0 and 2
 			if tc.exitCode == 0 || tc.exitCode == 2 {
 				// Set up mock expectations for git functions
-				mockGitRepo.On("GetLocalRepo").Return(testRepoInfo, nil)
+				mockGitRepo.On("GetLocalRepoInfo").Return(testRepoInfo, nil)
 				mockGitRepo.On("GetCurrentCommitSHA").Return("abc123def456", nil)
 
 				// Set up mock expectations for pro client

pkg/git/git.go

@@ -110,16 +110,16 @@ func GetRepoInfo(localRepo *git.Repository) (RepoInfo, error) {
 
 // GitRepoInterface defines the interface for git repository operations.
 type GitRepoInterface interface {
-	GetLocalRepo() (*RepoInfo, error)
+	GetLocalRepoInfo() (*RepoInfo, error)
 	GetRepoInfo(repo *git.Repository) (RepoInfo, error)
 	GetCurrentCommitSHA() (string, error)
 }
 
 // DefaultGitRepo is the default implementation of GitRepoInterface.
 type DefaultGitRepo struct{}
 
-// GetLocalRepo returns information about the local git repository.
-func (d *DefaultGitRepo) GetLocalRepo() (*RepoInfo, error) {
+// GetLocalRepoInfo returns information about the local git repository.
+func (d *DefaultGitRepo) GetLocalRepoInfo() (*RepoInfo, error) {
 	repo, err := GetLocalRepo()
 	if err != nil {
 		return nil, fmt.Errorf("%w: failed to get local repository: %v", errUtils.ErrFailedToGetLocalRepo, err)