Merge pull request #31 from sonic-screwdriver-cd/upload-artifacts

feat: enable to upload artifacts

Author: catto

store-cli.go

@@ -89,7 +89,9 @@ func makeURL(storeType, scope, key string) (*url.URL, error) {
 		encoded := url.QueryEscape(key)
 		path = "caches/" + scope + "s/" + scopeEnv + "/" + encoded
 	case "artifact":
-		path = "builds/" + os.Getenv("SD_BUILD_ID") + "/ARTIFACTS/" + key
+		key = strings.TrimLeft(key, "./")
+		encoded := url.QueryEscape(key)
+		path = "builds/" + os.Getenv("SD_BUILD_ID") + "/ARTIFACTS/" + encoded
 	case "log":
 		path = "builds/" + os.Getenv("SD_BUILD_ID") + "-" + key
 	default:

store-cli_test.go

@@ -70,6 +70,11 @@ func TestMakeURL(t *testing.T) {
 		{"cache", "event", "/!-_.*'()&@:,.$=+?; space", "http://store.screwdriver.cd/v1/caches/events/499/%2F%21-_.%2A%27%28%29%26%40%3A%2C.%24%3D%2B%3F%3B+space"},
 		{"artifact", "event", "artifact-1", "http://store.screwdriver.cd/v1/builds/10038/ARTIFACTS/artifact-1"},
 		{"artifact", "build", "test", "http://store.screwdriver.cd/v1/builds/10038/ARTIFACTS/test"},
+		{"artifact", "", "./test", "http://store.screwdriver.cd/v1/builds/10038/ARTIFACTS/test"},
+		{"artifact", "", "test/foo", "http://store.screwdriver.cd/v1/builds/10038/ARTIFACTS/test%2Ffoo"},
+		{"artifact", "", "test/foo./bar", "http://store.screwdriver.cd/v1/builds/10038/ARTIFACTS/test%2Ffoo.%2Fbar"},
+		{"artifact", "", "test/foo/あいうえお.txt", "http://store.screwdriver.cd/v1/builds/10038/ARTIFACTS/test%2Ffoo%2F%E3%81%82%E3%81%84%E3%81%86%E3%81%88%E3%81%8A.txt"},
+		{"artifact", "", "!-_.*'()&@:,.$=+?; space", "http://store.screwdriver.cd/v1/builds/10038/ARTIFACTS/%21-_.%2A%27%28%29%26%40%3A%2C.%24%3D%2B%3F%3B+space"},
 		{"log", "build", "testlog", "http://store.screwdriver.cd/v1/builds/10038-testlog"},
 		{"log", "build", "step-bookend", "http://store.screwdriver.cd/v1/builds/10038-step-bookend"},
 		{"log", "pipeline", "test-2", "http://store.screwdriver.cd/v1/builds/10038-test-2"},