Add regression test for locale affecting output

See #722

Author: sbarzowski

.gitignore

@@ -19,6 +19,7 @@ libjsonnet++.so
 libjsonnet++.so.*
 libjsonnet_test_file
 libjsonnet_test_snippet
+libjsonnet_test_locale
 **/core.*
 **/vgcore
 **/vgcore.*

Makefile

@@ -133,7 +133,7 @@ install: bins libs
 
 all: $(ALL)
 
-test: jsonnet jsonnetfmt libjsonnet.so libjsonnet_test_snippet libjsonnet_test_file
+test: jsonnet jsonnetfmt libjsonnet.so libjsonnet_test_snippet libjsonnet_test_file libjsonnet_test_locale
 	./tests.sh
 
 reformat:
@@ -146,7 +146,8 @@ MAKEDEPEND_SRCS = \
 	cmd/jsonnet.cpp \
 	cmd/jsonnetfmt.cpp \
 	core/libjsonnet_test_snippet.c \
-	core/libjsonnet_test_file.c
+	core/libjsonnet_test_file.c \
+	core/libjsonnet_test_locale.cpp
 
 depend: core/std.jsonnet.h
 	rm -f Makefile.depend
@@ -196,6 +197,14 @@ LIBJSONNET_TEST_FILE_SRCS = \
 libjsonnet_test_file: $(LIBJSONNET_TEST_FILE_SRCS)
 	$(CC) $(CFLAGS) $(LDFLAGS) $< -L. -ljsonnet -o $@
 
+LIBJSONNET_TEST_LOCALE_SRCS = \
+	core/libjsonnet_test_locale.cpp \
+	libjsonnet++.so \
+	include/libjsonnet++.h
+
+libjsonnet_test_locale: $(LIBJSONNET_TEST_LOCALE_SRCS)
+	$(CXX) $(CXXFLAGS) $(LDFLAGS) $< -L. -ljsonnet++ -o $@
+
 # Encode standard library for embedding in C
 core/%.jsonnet.h: stdlib/%.jsonnet
 	(($(OD) -v -Anone -t u1 $< \

core/libjsonnet_test_locale.cpp

@@ -0,0 +1,24 @@
+#include <iostream>
+#include <locale>
+#include <cassert>
+#include <libjsonnet++.h>
+
+// Regression test for the follwing issue: https://github.com/google/jsonnet/issues/722
+
+int main() {
+    std::string templatedJSONString { "2000" };
+    std::locale glocale("en_US.UTF-8"); 
+    std::locale::global(glocale);
+
+    jsonnet::Jsonnet jsonnet {};
+    jsonnet.init();
+
+    std::string expanded;
+    if (!jsonnet.evaluateSnippet("", templatedJSONString, &expanded)) {
+        std::cerr << "Error parsing Jsonnet: "+jsonnet.lastError();
+        exit(1);
+    }
+    std::string expected = "2000\n";
+    assert(expected == expanded);
+    return 0;
+}

tests.sh

@@ -11,6 +11,8 @@ if [ -z "$DISABLE_LIB_TESTS" ]; then
     LD_LIBRARY_PATH=. ./libjsonnet_test_snippet "${TEST_SNIPPET}" || FAIL=TRUE
     printf 'libjsonnet_test_file: '
     LD_LIBRARY_PATH=. ./libjsonnet_test_file "test_suite/object.jsonnet" || FAIL=TRUE
+    echo -n 'libjsonnet_test_locale: '
+    LD_LIBRARY_PATH=. ./libjsonnet_test_locale || FAIL=TRUE
 fi
 examples/check.sh || FAIL=TRUE
 examples/terraform/check.sh || FAIL=TRUE