Added some more cases to the tests for chars encoding (#3758)

Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>

cmd/asset-syncer/server/testdata/helm-index-spaces.yaml

@@ -5,3 +5,9 @@ entries:
   chart-without-spaces:
     - appVersion: v2
       name: "chart-without-spaces"
+  chart-with-special-chars-1:
+    - appVersion: v3
+      name: "chart\x23with\x23hashes"
+  chart-with-special-chars-4:
+    - appVersion: v1
+      name: "chart\x24with\x24chars"

cmd/asset-syncer/server/utils_test.go

@@ -1347,6 +1347,15 @@ func TestUnescapeChartsData(t *testing.T) {
 				{Name: "test/foo bar"},
 			},
 		},
+		{
+			"chart with encoded chars in name",
+			[]models.Chart{
+				{Name: "foo%23bar%2ebar"},
+			},
+			[]models.Chart{
+				{Name: "foo#bar.bar"},
+			},
+		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.description, func(t *testing.T) {
@@ -1364,13 +1373,27 @@ func TestHelmRepoAppliesUnescape(t *testing.T) {
 	repoIndexYAMLBytes, _ := ioutil.ReadFile("testdata/helm-index-spaces.yaml")
 	repoIndexYAML := string(repoIndexYAMLBytes)
 	expectedCharts := []models.Chart{
+		{
+			ID:            "test/chart$with$chars",
+			Name:          "chart$with$chars",
+			Repo:          expectedRepo,
+			Maintainers:   []chart.Maintainer{},
+			ChartVersions: []models.ChartVersion{{AppVersion: "v1"}},
+		},
 		{
 			ID:            "test/chart with spaces",
 			Name:          "chart with spaces",
 			Repo:          expectedRepo,
 			Maintainers:   []chart.Maintainer{},
 			ChartVersions: []models.ChartVersion{{AppVersion: "v1"}},
 		},
+		{
+			ID:            "test/chart#with#hashes",
+			Name:          "chart#with#hashes",
+			Repo:          expectedRepo,
+			Maintainers:   []chart.Maintainer{},
+			ChartVersions: []models.ChartVersion{{AppVersion: "v3"}},
+		},
 		{
 			ID:            "test/chart-without-spaces",
 			Name:          "chart-without-spaces",