From c75afade8ceffcb16dd026f43bcdf4a9e170d192 Mon Sep 17 00:00:00 2001 From: MDietermann Date: Tue, 14 Apr 2026 21:55:43 +0200 Subject: [PATCH] feat(gittools): implement PlanForce to reinstall all packages --- internal/modules/gittools/gittools.go | 11 +++++++-- internal/modules/gittools/gittools_test.go | 26 ++++++++++++++++++++++ 2 files changed, 35 insertions(+), 2 deletions(-) diff --git a/internal/modules/gittools/gittools.go b/internal/modules/gittools/gittools.go index d18f6f5..e226346 100644 --- a/internal/modules/gittools/gittools.go +++ b/internal/modules/gittools/gittools.go @@ -3,7 +3,6 @@ package gittools import ( "context" - "errors" "fmt" "strings" @@ -75,5 +74,13 @@ func (m Module) Verify(ctx context.Context) error { } func (m Module) PlanForce(ctx context.Context) (module.Plan, error) { - return module.Plan{}, errors.New("PlanForce not implemented") + if len(m.pkgs) == 0 { + return module.Plan{}, nil + } + pkgs := append([]string{}, m.pkgs...) + return module.Plan{Actions: []module.Action{{ + Description: "pacman -S (force) " + strings.Join(pkgs, ", "), + Kind: module.ActionPackageInstall, + Exec: func(ctx context.Context) error { return m.pm.Reinstall(ctx, pkgs) }, + }}}, nil } diff --git a/internal/modules/gittools/gittools_test.go b/internal/modules/gittools/gittools_test.go index c2adc5a..a004633 100644 --- a/internal/modules/gittools/gittools_test.go +++ b/internal/modules/gittools/gittools_test.go @@ -110,4 +110,30 @@ func TestVerifyAfterApply(t *testing.T) { } } +func TestPlanForceReinstallsAllPackages(t *testing.T) { + r := &fakeRunner{installed: map[string]bool{"lazygit": true}} + m := New([]string{"lazygit"}, platform.NewPacman(r)) + plan, err := m.PlanForce(context.Background()) + if err != nil { + t.Fatalf("PlanForce: %v", err) + } + if plan.IsEmpty() { + t.Fatal("force plan must not be empty even when packages are installed") + } + for _, a := range plan.Actions { + if err := a.Exec(context.Background()); err != nil { + t.Fatalf("Exec: %v", err) + } + } + sawReinstall := false + for _, c := range r.runCalls { + if len(c) >= 4 && c[0] == "sudo" && c[1] == "pacman" && c[2] == "-S" && c[3] == "--noconfirm" { + sawReinstall = true + } + } + if !sawReinstall { + t.Fatalf("no pacman reinstall (-S --noconfirm, no --needed) call; calls=%v", r.runCalls) + } +} + var _ module.Module = Module{}