fix travisci test errors

Author: Sazon, John Bryan J

cmd/gitlab_client_test.go

@@ -5,6 +5,7 @@ import (
 	"testing"
 
 	"github.com/stretchr/testify/assert"
+	gitlab "github.com/xanzy/go-gitlab"
 )
 
 // Get the correct test data from the users environment variables
@@ -80,7 +81,7 @@ func TestGitlabNewClient(t *testing.T) {
 				assert.Nil(t, err)
 			}
 			if !tc.negativeTest {
-				_, _, err = gitlabClient.Users.ListUsers(nil)
+				_, _, err = gitlabClient.Users.ListUsers(&gitlab.ListUsersOptions{})
 				assert.Nil(t, err)
 			}
 		})

cmd/helpers_for_test.go

@@ -136,7 +136,6 @@ func (execT *execTestCmdFlags) executeCommand() (string, testResult) {
 }
 
 // A helper to ignore os.Exit(1) errors when running a cobra Command
-// This was copied from https://github.com/spf13/cobra/blob/master/command_test.go
 func executeCommand(root *cobra.Command,
 	args ...string) (stdout string, err error) {
 	for i, arg := range args {
@@ -149,9 +148,7 @@ func executeCommand(root *cobra.Command,
 }
 
 // A helper to ignore os.Exit(1) errors when running a cobra Command
-// This was copied from https://github.com/spf13/cobra/blob/master/command_test.go
-func executeCommandC(root *cobra.Command,
-	args ...string) (stdout string, output string, err error) {
+func executeCommandC(root *cobra.Command, args ...string) (stdout string, output string, err error) {
 	buf := new(bytes.Buffer)
 	root.SetOutput(buf)
 	root.SetArgs(args)
@@ -168,7 +165,7 @@ func executeCommandC(root *cobra.Command,
 	go func() {
 		var buf bytes.Buffer
 		if _, err := io.Copy(&buf, r); err != nil {
-			tInfo(err)
+			panic(err)
 		}
 		outC <- buf.String()
 	}()