test(gmail): satisfy inactive-guard lint

steipete · veteranbv · steipete · commit 3b7e8a11eead · 2026-07-13T00:03:27.000-07:00
Co-authored-by: Henry Sowell &lt;henrysowell@gmail.com&gt;
diff --git a/internal/cmd/api_test.go b/internal/cmd/api_test.go
@@ -162,7 +162,7 @@ func TestDiscoveryGmailSendWithInactiveGuardsSkipsAccountResolution(t *testing.T
 	flags := &RootFlags{authOperations: app.AuthOperations{
 		OpenSecretsStore: func() (secrets.Store, error) {
 			t.Fatal("inactive no-send entries must not trigger account resolution")
-			return nil, nil
+			return nil, errors.New("unexpected account resolution")
 		},
 	}}
 
diff --git a/internal/cmd/root_more_test.go b/internal/cmd/root_more_test.go
@@ -475,7 +475,7 @@ func TestGmailSendDryRunWithInactiveGuardsSkipsAccountResolution(t *testing.T) {
 	runtime := &app.Runtime{Config: store}
 	runtime.Auth.OpenSecretsStore = func() (secrets.Store, error) {
 		t.Fatal("inactive no-send entries must not trigger account resolution")
-		return nil, nil
+		return nil, errors.New("unexpected account resolution")
 	}
 	args := []string{"gmail", "send", "--to", "a@example.com", "--subject", "S", "--body", "B", "--dry-run"}
 	result := executeWithTestRuntime(t, args, runtime)

Original file line number	Diff line number	Diff line change
`@@ -475,7 +475,7 @@ func TestGmailSendDryRunWithInactiveGuardsSkipsAccountResolution(t *testing.T) {`
`475`	`475`	`runtime := &app.Runtime{Config: store}`
`476`	`476`	`runtime.Auth.OpenSecretsStore = func() (secrets.Store, error) {`
`477`	`477`	`t.Fatal("inactive no-send entries must not trigger account resolution")`
`478`		`- return nil, nil`
	`478`	`+ return nil, errors.New("unexpected account resolution")`
`479`	`479`	`}`
`480`	`480`	`args := []string{"gmail", "send", "--to", "[email protected]", "--subject", "S", "--body", "B", "--dry-run"}`
`481`	`481`	`result := executeWithTestRuntime(t, args, runtime)`