commands: pass raw command line down to template evaluation

Some commands need to invoke others and/or run shell commands. For this,
we need the raw command line as entered by the user. Pass it down the
call chain just before it is split to invoke the command Execute method.

Remove unit tests for the template expand() test which does have any
added value now that it is performed on a single string without any
quote juggling.

Update all code to handle a single string instead of a list of
arguments.

Introduce a new dependency on git.sr.ht/~rjarry/go-opt to deal with
shell splitting. This is in preparation for using opt.ArgsToStruct to
parse arguments for all aerc commands.

There should be no functional change after this patch.

Signed-off-by: Robin Jarry <robin@jarry.cc>
Reviewed-by: Koni Marti <koni.marti@gmail.com>
Tested-by: Moritz Poldrack <moritz@poldrack.dev>
Tested-by: Inwit <inwit@sindominio.net>

1 files changed, 0 insertions, 114 deletions

diff --git a/commands/commands_test.go b/commands/commands_test.go
deleted file mode 100644
index 7edd0228..00000000
--- a/commands/commands_test.go
+++ /dev/null
@@ -1,114 +0,0 @@
-package commands
-
-import (
-	"reflect"
-	"testing"
-	"time"
-
-	"git.sr.ht/~rjarry/aerc/models"
-	"github.com/emersion/go-message/mail"
-)
-
-func TestExecuteCommand_expand(t *testing.T) {
-	tests := []struct {
-		args []string
-		want []string
-	}{
-		{
-			args: []string{"prompt", "Really quit? ", "quit"},
-			want: []string{"prompt", "Really quit? ", "quit"},
-		},
-		{
-			args: []string{"{{", "print", "\"hello\"", "}}"},
-			want: []string{"hello"},
-		},
-		{
-			args: []string{"prompt", "Really quit  ? ", "  quit "},
-			want: []string{"prompt", "Really quit  ? ", "  quit "},
-		},
-		{
-			args: []string{
-				"prompt", "Really quit? ", "{{",
-				"print", "\"quit\"", "}}",
-			},
-			want: []string{"prompt", "Really quit? ", "quit"},
-		},
-		{
-			args: []string{
-				"prompt", "Really quit? ", "{{",
-				"if", "1", "}}", "quit", "{{end}}",
-			},
-			want: []string{"prompt", "Really quit? ", "quit"},
-		},
-	}
-
-	var data dummyData
-
-	for i, test := range tests {
-		got, err := expand(&data, test.args)
-		if err != nil {
-			t.Errorf("test %d failed with err: %v", i, err)
-		} else if !reflect.DeepEqual(got, test.want) {
-			t.Errorf("test %d failed: "+
-				"got: %v, but want: %v", i, got, test.want)
-		}
-	}
-}
-
-// only for validation
-type dummyData struct{}
-
-var (
-	addr1 = mail.Address{Name: "John Foo", Address: "foo@bar.org"}
-	addr2 = mail.Address{Name: "John Bar", Address: "bar@foo.org"}
-)
-
-func (d *dummyData) Account() string                           { return "work" }
-func (d *dummyData) Folder() string                            { return "INBOX" }
-func (d *dummyData) To() []*mail.Address                       { return []*mail.Address{&addr1} }
-func (d *dummyData) Cc() []*mail.Address                       { return nil }
-func (d *dummyData) Bcc() []*mail.Address                      { return nil }
-func (d *dummyData) From() []*mail.Address                     { return []*mail.Address{&addr2} }
-func (d *dummyData) Peer() []*mail.Address                     { return d.From() }
-func (d *dummyData) ReplyTo() []*mail.Address                  { return nil }
-func (d *dummyData) Date() time.Time                           { return time.Now() }
-func (d *dummyData) DateAutoFormat(time.Time) string           { return "" }
-func (d *dummyData) Header(string) string                      { return "" }
-func (d *dummyData) ThreadPrefix() string                      { return "└─>" }
-func (d *dummyData) ThreadCount() int                          { return 0 }
-func (d *dummyData) ThreadFolded() bool                        { return false }
-func (d *dummyData) ThreadContext() bool                       { return false }
-func (d *dummyData) Subject() string                           { return "Re: [PATCH] hey" }
-func (d *dummyData) SubjectBase() string                       { return "[PATCH] hey" }
-func (d *dummyData) Number() int                               { return 0 }
-func (d *dummyData) Labels() []string                          { return nil }
-func (d *dummyData) Flags() []string                           { return nil }
-func (d *dummyData) IsReplied() bool                           { return true }
-func (d *dummyData) HasAttachment() bool                       { return true }
-func (d *dummyData) Attach(string) string                      { return "" }
-func (d *dummyData) IsRecent() bool                            { return false }
-func (d *dummyData) IsUnread() bool                            { return false }
-func (d *dummyData) IsFlagged() bool                           { return false }
-func (d *dummyData) IsMarked() bool                            { return false }
-func (d *dummyData) MessageId() string                         { return "123456789@foo.org" }
-func (d *dummyData) Size() int                                 { return 420 }
-func (d *dummyData) OriginalText() string                      { return "Blah blah blah" }
-func (d *dummyData) OriginalDate() time.Time                   { return time.Now() }
-func (d *dummyData) OriginalFrom() []*mail.Address             { return d.From() }
-func (d *dummyData) OriginalMIMEType() string                  { return "text/plain" }
-func (d *dummyData) OriginalHeader(string) string              { return "" }
-func (d *dummyData) Recent(...string) int                      { return 1 }
-func (d *dummyData) Unread(...string) int                      { return 3 }
-func (d *dummyData) Exists(...string) int                      { return 14 }
-func (d *dummyData) RUE(...string) string                      { return "1/3/14" }
-func (d *dummyData) Connected() bool                           { return false }
-func (d *dummyData) ConnectionInfo() string                    { return "" }
-func (d *dummyData) ContentInfo() string                       { return "" }
-func (d *dummyData) StatusInfo() string                        { return "" }
-func (d *dummyData) TrayInfo() string                          { return "" }
-func (d *dummyData) PendingKeys() string                       { return "" }
-func (d *dummyData) Role() string                              { return "inbox" }
-func (d *dummyData) Style(string, string) string               { return "" }
-func (d *dummyData) StyleSwitch(string, ...models.Case) string { return "" }
-
-func (d *dummyData) StyleMap([]string, ...models.Case) []string { return []string{} }