repo_tag: able to list tags using sort and filter options (#71)

Co-authored-by: Joe Chen <[email protected]>

Author: sermojohn

repo_tag.go

@@ -143,6 +143,10 @@ func (r *Repository) Tag(name string, opts ...TagOptions) (*Tag, error) {
 //
 // Docs: https://git-scm.com/docs/git-tag#Documentation/git-tag.txt---list
 type TagsOptions struct {
+	// SortKet sorts tags with provided tag key, optionally prefixed with '-' to sort tags in descending order.
+	SortKey string
+	// Pattern filters tags matching the specified pattern.
+	Pattern string
 	// The timeout duration before giving up for each shell command execution. The
 	// default timeout duration will be used when not supplied.
 	Timeout time.Duration
@@ -161,8 +165,18 @@ func RepoTags(repoPath string, opts ...TagsOptions) ([]string, error) {
 	}
 
 	cmd := NewCommand("tag", "--list")
-	if goversion.Compare(version, "2.4.9", ">=") {
+
+	var sorted bool
+	if opt.SortKey != "" {
+		cmd.AddArgs("--sort=" + opt.SortKey)
+		sorted = true
+	} else if goversion.Compare(version, "2.4.9", ">=") {
 		cmd.AddArgs("--sort=-creatordate")
+		sorted = true
+	}
+
+	if opt.Pattern != "" {
+		cmd.AddArgs(opt.Pattern)
 	}
 
 	stdout, err := cmd.RunInDirWithTimeout(opt.Timeout, repoPath)
@@ -173,7 +187,7 @@ func RepoTags(repoPath string, opts ...TagsOptions) ([]string, error) {
 	tags := strings.Split(string(stdout), "\n")
 	tags = tags[:len(tags)-1]
 
-	if goversion.Compare(version, "2.4.9", "<") {
+	if !sorted {
 		goversion.Sort(tags)
 
 		// Reverse order

repo_tag_test.go

@@ -58,6 +58,37 @@ func TestRepository_Tags(t *testing.T) {
 	assert.NotEmpty(t, tags)
 }
 
+func TestRepository_Tags_VersionSort(t *testing.T) {
+	r, cleanup, err := setupTempRepo()
+	if err != nil {
+		t.Fatal(err)
+	}
+	defer cleanup()
+
+	err = r.CreateTag("v3.0.0", "master")
+	if err != nil {
+		t.Fatal(err)
+	}
+	err = r.CreateTag("v2.999.0", "master")
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	tags, err := r.Tags(TagsOptions{
+		SortKey: "-version:refname",
+		Pattern: "v*",
+	})
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	if len(tags) < 2 {
+		t.Fatalf("Should have at least two tags but got %d", len(tags))
+	}
+	assert.Equal(t, "v3.0.0", tags[0])
+	assert.Equal(t, "v2.999.0", tags[1])
+}
+
 func TestRepository_CreateTag(t *testing.T) {
 	r, cleanup, err := setupTempRepo()
 	if err != nil {