From abd0a617e2485456814e8ebfdbbb899aaf5ee0c9 Mon Sep 17 00:00:00 2001 From: MDietermann Date: Tue, 14 Apr 2026 22:00:42 +0200 Subject: [PATCH] feat(yazi): implement PlanForce to reinstall all packages --- internal/modules/yazi/yazi.go | 11 +++++++++-- internal/modules/yazi/yazi_test.go | 26 ++++++++++++++++++++++++++ 2 files changed, 35 insertions(+), 2 deletions(-) diff --git a/internal/modules/yazi/yazi.go b/internal/modules/yazi/yazi.go index f573b51..eb03db7 100644 --- a/internal/modules/yazi/yazi.go +++ b/internal/modules/yazi/yazi.go @@ -3,7 +3,6 @@ package yazi 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/yazi/yazi_test.go b/internal/modules/yazi/yazi_test.go index 0959d12..d851cd9 100644 --- a/internal/modules/yazi/yazi_test.go +++ b/internal/modules/yazi/yazi_test.go @@ -110,4 +110,30 @@ func TestVerifyAfterApply(t *testing.T) { } } +func TestPlanForceReinstallsAllPackages(t *testing.T) { + r := &fakeRunner{installed: map[string]bool{"yazi": true}} + m := New([]string{"yazi"}, 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 call; calls=%v", r.runCalls) + } +} + var _ module.Module = Module{}