SDI-2101:issue1273 Fixed the parseNamespace to allow arbitary separators

mgmt/rest/server.go

@@ -516,11 +516,17 @@ func respond(code int, b rbody.Body, w http.ResponseWriter) {
 }
 
 func parseNamespace(ns string) []string {
-	if strings.Index(ns, "/") == 0 {
-		ns = ns[1:]
-	}
-	if ns[len(ns)-1] == '/' {
-		ns = ns[:len(ns)-1]
+	fc := getFirstChar(ns)
+	ns = strings.Trim(ns, fc)
+	return strings.Split(ns, fc)
+}
+
+// GetFirstChar returns the first character from the input string.
+func getFirstChar(s string) string {
+	firstChar := ""
+	for _, r := range s {
+		firstChar = fmt.Sprintf("%c", r)
+		break
 	}
-	return strings.Split(ns, "/")
+	return firstChar
 }

mgmt/rest/server_test.go

@@ -145,3 +145,44 @@ func TestRestAPIDefaultConfig(t *testing.T) {
 		})
 	})
 }
+
+func TestParseNamespace(t *testing.T) {
+	tcs := getNsTestCases()
+
+	Convey("Test parseNamespace", t, func() {
+		for _, c := range tcs {
+			Convey("Test parseNamespace "+c.input, func() {
+				So(c.output, ShouldResemble, parseNamespace(c.input))
+			})
+		}
+	})
+}
+
+type nsTestCase struct {
+	input  string
+	output []string
+}
+
+func getNsTestCases() []nsTestCase {
+	tcs := []nsTestCase{
+		{
+			input:  "小a小b小c",
+			output: []string{"a", "b", "c"}},
+		{
+			input:  "%a%b%c",
+			output: []string{"a", "b", "c"}},
+		{
+			input:  "-aヒ-b/-c|",
+			output: []string{"aヒ", "b/", "c|"}},
+		{
+			input:  ">a>b=>c=",
+			output: []string{"a", "b=", "c="}},
+		{
+			input:  ">a>b<>c<",
+			output: []string{"a", "b<", "c<"}},
+		{
+			input:  "㊽a㊽b%㊽c/|",
+			output: []string{"a", "b%", "c/|"}},
+	}
+	return tcs
+}