Added test case for --build-name with spaces

Author: agrasthn

artifactory_test.go

@@ -3267,6 +3267,39 @@ func TestArtifactoryDownloadByShaAndBuild(t *testing.T) {
 	cleanArtifactoryTest()
 }
 
+func TestArtifactoryUploadWithBuildNameWithSpaces(t *testing.T) {
+	initArtifactoryTest(t, "")
+
+	buildNumber := "1"
+
+	// Delete the build if it already exists
+	inttestutils.DeleteBuild(serverDetails.ArtifactoryUrl, tests.RtBuildNameWithSpaces, artHttpDetails)
+
+	// Upload a file with build name containing spaces
+	runRt(t, "upload", "testdata/a/a1.in", tests.RtRepo1+"/data/a1.in",
+		"--build-name="+tests.RtBuildNameWithSpaces, "--build-number="+buildNumber)
+
+	// Publish build info
+	runRt(t, "build-publish", tests.RtBuildNameWithSpaces, buildNumber)
+
+	// Validate that the file was uploaded successfully
+	specFile, err := tests.CreateSpec(tests.BuildDownloadSpecNoBuildNumber)
+	assert.NoError(t, err)
+
+	runRt(t, "download", "--spec="+specFile)
+
+	// Validate downloaded files
+	paths, err := fileutils.ListFilesRecursiveWalkIntoDirSymlink(tests.Out, false)
+	assert.NoError(t, err)
+	err = tests.ValidateListsIdentical(tests.GetBuildDownload(), paths)
+	assert.NoError(t, err)
+
+	// Cleanup
+	inttestutils.DeleteBuild(serverDetails.ArtifactoryUrl, tests.RtBuildNameWithSpaces, artHttpDetails)
+	cleanArtifactoryTest()
+}
+
+
 // Upload a file to 2 different builds.
 // Verify that we don't download files with same sha and build name and different build number.
 func TestArtifactoryDownloadByShaAndBuildName(t *testing.T) {

utils/tests/consts.go

@@ -221,6 +221,7 @@ var (
 	PipenvBuildName             = "cli-pipenv-build"
 	RtBuildName1                = "cli-rt-build1"
 	RtBuildName2                = "cli-rt-build2"
+	RtBuildNameWithSpaces       = "cli build with spaces"
 	RtBuildNameWithSpecialChars = "cli-rt-a$+~&^a#-build3"
 	RtPermissionTargetName      = "cli-rt-pt"
 	LcBuildName1                = "cli-lc-build1"