Implement std.trim for string

builtins.go

@@ -1289,6 +1289,15 @@ func builtinIsEmpty(i *interpreter, strv value) (value, error) {
 	return makeValueBoolean(len(sStr) == 0), nil
 }
 
+func builtinTrim(i *interpreter, strv value) (value, error) {
+	str, err := i.getString(strv)
+	if err != nil {
+		return nil, err
+	}
+	sStr := str.getGoString()
+	return makeValueString(strings.TrimSpace(sStr)), nil
+}
+
 func base64DecodeGoBytes(i *interpreter, str string) ([]byte, error) {
 	strLen := len(str)
 	if strLen%4 != 0 {
@@ -2230,6 +2239,7 @@ var funcBuiltins = buildBuiltinMap([]builtin{
 	&ternaryBuiltin{name: "splitLimit", function: builtinSplitLimit, params: ast.Identifiers{"str", "c", "maxsplits"}},
 	&ternaryBuiltin{name: "strReplace", function: builtinStrReplace, params: ast.Identifiers{"str", "from", "to"}},
 	&unaryBuiltin{name: "isEmpty", function: builtinIsEmpty, params: ast.Identifiers{"str"}},
+	&unaryBuiltin{name: "trim", function: builtinTrim, params: ast.Identifiers{"str"}},
 	&unaryBuiltin{name: "base64Decode", function: builtinBase64Decode, params: ast.Identifiers{"str"}},
 	&unaryBuiltin{name: "base64DecodeBytes", function: builtinBase64DecodeBytes, params: ast.Identifiers{"str"}},
 	&unaryBuiltin{name: "parseInt", function: builtinParseInt, params: ast.Identifiers{"str"}},

linter/internal/types/stdlib.go

@@ -90,6 +90,7 @@ func prepareStdlib(g *typeGraph) {
 		"stringChars": g.newSimpleFuncType(stringType, "str"),
 		"format":      g.newSimpleFuncType(stringType, "str", "vals"),
 		"isEmpty":     g.newSimpleFuncType(boolType, "str"),
+		"trim":        g.newSimpleFuncType(stringType, "str"),
 		// TODO(sbarzowski) Fix when they match the documentation
 		"escapeStringBash":    g.newSimpleFuncType(stringType, "str_"),
 		"escapeStringDollars": g.newSimpleFuncType(stringType, "str_"),
@@ -176,8 +177,8 @@ func prepareStdlib(g *typeGraph) {
 
 		// Boolean
 
-		"xor":	g.newSimpleFuncType(boolType, "x", "y"),
-		"xnor":	g.newSimpleFuncType(boolType, "x", "y"),
+		"xor":  g.newSimpleFuncType(boolType, "x", "y"),
+		"xnor": g.newSimpleFuncType(boolType, "x", "y"),
 	}
 
 	fieldContains := map[string][]placeholderID{}

testdata/builtinTrim.golden

@@ -0,0 +1 @@
+"already trimmed string"

testdata/builtinTrim.jsonnet

@@ -0,0 +1 @@
+std.trim("already trimmed string")

testdata/builtinTrim1.golden

@@ -0,0 +1 @@
+"string with spaces on both ends"

testdata/builtinTrim1.jsonnet

@@ -0,0 +1 @@
+std.trim("    string with spaces on both ends     ")

testdata/builtinTrim2.golden

@@ -0,0 +1 @@
+"string with newline character at end"

testdata/builtinTrim2.jsonnet

@@ -0,0 +1 @@
+std.trim("string with newline character at end\n")

testdata/builtinTrim3.jsonnet

@@ -0,0 +1 @@
+std.trim("string with tabs at end\t\t")

testdata/builtinTrim4.golden

@@ -0,0 +1,9 @@
+RUNTIME ERROR: Unexpected type number, expected string
+-------------------------------------------------
+	testdata/builtinTrim4:1:1-13	$
+
+std.trim(10)
+
+-------------------------------------------------
+	During evaluation
+

testdata/builtinTrim4.jsonnet

@@ -0,0 +1 @@
+std.trim(10)