Merge pull request #47 from nginxinc/ps-dev-split-clients

Update to handle split_clients

Author: puneetsarna

README.md

@@ -75,7 +75,7 @@ func main() {
 
 ## Contributing
 
-If you'd like to contribute to the project, please read our [Contributing guide](docs/CONTRIBUTING.md).
+If you'd like to contribute to the project, please read our [Contributing guide](CONTRIBUTING.md).
 
 ## License
 

analyze_map.go

@@ -35,6 +35,9 @@ var mapBodies = map[string]mapParameterMasks{
 	"types": {
 		defaultMasks: ngxConf1More,
 	},
+	"split_clients": {
+		defaultMasks: ngxConfTake1,
+	},
 }
 
 // analyzeMapBody validates the body of a map-like directive. Map-like directives are block directives

analyze_map_test.go

@@ -144,6 +144,27 @@ func TestAnalyzeMapBody(t *testing.T) {
 			term:    ";",
 			wantErr: &ParseError{What: "invalid number of parameters"},
 		},
+		"valid split_clients": {
+			mapDirective: "split_clients",
+			parameter: &Directive{
+				Directive: "0.5%",
+				Args:      []string{"google.com"},
+				Line:      5,
+				Block:     Directives{},
+			},
+			term: ";",
+		},
+		"invalid split_clients": {
+			mapDirective: "split_clients",
+			parameter: &Directive{
+				Directive: "0.5%",
+				Args:      []string{"google.com", "testme"},
+				Line:      5,
+				Block:     Directives{},
+			},
+			term:    ";",
+			wantErr: &ParseError{What: "invalid number of parameters"},
+		},
 		"missing semicolon": {
 			mapDirective: "map",
 			parameter: &Directive{