Add query formatter

Author: stilvoid

formatters/formatters.go

@@ -16,6 +16,7 @@ func init() {
 	Register("json", formatJSON)
 	Register("xml", formatXML)
 	Register("yaml", formatYAML)
+	Register("query", formatQuery)
 }
 
 // Names returns a sorted list of valid options for the "format" parameter of Format

formatters/query.go

@@ -0,0 +1,46 @@
+package formatters
+
+import (
+	"fmt"
+	"net/url"
+	"reflect"
+
+	"github.com/stilvoid/please/util"
+)
+
+// TODO: Tidy this up; it's horrid.
+func formatQuery(in interface{}) (string, error) {
+	in = util.ForceStringKeys(in)
+
+	inMap, ok := in.(map[string]interface{})
+
+	if !ok {
+		return "", fmt.Errorf("query formatter expects a map as input")
+	}
+
+	var output url.Values = make(map[string][]string)
+
+	for key, value := range inMap {
+		val := reflect.ValueOf(value)
+
+		switch val.Kind() {
+		case reflect.Map:
+			return "", fmt.Errorf("query formatter cannot deal with nested values")
+		case reflect.Array, reflect.Slice:
+			for i := 0; i < val.Len(); i++ {
+				iVal := val.Index(i)
+
+				switch iVal.Kind() {
+				case reflect.Map, reflect.Array, reflect.Slice:
+					return "", fmt.Errorf("query formatter cannot deal with nested values")
+				default:
+					output.Add(key, fmt.Sprint(iVal.Interface()))
+				}
+			}
+		default:
+			output.Add(key, fmt.Sprint(value))
+		}
+	}
+
+	return output.Encode(), nil
+}

formatters/query_test.go

@@ -0,0 +1,49 @@
+package formatters
+
+import (
+	"reflect"
+	"testing"
+)
+
+func TestQuery(t *testing.T) {
+	inputs := []interface{}{
+		map[string]string{
+			"foo": "bar",
+		},
+		map[string][]string{
+			"foo": {"bar", "baz"},
+		},
+		map[string]int{
+			"foo": 123,
+		},
+		map[int]int{
+			123: 456,
+		},
+		map[string]string{
+			"foo": "bar",
+			"baz": "quux",
+		},
+	}
+
+	expecteds := []string{
+		"foo=bar",
+		"foo=bar&foo=baz",
+		"foo=123",
+		"123=456",
+		"baz=quux&foo=bar",
+	}
+
+	for i, input := range inputs {
+		actual, err := formatQuery(input)
+
+		if err != nil {
+			t.Errorf("unexpected error: %v", err)
+		}
+
+		expected := expecteds[i]
+
+		if !reflect.DeepEqual(actual, expected) {
+			t.Errorf("unexpected '%v', want '%v'", actual, expected)
+		}
+	}
+}