Skip to content

Commit 5b6ff3e

Browse files
mmartinvclaude
andcommitted
fix: reset init-admin subcommand flags in test helper
The resetState() test helper re-initializes all CLI commands between tests. Without resetting the init-admin subcommand flags, pflag panics on flag redefinition when *CmdInit() is called a second time. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 6ad29ec commit 5b6ff3e

2 files changed

Lines changed: 5 additions & 1 deletion

File tree

cmd/config_test.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,14 +38,17 @@ func resetState(t *testing.T) {
3838
manufacturingCmd.ResetFlags()
3939
manufacturingCmd.ResetCommands()
4040
manufacturingCmd.SetArgs(nil)
41+
manufacturingInitAdminCmd.ResetFlags()
4142

4243
ownerCmd.ResetFlags()
4344
ownerCmd.ResetCommands()
4445
ownerCmd.SetArgs(nil)
46+
ownerInitAdminCmd.ResetFlags()
4547

4648
rendezvousCmd.ResetFlags()
4749
rendezvousCmd.ResetCommands()
4850
rendezvousCmd.SetArgs(nil)
51+
rendezvousInitAdminCmd.ResetFlags()
4952

5053
rootCmdInit()
5154
ownerCmdInit()

internal/state/seed.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,8 @@ import (
1414
"gorm.io/gorm"
1515
)
1616

17-
// SeedAdminConfig holds admin user configuration for database seeding.
17+
// SeedAdminConfig mirrors config.SeedAdminConfig to avoid an import cycle
18+
// (state → config → serviceinfo → state). Keep fields in sync.
1819
type SeedAdminConfig struct {
1920
Name string
2021
Email string

0 commit comments

Comments
 (0)