diff options
| author | Paul Buetow <paul@buetow.org> | 2025-09-07 14:29:35 +0300 |
|---|---|---|
| committer | Paul Buetow <paul@buetow.org> | 2025-09-07 14:29:35 +0300 |
| commit | 23482b5d8da5c67da1fc501ddbafdd123be3972c (patch) | |
| tree | 452dc7c418055ebb79a88a303e50d2dbc1877f09 /internal/hexaiaction/cmdentry_runcommand_test.go | |
| parent | 76f388f9759cdc15cb1eba985cd87cde1906208b (diff) | |
feat: rename hexai-action -> hexai-tmux-action; remove --tmux/--no-tmux; tmux-only flow; update docs and Magefile
Diffstat (limited to 'internal/hexaiaction/cmdentry_runcommand_test.go')
| -rw-r--r-- | internal/hexaiaction/cmdentry_runcommand_test.go | 22 |
1 files changed, 4 insertions, 18 deletions
diff --git a/internal/hexaiaction/cmdentry_runcommand_test.go b/internal/hexaiaction/cmdentry_runcommand_test.go index 7c8aa5c..092e43b 100644 --- a/internal/hexaiaction/cmdentry_runcommand_test.go +++ b/internal/hexaiaction/cmdentry_runcommand_test.go @@ -6,7 +6,6 @@ import ( "io" "os" "path/filepath" - "strings" "testing" "codeberg.org/snonux/hexai/internal/tmux" @@ -30,12 +29,10 @@ func TestRunCommand_UIChild(t *testing.T) { func TestRunCommand_Tmux(t *testing.T) { oldTTY := isTTYFn - oldAvail := tmuxAvailableFn oldExec := osExecutableFn oldSplit := splitRunFn isTTYFn = func(_ uintptr) bool { return false } - tmuxAvailableFn = func() bool { return true } - osExecutableFn = func() (string, error) { return "/bin/hexai-action", nil } + osExecutableFn = func() (string, error) { return "/bin/hexai-tmux-action", nil } splitRunFn = func(_ tmux.SplitOpts, argv []string) error { for i := 0; i < len(argv)-1; i++ { if argv[i] == "-outfile" && i+1 < len(argv) { @@ -45,9 +42,9 @@ func TestRunCommand_Tmux(t *testing.T) { } return nil } - defer func(){ isTTYFn = oldTTY; tmuxAvailableFn = oldAvail; osExecutableFn = oldExec; splitRunFn = oldSplit }() + defer func(){ isTTYFn = oldTTY; osExecutableFn = oldExec; splitRunFn = oldSplit }() var out bytes.Buffer - if err := RunCommand(context.Background(), Options{ForceTmux: true}, bytes.NewBufferString("X"), &out, io.Discard); err != nil { + if err := RunCommand(context.Background(), Options{}, bytes.NewBufferString("X"), &out, io.Discard); err != nil { t.Fatalf("RunCommand tmux: %v", err) } if out.String() != "OUT" { t.Fatalf("stdout: %q", out.String()) } @@ -56,15 +53,4 @@ func TestRunCommand_Tmux(t *testing.T) { // Inline TTY path is exercised implicitly via other helpers; testing it directly // would require TTY simulation which is brittle in unit tests. -func TestRunCommand_FallbackEcho(t *testing.T) { - oldTTY := isTTYFn - oldAvail := tmuxAvailableFn - isTTYFn = func(_ uintptr) bool { return false } - tmuxAvailableFn = func() bool { return false } - defer func(){ isTTYFn = oldTTY; tmuxAvailableFn = oldAvail }() - var out bytes.Buffer - if err := RunCommand(context.Background(), Options{NoTmux: true}, bytes.NewBufferString("Z"), &out, io.Discard); err != nil { - t.Fatalf("RunCommand fallback: %v", err) - } - if strings.TrimSpace(out.String()) != "Z" { t.Fatalf("stdout: %q", out.String()) } -} +// Fallback echo path removed in tmux-only flow. |
