Merge pull request #18 from cisco-ie/test-fix

Test Fixes

Author: remingtonc

.gitignore

@@ -16,4 +16,13 @@
 # vendor/
 
 ## Custom
-.DS_Store
+.DS_Store
+
+## Test-related (should be fixed)
+dump.bin
+dump.txt
+dump1.txt
+dumpfiltererd.txt
+mdt_msg_samples/hexdump.bin
+models.txt
+pipeline.log

Makefile

@@ -4,7 +4,7 @@ BINARY=pipeline
 include skeleton/pipeline.mk
 
 # Setup pretest as a prerequisite of tests.
-test: pretest
-pretest:
-	@echo Setting up zookeeper, kafka. Docker required.
+testall: pretestinfra
+pretestinfra:
+	@echo Setting up Zookeeper and Kafka. Docker required.
 	tools/test/run.sh

skeleton/pipeline.mk

@@ -38,6 +38,10 @@ bin/$(BINARY): $(SOURCES)
 generated_source:
 	go generate -x
 
+.PHONY: testall
+testall:
+	go test -v -tags=integration -run=. -bench=. $(PROFILE)
+
 .PHONY: test
 test:
 	go test -v -run=. -bench=. $(PROFILE)

xport_kafka_test.go

@@ -5,16 +5,18 @@
 // All rights reserved.
 //
 //
+// +build integration
 
 package main
 
 import (
 	"encoding/json"
 	"fmt"
-	"github.com/dlintw/goconf"
 	"strings"
 	"testing"
 	"time"
+
+	"github.com/dlintw/goconf"
 )
 
 var kmod kafkaOutputModule