THRIFT-2445 (code generation for go maps with binary keys) should be tested

Patch: Aleksey Pesternikov

diff --git a/lib/go/test/BinaryKeyTest.thrift b/lib/go/test/BinaryKeyTest.thrift
new file mode 100644 (file)
index 0000000..71cb614
--- /dev/null
+++ b/lib/go/test/BinaryKeyTest.thrift
@@ -0,0 +1,24 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership. The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License. You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied. See the License for the
+# specific language governing permissions and limitations
+# under the License.
+#
+
+# Make sure that thrift produce compilable code for binary key
+struct testStruct {
+  1: required map<binary,string> bin_to_string
+}
+

diff --git a/lib/go/test/Makefile.am b/lib/go/test/Makefile.am
index 4e71cd0..cb6073c 100644 (file)
--- a/lib/go/test/Makefile.am
+++ b/lib/go/test/Makefile.am
@@ -17,25 +17,25 @@
 # under the License.
 #
 
-THRIFT = $(top_srcdir)/compiler/cpp/thrift
+THRIFT = $(top_srcdir)/compiler/cpp/thrift -out gopath/src/ --gen go:thrift_import=thrift
 THRIFTTEST = $(top_srcdir)/test/ThriftTest.thrift
 
 # Thrift for GO has problems with complex map keys: THRIFT-2063
-gopath: $(THRIFT) $(THRIFTTEST) IncludesTest.thrift NamespacedTest.thrift
+gopath: $(top_srcdir)/compiler/cpp/thrift $(THRIFTTEST) IncludesTest.thrift NamespacedTest.thrift
        mkdir -p gopath/src
        grep -v list.*map.*list.*map $(THRIFTTEST) > ThriftTest.thrift
-       $(THRIFT) --gen go:thrift_import=thrift -r IncludesTest.thrift
-       ln -nfs ../../gen-go/ThriftTest gopath/src/ThriftTest
-       ln -nfs ../../gen-go/IncludesTest gopath/src/IncludesTest
-       ln -nfs ../../gen-go/lib gopath/src/lib
+       $(THRIFT) -r IncludesTest.thrift
+       $(THRIFT) BinaryKeyTest.thrift
        ln -nfs ../../../thrift gopath/src/thrift
+       ln -nfs ../../tests gopath/src/tests
        touch gopath
 
 check: gopath
-       GOPATH=`pwd`/gopath $(GO) build IncludesTest
+       GOPATH=`pwd`/gopath $(GO) build IncludesTest BinaryKeyTest
+       GOPATH=`pwd`/gopath $(GO) test thrift tests
 
 clean-local:
-       $(RM) -r gen-go gopath ThriftTest.thrift
+       $(RM) -r gopath ThriftTest.thrift
 
 client: stubs
        $(GO) run TestClient.go

diff --git a/lib/go/test/tests/binary_key_test.go b/lib/go/test/tests/binary_key_test.go
new file mode 100644 (file)
index 0000000..271b244
--- /dev/null
+++ b/lib/go/test/tests/binary_key_test.go
@@ -0,0 +1,12 @@
+package tests
+
+import (
+       "BinaryKeyTest"
+       "testing"
+)
+
+func TestBinaryMapKeyGeneratesString(t *testing.T) {
+       s := BinaryKeyTest.NewTestStruct()
+       //This will only compile if BinToString has type of map[string]string
+       s.BinToString = make(map[string]string)
+}