fix: Add scope a second time to getdeploypath (#273)

jfrog · Sep 21, 2024 · c694d21 · c694d21
1 parent ad5b104
commit c694d21
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/build/utils/npm.go b/build/utils/npm.go
@@ -514,7 +514,7 @@ func (pi *PackageInfo) GetDeployPath() string {
 	if pi.Scope == "" {
 		return fmt.Sprintf("%s/-/%s", pi.Name, fileName)
 	}
-	return fmt.Sprintf("%s/%s/-/%s", pi.Scope, pi.Name, fileName)
+	return fmt.Sprintf("%s/%s/-/%s/%s", pi.Scope, pi.Name, pi.Scope, fileName)
 }
 
 func (pi *PackageInfo) FullName() string {

diff --git a/build/utils/npm_test.go b/build/utils/npm_test.go
@@ -99,7 +99,7 @@ func TestGetDeployPath(t *testing.T) {
 		pi           *PackageInfo
 	}{
 		{`build-info-go-tests/-/build-info-go-tests-1.0.0.tgz`, &PackageInfo{Name: "build-info-go-tests", Version: "1.0.0", Scope: ""}},
-		{`@jfrog/build-info-go-tests/-/build-info-go-tests-1.0.0.tgz`, &PackageInfo{Name: "build-info-go-tests", Version: "1.0.0", Scope: "@jfrog"}},
+		{`@jfrog/build-info-go-tests/-/@jfrog/build-info-go-tests-1.0.0.tgz`, &PackageInfo{Name: "build-info-go-tests", Version: "1.0.0", Scope: "@jfrog"}},
 	}
 	for _, test := range testcases {
 		t.Run(test.expectedPath, func(t *testing.T) {