source: use subtests cases for git identifier

Signed-off-by: CrazyMax <[email protected]>

Author: crazy-max

source/gitidentifier_test.go

@@ -7,27 +7,43 @@ import (
 )
 
 func TestNewGitIdentifier(t *testing.T) {
-	gi, err := NewGitIdentifier("ssh://[email protected]:2222/root/my/really/weird/path/foo.git")
-	require.Nil(t, err)
-	require.Equal(t, "ssh://[email protected]:2222/root/my/really/weird/path/foo.git", gi.Remote)
-	require.Equal(t, "", gi.Ref)
-	require.Equal(t, "", gi.Subdir)
-
-	gi, err = NewGitIdentifier("ssh://[email protected]:2222/root/my/really/weird/path/foo.git#main")
-	require.Nil(t, err)
-	require.Equal(t, "ssh://[email protected]:2222/root/my/really/weird/path/foo.git", gi.Remote)
-	require.Equal(t, "main", gi.Ref)
-	require.Equal(t, "", gi.Subdir)
-
-	gi, err = NewGitIdentifier("[email protected]:moby/buildkit.git")
-	require.Nil(t, err)
-	require.Equal(t, "[email protected]:moby/buildkit.git", gi.Remote)
-	require.Equal(t, "", gi.Ref)
-	require.Equal(t, "", gi.Subdir)
-
-	gi, err = NewGitIdentifier("github.com/moby/buildkit.git#main")
-	require.Nil(t, err)
-	require.Equal(t, "https://github.com/moby/buildkit.git", gi.Remote)
-	require.Equal(t, "main", gi.Ref)
-	require.Equal(t, "", gi.Subdir)
+	tests := []struct {
+		url      string
+		expected GitIdentifier
+	}{
+		{
+			url: "ssh://[email protected]:2222/root/my/really/weird/path/foo.git",
+			expected: GitIdentifier{
+				Remote: "ssh://[email protected]:2222/root/my/really/weird/path/foo.git",
+			},
+		},
+		{
+			url: "ssh://[email protected]:2222/root/my/really/weird/path/foo.git#main",
+			expected: GitIdentifier{
+				Remote: "ssh://[email protected]:2222/root/my/really/weird/path/foo.git",
+				Ref:    "main",
+			},
+		},
+		{
+			url: "[email protected]:moby/buildkit.git",
+			expected: GitIdentifier{
+				Remote: "[email protected]:moby/buildkit.git",
+			},
+		},
+		{
+			url: "github.com/moby/buildkit.git#main",
+			expected: GitIdentifier{
+				Remote: "https://github.com/moby/buildkit.git",
+				Ref:    "main",
+			},
+		},
+	}
+	for _, tt := range tests {
+		tt := tt
+		t.Run(tt.url, func(t *testing.T) {
+			gi, err := NewGitIdentifier(tt.url)
+			require.NoError(t, err)
+			require.Equal(t, tt.expected, *gi)
+		})
+	}
 }