Add notion of integration vs unit tests

cisco-ie · May 22, 2019 · af33839 · af33839
1 parent 6761552
commit af33839
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 4 deletions.
diff --git a/Makefile b/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
diff --git a/skeleton/pipeline.mk b/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)

diff --git a/xport_kafka_test.go b/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