commit_submodule: make fields public (#48)

Author: unknwon

commit_submodule.go

@@ -10,6 +10,16 @@ import (
 	"strings"
 )
 
+// Submodule contains information of a Git submodule.
+type Submodule struct {
+	// The name of the submodule.
+	Name string
+	// The URL of the submodule.
+	URL string
+	// The commit ID of the subproject.
+	Commit string
+}
+
 // Submodules contains information of submodules.
 type Submodules = *objectCache
 
@@ -46,11 +56,11 @@ func (c *Commit) Submodules() (Submodules, error) {
 				path = strings.TrimSpace(fields[1])
 			case "url":
 				mod := &Submodule{
-					name: path,
-					url:  strings.TrimSpace(fields[1]),
+					Name: path,
+					URL:  strings.TrimSpace(fields[1]),
 				}
 
-				mod.commit, c.submodulesErr = c.repo.RevParse("@:" + mod.name)
+				mod.Commit, c.submodulesErr = c.repo.RevParse("@:" + mod.Name)
 				if c.submodulesErr != nil {
 					return
 				}

commit_submodule_test.go

@@ -20,9 +20,9 @@ func TestCommit_Submodule(t *testing.T) {
 	if err != nil {
 		t.Fatal(err)
 	}
-	assert.Equal(t, "gogs/docs-api", mod.Name())
-	assert.Equal(t, "https://github.com/gogs/docs-api.git", mod.URL())
-	assert.Equal(t, "6b08f76a5313fa3d26859515b30aa17a5faa2807", mod.Commit())
+	assert.Equal(t, "gogs/docs-api", mod.Name)
+	assert.Equal(t, "https://github.com/gogs/docs-api.git", mod.URL)
+	assert.Equal(t, "6b08f76a5313fa3d26859515b30aa17a5faa2807", mod.Commit)
 
 	_, err = c.Submodule("404")
 	assert.Equal(t, ErrSubmoduleNotExist, err)