Kharec/goyco
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat: update tests to verify clamping

Browse Source
This commit is contained in:
Sandro CURY CAZZANIGA
2025-11-21 15:21:05 +01:00
parent 73083e4188
commit 14ae6f815b
1 changed files with 14 additions and 26 deletions
Download Patch File Download Diff File Expand all files Collapse all files

40
cmd/goyco/commands/seed_test.go
View File

@@ -179,47 +179,35 @@ func TestSeedDatabaseFlagParsing(t *testing.T) {
} }
}) })
t.Run("negative users value", func(t *testing.T) { t.Run("negative users value is clamped", func(t *testing.T) {
err := seedDatabase(userRepo, postRepo, voteRepo, []string{"--users", "-1"}) err := seedDatabase(userRepo, postRepo, voteRepo, []string{"--users", "-1", "--posts", "1"})
if err == nil { if err != nil {
t.Error("expected error for negative users value") t.Errorf("negative users should be clamped, not rejected. Got error: %v", err)
}
if err != nil && err.Error() != "invalid value for --users: -1 (must be >= 0)" {
t.Errorf("expected specific error message, got: %v", err)
} }
}) })
t.Run("negative posts value", func(t *testing.T) { t.Run("negative posts value is clamped", func(t *testing.T) {
err := seedDatabase(userRepo, postRepo, voteRepo, []string{"--posts", "-5"}) err := seedDatabase(userRepo, postRepo, voteRepo, []string{"--posts", "-5"})
if err == nil { if err != nil {
t.Error("expected error for negative posts value") t.Errorf("negative posts should be clamped, not rejected. Got error: %v", err)
}
if err != nil && err.Error() != "invalid value for --posts: -5 (must be > 0)" {
t.Errorf("expected specific error message, got: %v", err)
} }
}) })
t.Run("zero posts value", func(t *testing.T) { t.Run("zero posts value is clamped", func(t *testing.T) {
err := seedDatabase(userRepo, postRepo, voteRepo, []string{"--posts", "0"}) err := seedDatabase(userRepo, postRepo, voteRepo, []string{"--posts", "0"})
if err == nil { if err != nil {
t.Error("expected error for zero posts value") t.Errorf("zero posts should be clamped, not rejected. Got error: %v", err)
}
if err != nil && err.Error() != "invalid value for --posts: 0 (must be > 0)" {
t.Errorf("expected specific error message, got: %v", err)
} }
}) })
t.Run("negative votes-per-post value", func(t *testing.T) { t.Run("negative votes-per-post value is clamped", func(t *testing.T) {
err := seedDatabase(userRepo, postRepo, voteRepo, []string{"--votes-per-post", "-10"}) err := seedDatabase(userRepo, postRepo, voteRepo, []string{"--votes-per-post", "-10", "--posts", "1"})
if err == nil { if err != nil {
t.Error("expected error for negative votes-per-post value") t.Errorf("negative votes-per-post should be clamped, not rejected. Got error: %v", err)
}
if err != nil && err.Error() != "invalid value for --votes-per-post: -10 (must be >= 0)" {
t.Errorf("expected specific error message, got: %v", err)
} }
}) })