add go vet tests

graphqlsch/graphqlsch.go

@@ -45,25 +45,23 @@ type Field struct {
 }
 
 func (f Field) ToString() string {
-	var fmtStr string
-	var typeFmtStr string
+	var typeStr string
 	if f.IsArray {
 		if f.IsElemNullable {
-			typeFmtStr = "%v"
+			typeStr = fmt.Sprintf("%v", f.Type)
 		} else {
-			typeFmtStr = "%v!"
+			typeStr = fmt.Sprintf("%v!", f.Type)
 		}
 		for level := uint(0); level < f.ArrayDim; level++ {
-			typeFmtStr = fmt.Sprintf("[%v]", typeFmtStr)
+			typeStr = fmt.Sprintf("[%v]", typeStr)
 		}
 	} else {
-		typeFmtStr = "%v"
+		typeStr = fmt.Sprintf("%v", f.Type)
 	}
 	if !f.IsNullable {
-		typeFmtStr = fmt.Sprintf("%v!", typeFmtStr)
+		typeStr = fmt.Sprintf("%v!", typeStr)
 	}
-	fmtStr = fmt.Sprintf("    %%v: %v", typeFmtStr)
-	return fmt.Sprintf(fmtStr, f.Name, f.Type)
+	return fmt.Sprintf("    %v: %v", f.Name, typeStr)
 }
 
 type PrimitiveType string

test.sh

@@ -17,6 +17,15 @@ if [ ! -z "$gofmt_output" ]; then
 fi
 printf "OK\n"
 
+printf "Running go vet tests..."
+go_vet_output="$(go vet -v ./... 2>&1 | grep ':' )"
+# grep ':' is a hack-ish way to only match path/to/file.go:lineno:colno: vet msg
+if [ ! -z "$go_vet_output" ]; then
+    printf "FAIL. output:\n$go_vet_output" 1>&2
+    exit 1
+fi
+printf "OK\n"
+
 printf "Running 'go install'..."
 go_install_output=`go install 2>&1`
 if [ ! -z "$go_install_output" ]; then