Merge pull request securego#162 from gcmurphy/bugfix

Cleanup vet failures and unused files
ccojocar · Jan 29, 2018 · 8125622 · 8125622
2 parents 07a2eec + a97a196
commit 8125622
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 32 deletions.
diff --git a/analyzer_test.go b/analyzer_test.go
@@ -1,7 +1,6 @@
 package gas_test
 
 import (
-	"bytes"
 	"io/ioutil"
 	"log"
 	"os"
@@ -20,10 +19,9 @@ var _ = Describe("Analyzer", func() {
 	var (
 		analyzer *gas.Analyzer
 		logger   *log.Logger
-		output   *bytes.Buffer
 	)
 	BeforeEach(func() {
-		logger, output = testutils.NewLogger()
+		logger, _ = testutils.NewLogger()
 		analyzer = gas.NewAnalyzer(nil, logger)
 	})
 

diff --git a/glide.lock b/glide.lock
diff --git a/import_tracker_test.go b/import_tracker_test.go
@@ -2,7 +2,7 @@ package gas_test
 
 import (
 	. "github.com/onsi/ginkgo"
-	//. "github.com/onsi/gomega"
+	. "github.com/onsi/gomega"
 )
 
 var _ = Describe("ImportTracker", func() {
@@ -15,6 +15,7 @@ var _ = Describe("ImportTracker", func() {
 	})
 	Context("when I have a valid go package", func() {
 		It("should record all import specs", func() {
+            Expect(source).To(Equal(source))
 			Skip("Not implemented")
 		})
 

diff --git a/rules/rules_test.go b/rules/rules_test.go
@@ -1,7 +1,6 @@
 package rules_test
 
 import (
-	"bytes"
 	"fmt"
 	"log"
 
@@ -17,14 +16,13 @@ var _ = Describe("gas rules", func() {
 
 	var (
 		logger   *log.Logger
-		output   *bytes.Buffer
 		config   gas.Config
 		analyzer *gas.Analyzer
 		runner   func(string, []testutils.CodeSample)
 	)
 
 	BeforeEach(func() {
-		logger, output = testutils.NewLogger()
+		logger, _ = testutils.NewLogger()
 		config = gas.NewConfig()
 		analyzer = gas.NewAnalyzer(config, logger)
 		runner = func(rule string, samples []testutils.CodeSample) {