From 449858158d1a17006300d74902c633e09f71861b Mon Sep 17 00:00:00 2001
From: ravocean <iamericanec@gmail.com>
Date: Mon, 7 Jun 2021 12:57:36 -0700
Subject: [PATCH] Continued to work on the output of test file

---
 pkg/cmd/browse/browse.go      |  2 +-
 pkg/cmd/browse/browse_test.go | 30 +++++++++++++++---------------
 2 files changed, 16 insertions(+), 16 deletions(-)

diff --git a/pkg/cmd/browse/browse.go b/pkg/cmd/browse/browse.go
index 442a4e8f1..b0279b7f4 100644
--- a/pkg/cmd/browse/browse.go
+++ b/pkg/cmd/browse/browse.go
@@ -112,7 +112,7 @@ func openInBrowser(cmd *cobra.Command, opts *BrowseOptions) error {
 	baseRepo, err := opts.BaseRepo()
 	httpClient, _ := opts.HttpClient()
 	apiClient := api.NewClientFromHTTP(httpClient)
-	branchName, err := api.RepoDefaultBranch(apiClient, baseRepo)
+	branchName, _ := api.RepoDefaultBranch(apiClient, baseRepo)
 
 	if !inRepo(err) {
 		return printExit(exitNotInRepo, cmd, opts, "")
diff --git a/pkg/cmd/browse/browse_test.go b/pkg/cmd/browse/browse_test.go
index 780d69e22..abb9975bd 100644
--- a/pkg/cmd/browse/browse_test.go
+++ b/pkg/cmd/browse/browse_test.go
@@ -2,7 +2,7 @@ package browse
 
 import (
 	"bytes"
-	"io/ioutil"
+	//"io/ioutil"
 	"net/http"
 	"testing"
 
@@ -53,8 +53,8 @@ func runCommand(rt http.RoundTripper, t testCase) (*test.CmdOut, error) {
 	cmd.SetArgs(argv)
 
 	cmd.SetIn(&bytes.Buffer{})
-	cmd.SetOut(ioutil.Discard)
-	cmd.SetErr(ioutil.Discard)
+	cmd.SetOut(stdout)
+	cmd.SetErr(stderr)
 
 	_, err = cmd.ExecuteC()
 	return &test.CmdOut{
@@ -76,16 +76,16 @@ func TestNewCmdBrowse(t *testing.T) {
 			stdoutExpected: "",
 			stderrExpected: "Error: accepts 1 flag, 2 flag(s) were recieved\nUse 'gh browse --help' for more information about browse\n\n",
 		},
-		{
-			name: "test2",
-			args: args{
-				repo: ghrepo.New("bchadwic", "cli"),
-				cli:  "--settings",
-			},
-			errorExpected:  false,
-			stdoutExpected: "hello world",
-			stderrExpected: "hello world",
-		},
+		// {
+		// 	name: "test2",
+		// 	args: args{
+		// 		repo: ghrepo.New("bchadwic", "cli"),
+		// 		cli:  "--settings",
+		// 	},
+		// 	errorExpected:  false,
+		// 	stdoutExpected: "hello world",
+		// 	stderrExpected: "hello world",
+		// },
 	}
 
 	for _, tt := range tests {
@@ -103,9 +103,9 @@ func TestNewCmdBrowse(t *testing.T) {
 				assert.Error(t, err)
 			}
 
-			assert.Contains(t, output.String(), tt.stdoutExpected) // success outputs
+			assert.Contains(t, output.OutBuf.String(), tt.stdoutExpected) // success outputs
 
-			assert.Contains(t, output.Stderr(), tt.stderrExpected) // error outputs
+			assert.Contains(t, output.ErrBuf.String(), tt.stderrExpected) // error outputs
 
 		})
 	}