fix: tighten task 2 constructor contract

internal/cli/app.go

@@ -10,7 +10,7 @@ type App struct {
 	store secretstore.Store
 }
 
-func NewApp(_ ...any) *App {
+func NewApp() *App {
 	return &App{}
 }
 

internal/cli/app_test.go

@@ -5,6 +5,8 @@ import (
 	"testing"
 )
 
+var _ func() *App = NewApp
+
 func TestAppRunRejectsUnknownCommand(t *testing.T) {
 	app := NewApp()
 

internal/secretstore/store_test.go

@@ -3,9 +3,33 @@ package secretstore
 import "testing"
 
 func TestCredentialValidateRequiresAllFields(t *testing.T) {
-	cred := Credential{Host: "imap.example.com", Username: "alice"}
+	tests := []struct {
+		name string
+		cred Credential
+	}{
+		{
+			name: "missing host",
+			cred: Credential{Username: "alice", Password: "secret"},
+		},
+		{
+			name: "missing username",
+			cred: Credential{Host: "imap.example.com", Password: "secret"},
+		},
+		{
+			name: "missing password",
+			cred: Credential{Host: "imap.example.com", Username: "alice"},
+		},
+		{
+			name: "whitespace only fields",
+			cred: Credential{Host: " \t ", Username: "\n", Password: "   "},
+		},
+	}
 
-	if err := cred.Validate(); err == nil {
+	for _, tt := range tests {
-		t.Fatal("expected validation error when password is missing")
+		t.Run(tt.name, func(t *testing.T) {
+			if err := tt.cred.Validate(); err == nil {
+				t.Fatal("expected validation error when a required field is missing")
+			}
+		})
 	}
 }