summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
-rw-r--r--config/setup/websocket.go4
-rw-r--r--config/setup/websocket_test.go49
2 files changed, 51 insertions, 2 deletions
diff --git a/config/setup/websocket.go b/config/setup/websocket.go
index 7628305b4..9178bd139 100644
--- a/config/setup/websocket.go
+++ b/config/setup/websocket.go
@@ -8,7 +8,7 @@ import (
// WebSocket configures a new WebSockets middleware instance.
func WebSocket(c *Controller) (middleware.Middleware, error) {
- websocks, err := websocketParse(c)
+ websocks, err := webSocketParse(c)
if err != nil {
return nil, err
}
@@ -20,7 +20,7 @@ func WebSocket(c *Controller) (middleware.Middleware, error) {
}, nil
}
-func websocketParse(c *Controller) ([]websockets.Config, error) {
+func webSocketParse(c *Controller) ([]websockets.Config, error) {
var websocks []websockets.Config
var respawn bool
diff --git a/config/setup/websocket_test.go b/config/setup/websocket_test.go
index 07a8e2cf3..86af253d0 100644
--- a/config/setup/websocket_test.go
+++ b/config/setup/websocket_test.go
@@ -34,3 +34,52 @@ func TestWebSocket(t *testing.T) {
}
}
+func TestWebSocketParse(t *testing.T) {
+ tests := []struct {
+ inputWebSocketConfig string
+ shouldErr bool
+ expectedWebSocketConfig []websockets.Config
+ }{
+ {`websocket /api1 cat`, false, []websockets.Config{{
+ Path: "/api1",
+ Command: "cat",
+ }}},
+
+ {`websocket /api3 cat
+ websocket /api4 cat `, false, []websockets.Config{{
+ Path: "/api3",
+ Command: "cat",
+ }, {
+ Path: "/api4",
+ Command: "cat",
+ }}},
+ }
+ for i, test := range tests {
+ c := NewTestController(test.inputWebSocketConfig)
+ actualWebSocketConfigs, err := webSocketParse(c)
+
+ if err == nil && test.shouldErr {
+ t.Errorf("Test %d didn't error, but it should have", i)
+ } else if err != nil && !test.shouldErr {
+ t.Errorf("Test %d errored, but it shouldn't have; got '%v'", i, err)
+ }
+ if len(actualWebSocketConfigs) != len(test.expectedWebSocketConfig) {
+ t.Fatalf("Test %d expected %d no of WebSocket configs, but got %d ",
+ i, len(test.expectedWebSocketConfig), len(actualWebSocketConfigs))
+ }
+ for j, actualWebSocketConfig := range actualWebSocketConfigs {
+
+ if actualWebSocketConfig.Path != test.expectedWebSocketConfig[j].Path {
+ t.Errorf("Test %d expected %dth WebSocket Config Path to be %s , but got %s",
+ i, j, test.expectedWebSocketConfig[j].Path, actualWebSocketConfig.Path)
+ }
+
+ if actualWebSocketConfig.Command != test.expectedWebSocketConfig[j].Command {
+ t.Errorf("Test %d expected %dth WebSocket Config Command to be %s , but got %s",
+ i, j, test.expectedWebSocketConfig[j].Command, actualWebSocketConfig.Command)
+ }
+
+ }
+ }
+
+}