Add TODO to handle path-parsing error

Remove non-parsable path test due to pending decision on intended behavior

api/internal/git/repospec.go

@@ -219,6 +219,7 @@ func parseQuery(query string) (string, time.Duration, bool) {
 func parsePath(n string) string {
 	parsed, err := url.Parse(n)
 	// in event of parse failure, return default
+	// TODO(annasong): decide how to handle error, i.e. return error, empty string, etc.
 	if err != nil {
 		return n
 	}

api/internal/git/repospec_test.go

@@ -443,18 +443,6 @@ func TestNewRepoSpecFromUrl_Smoke(t *testing.T) {
 				GitSuffix: ".git",
 			},
 		},
-		{
-			name:      "non_parsable_path",
-			input:     "https://authority/org/repo/%-invalid-uri-so-not-parsable-by-net/url.Parse",
-			cloneSpec: "https://authority/org/repo.git",
-			absPath:   notCloned.Join("%-invalid-uri-so-not-parsable-by-net/url.Parse"),
-			repoSpec: RepoSpec{
-				Host:      "https://authority/",
-				OrgRepo:   "org/repo",
-				Path:      "%-invalid-uri-so-not-parsable-by-net/url.Parse",
-				GitSuffix: ".git",
-			},
-		},
 	}
 	for _, tc := range testcases {
 		t.Run(tc.name, func(t *testing.T) {