Merge pull request #272 from silasdavis/no-test-wrapper-for-unit-tests

Might be running the integration test wrapper when we didn't intend to and multiple times

circle.yml

@@ -42,9 +42,9 @@ test:
   override:
     # We only wish to test our packages not vendored ones
     - echo "Running unit tests..."
-    - cd $GOPATH_REPO && glide novendor | xargs go test -v
+    - cd $GOPATH_REPO && go test -v `glide novendor`
     - echo "Running integration tests..."
-    - cd $GOPATH_REPO && glide novendor | xargs go test -v -tags integration
+    - cd $GOPATH_REPO && go test -v -tags integration `glide novendor`
 
 
 deployment:

rpc/tendermint/test/common.go

+// +build integration
+
+// Space above here matters
 package test
 
 import (
 	"github.com/eris-ltd/eris-db/test/fixtures"
 	rpc_core "github.com/eris-ltd/eris-db/rpc/tendermint/core"
 	"testing"
+	"fmt"
 )
 
-// Needs to be in a _test.go file to be picked up
+// Needs to be referenced by a *_test.go file to be picked up
 func TestWrapper(runner func() int) int {
+	fmt.Println("Running with integration TestWrapper (rpc/tendermint/test/common.go)...")
 	ffs := fixtures.NewFileFixtures("Eris-DB")
 
 	defer ffs.RemoveAll()

rpc/tendermint/test/runner/main.go

+// +build integration
+
+// Space above here matters
 package main
 
 import (