Skip to content

Commit a7f1457

Browse files
committed
✅ (repo delete) remove comprehensive assertions on check stderr output
1 parent 7083f6d commit a7f1457

1 file changed

Lines changed: 1 addition & 4 deletions

File tree

pkg/cmd/repo/delete/delete_test.go

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,6 @@ func TestNewCmdDelete(t *testing.T) {
4242
output: DeleteOptions{RepoArg: "OWNER/REPO"},
4343
wantErr: true,
4444
wantErrMsg: "--yes required when not running interactively",
45-
wantStderr: "Error: --yes required when not running interactively\n",
4645
},
4746
{
4847
name: "base repo resolution",
@@ -63,14 +62,12 @@ func TestNewCmdDelete(t *testing.T) {
6362
input: "--yes",
6463
wantErr: true,
6564
wantErrMsg: "cannot non-interactively delete current repository. Please specify a repository or run interactively",
66-
wantStderr: "Error: cannot non-interactively delete current repository. Please specify a repository or run interactively\n",
6765
},
6866
{
6967
name: "confirm flag error when no argument notty",
7068
input: "--confirm",
7169
wantErr: true,
7270
wantErrMsg: "cannot non-interactively delete current repository. Please specify a repository or run interactively",
73-
wantStderr: "Error: cannot non-interactively delete current repository. Please specify a repository or run interactively\n",
7471
},
7572
{
7673
name: "confirm flag also ignored when no argument tty",
@@ -103,13 +100,13 @@ func TestNewCmdDelete(t *testing.T) {
103100

104101
_, err = cmd.ExecuteC()
105102

106-
assert.Equal(t, tt.wantStderr, stdErr.String())
107103
if tt.wantErr {
108104
assert.Error(t, err)
109105
assert.Equal(t, tt.wantErrMsg, err.Error())
110106
return
111107
}
112108
assert.NoError(t, err)
109+
assert.Equal(t, tt.wantStderr, stdErr.String())
113110
assert.Equal(t, tt.output.RepoArg, gotOpts.RepoArg)
114111
})
115112
}

0 commit comments

Comments
 (0)