assert for runAdd err in test

skarim · skarim · commit 8da398357403 · 2026-05-23T20:47:27.000-04:00
diff --git a/cmd/add_test.go b/cmd/add_test.go
@@ -542,9 +542,10 @@ func TestAdd_RejectsExistingBranchInStack(t *testing.T) {
 	defer restore()
 
 	cfg, outR, errR := config.NewTestConfig()
-	runAdd(cfg, &addOptions{}, []string{"taken-branch"})
+	err := runAdd(cfg, &addOptions{}, []string{"taken-branch"})
 	output := collectOutput(cfg, outR, errR)
 
+	assert.ErrorIs(t, err, ErrInvalidArgs)
 	assert.Contains(t, output, "already exists in the stack")
 }
 

Original file line number	Diff line number	Diff line change
`@@ -542,9 +542,10 @@ func TestAdd_RejectsExistingBranchInStack(t *testing.T) {`
`542`	`542`	`defer restore()`
`543`	`543`
`544`	`544`	`cfg, outR, errR := config.NewTestConfig()`
`545`		`- runAdd(cfg, &addOptions{}, []string{"taken-branch"})`
	`545`	`+ err := runAdd(cfg, &addOptions{}, []string{"taken-branch"})`
`546`	`546`	`output := collectOutput(cfg, outR, errR)`
`547`	`547`
	`548`	`+ assert.ErrorIs(t, err, ErrInvalidArgs)`
`548`	`549`	`assert.Contains(t, output, "already exists in the stack")`
`549`	`550`	`}`
`550`	`551`