revert changes for parsing the curl cmd arg

Author: reshmifrog

common/commands/curl.go

@@ -146,34 +146,18 @@ func (curlCmd *CurlCommand) GetServerDetails() (*config.ServerDetails, error) {
 // Use this method ONLY after removing all JFrog-CLI flags, i.e. flags in the form: '--my-flag=value' are not allowed.
 // An argument is any provided candidate which is not a flag or a flag value.
 func (curlCmd *CurlCommand) findUriValueAndIndex() (int, string) {
-	cntArgs := 0
-	for range curlCmd.arguments {
-		cntArgs++
-	}
-
 	skipThisArg := false
 	for index, arg := range curlCmd.arguments {
-		if skipThisArg && !strings.HasPrefix(arg, "-") && index == cntArgs-1 {
-			return index, arg
-		}
-
 		// Check if shouldn't check current arg.
-		if skipThisArg && !strings.HasPrefix(arg, "-") {
+		if skipThisArg {
 			skipThisArg = false
 			continue
 		}
-
-		// If we were expecting a value but found a flag instead, reset skipThisArg
-		if skipThisArg && strings.HasPrefix(arg, "-") {
-			skipThisArg = false
-		}
-
 		// If starts with '--', meaning a flag which its value is at next slot.
 		if strings.HasPrefix(arg, "--") {
 			skipThisArg = true
 			continue
 		}
-
 		// Check if '-'.
 		if strings.HasPrefix(arg, "-") {
 			if len(arg) > 2 {

common/commands/curl_test.go

@@ -10,16 +10,6 @@ func TestFindNextArg(t *testing.T) {
 		{"-X", "GET", "arg1", "--foo", "bar"},
 		{"-X", "GET", "--server-idea", "foo", "/api/arg2"},
 		{"-XGET", "--foo", "bar", "--foo-bar", "meow", "arg3"},
-		{"-o", "helm.tar.gz", "-L", "-vvv", "helm-sh/helm-v3.19.0-linux-amd64.tar.gz"},
-		{"-o", "helm.tar.gz", "-vvv", "-L", "helm-sh/helm-v3.19.0-linux-amd64.tar.gz"},
-		{"-o", "helm.tar.gz", "-L", "-s", "helm-sh/helm-v3.19.0-linux-amd64.tar.gz"},
-		{"-L", "-o", "helm.tar.gz", "helm-sh/helm-v3.19.0-linux-amd64.tar.gz"},
-		{"-o", "helm.tar.gz", "-L", "helm-sh/helm-v3.19.0-linux-amd64.tar.gz"},
-		{"-L", "-vvv", "-o", "helm.tar.gz", "helm-sh/helm-v3.19.0-linux-amd64.tar.gz"},
-		{"-L", "-o", "helm.tar.gz", "-vvv", "helm-sh/helm-v3.19.0-linux-amd64.tar.gz"},
-		{"-vvv", "-s", "-L", "api/test"},
-		{"-LsS", "api/test2"},
-		{"-ofile.txt", "-L", "api/test3"},
 	}
 
 	expected := []struct {
@@ -29,24 +19,6 @@ func TestFindNextArg(t *testing.T) {
 		{2, "arg1"},
 		{4, "/api/arg2"},
 		{5, "arg3"},
-		{4, "helm-sh/helm-v3.19.0-linux-amd64.tar.gz"},
-		{4, "helm-sh/helm-v3.19.0-linux-amd64.tar.gz"},
-		{4, "helm-sh/helm-v3.19.0-linux-amd64.tar.gz"},
-		{3, "helm-sh/helm-v3.19.0-linux-amd64.tar.gz"},
-		{3, "helm-sh/helm-v3.19.0-linux-amd64.tar.gz"},
-		{4, "helm-sh/helm-v3.19.0-linux-amd64.tar.gz"},
-		{4, "helm-sh/helm-v3.19.0-linux-amd64.tar.gz"},
-		{3, "api/test"},
-		{1, "api/test2"},
-		{2, "api/test3"},
-		{-1, ""},
-		{4, "api/endpoint"},
-		{0, "api/endpoint"},
-		{2, "api/endpoint"},
-		{2, "api/endpoint"},
-		{-1, ""},
-		{-1, ""},
-		{-1, ""},
 	}
 
 	for index, test := range args {
@@ -99,7 +71,7 @@ func TestBuildCommandUrl(t *testing.T) {
 		{"test1", []string{"-X", "GET", "/api/build/test1", "--server-id", "test1", "--foo", "bar"}, 2, "https://artifactory:8081/artifactory/api/build/test1", false},
 		{"test2", []string{"-X", "GET", "/api/build/test2", "--server-idea", "foo", "--server-id=test2"}, 2, "https://artifactory:8081/artifactory/api/build/test2", false},
 		{"test3", []string{"-XGET", "--/api/build/test3", "--server-id="}, 1, "https://artifactory:8081/artifactory/api/build/test3", true},
-		{"test4", []string{"-XGET", "-Test4", "--server-id", "bar"}, 3, "https://artifactory:8081/artifactory/bar", false},
+		{"test4", []string{"-XGET", "-Test4", "--server-id", "bar"}, 1, "https://artifactory:8081/artifactory/api/build/test4", true},
 		{"test5", []string{"-X", "GET", "api/build/test5", "--server-id", "test5", "--foo", "bar"}, 2, "https://artifactory:8081/artifactory/api/build/test5", false},
 	}