test: Cleanup detritus, fix doc comments

Author: sdboyer

internal/test/integration/testcase.go

@@ -174,6 +174,8 @@ func (tc *TestCase) CompareError(err error, stderr string) {
 	}
 }
 
+// CompareCmdFailure checks to see if the failure/success (in the sense of an
+// exit code) was as expected by the test fixture.
 func (tc *TestCase) CompareCmdFailure(gotFail bool) {
 	if gotFail == tc.ShouldFail {
 		return

internal/test/integration/testproj.go

@@ -37,7 +37,6 @@ type TestProject struct {
 	origWd     string
 	stdout     bytes.Buffer
 	stderr     bytes.Buffer
-	failed     bool
 	run        RunFunc
 }
 
@@ -162,12 +161,6 @@ func (p *TestProject) GetStderr() string {
 	return p.stderr.String()
 }
 
-// Indicates if the command specified in the test run failed, whether or not
-// there was error output.
-func (p *TestProject) Failed() bool {
-	return p.failed
-}
-
 // GetVendorGit populates the initial vendor directory for a test project.
 func (p *TestProject) GetVendorGit(ip string) {
 	parse := strings.Split(ip, "/")