Merge pull request #182 from DeepakPatankar/deepak/add-path-as-a-para…

…m-in-list-branches

Add the support to fetch commit of a particular file

scm/driver/stash/git.go

@@ -81,9 +81,14 @@ func (s *gitService) ListBranches(ctx context.Context, repo string, opts scm.Lis
 
 func (s *gitService) ListCommits(ctx context.Context, repo string, opts scm.CommitListOptions) ([]*scm.Commit, *scm.Response, error) {
 	namespace, name := scm.Split(repo)
-	path := fmt.Sprintf("rest/api/1.0/projects/%s/repos/%s/commits", namespace, name)
+	var requestPath string
+	if opts.Path != "" {
+		requestPath = fmt.Sprintf("rest/api/1.0/projects/%s/repos/%s/commits?path=%s", namespace, name, opts.Path)
+	} else {
+		requestPath = fmt.Sprintf("rest/api/1.0/projects/%s/repos/%s/commits", namespace, name)
+	}
 	out := new(commits)
-	res, err := s.client.do(ctx, "GET", path, nil, out)
+	res, err := s.client.do(ctx, "GET", requestPath, nil, out)
 	copyPagination(out.pagination, res)
 	return convertCommitList(out), res, err
 }

scm/driver/stash/integration/git_test.go

@@ -34,5 +34,31 @@ func TestCreateBranch(t *testing.T) {
 	if response.Status != http.StatusOK {
 		t.Errorf("CreateBranch did not get a 200 back %v", response.Status)
 	}
+}
 
+func TestGetLatestCommitOfBranch(t *testing.T) {
+	if token == "" {
+		t.Skip("Skipping, Acceptance test")
+	}
+	client, _ = stash.New(endpoint)
+	client.Client = &http.Client{
+		Transport: &transport.BasicAuth{
+			Username: username,
+			Password: token,
+		},
+	}
+
+	commits, response, err := client.Git.ListCommits(context.Background(), repoID, scm.CommitListOptions{Ref: "master", Path: "README"})
+
+	if err != nil {
+		t.Errorf("GetLatestCommitOfFile got an error %v", err)
+	} else {
+		if response.Status != http.StatusOK {
+			t.Errorf("GetLatestCommitOfFile did not get a 200 back %v", response.Status)
+		}
+
+		if commits[0].Sha != "2cc4dbe084f0d66761318b305c408cb0ea300c9a" {
+			t.Errorf("Got the commitId %s instead of the top commit of the file", commits[0].Sha)
+		}
+	}
 }