| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297 |
- package executor
- import (
- "testing"
- "github.com/stretchr/testify/assert"
- "github.com/OliveTin/OliveTin/internal/auth"
- authpublic "github.com/OliveTin/OliveTin/internal/auth/authpublic"
- config "github.com/OliveTin/OliveTin/internal/config"
- )
- func testingExecutor() (*Executor, *config.Config) {
- cfg := config.DefaultConfig()
- e := DefaultExecutor(cfg)
- a1 := &config.Action{
- Title: "Do some tickles",
- Shell: "echo 'Tickling {{ person }}'",
- Arguments: []config.ActionArgument{
- {
- Name: "person",
- Type: "ascii",
- },
- },
- }
- cfg.Actions = append(cfg.Actions, a1)
- cfg.Sanitize()
- return e, cfg
- }
- func TestCreateExecutorAndExec(t *testing.T) {
- e, cfg := testingExecutor()
- req := ExecutionRequest{
- AuthenticatedUser: &authpublic.AuthenticatedUser{Username: "Mr Tickle"},
- Cfg: cfg,
- Arguments: map[string]string{
- "person": "yourself",
- },
- }
- // Ensure bindings are available and set the binding to the only configured action
- e.RebuildActionMap()
- if len(cfg.Actions) > 0 {
- req.Binding = e.FindBindingWithNoEntity(cfg.Actions[0])
- }
- assert.NotNil(t, e, "Create an executor")
- wg, _ := e.ExecRequest(&req)
- wg.Wait()
- assert.Equal(t, int32(0), req.logEntry.ExitCode, "Exit code is zero")
- }
- func TestExecNonExistant(t *testing.T) {
- e, cfg := testingExecutor()
- req := ExecutionRequest{
- // Binding: e.FindBindingWithNoEntity("waffles"),
- logEntry: &InternalLogEntry{},
- Cfg: cfg,
- }
- wg, _ := e.ExecRequest(&req)
- wg.Wait()
- assert.Equal(t, int32(-1337), req.logEntry.ExitCode, "Log entry is set to an internal error code")
- assert.Equal(t, "💩", req.logEntry.ActionIcon, "Log entry icon is a poop (not found)")
- }
- func TestArgumentNameCamelCase(t *testing.T) {
- req := newExecRequest()
- req.Binding.Action = &config.Action{
- Title: "Do some tickles",
- Shell: "echo 'Tickling {{ personName }}'",
- Arguments: []config.ActionArgument{
- {
- Name: "personName",
- Type: "ascii",
- },
- },
- }
- req.Arguments = map[string]string{
- "personName": "Fred",
- }
- out, err := parseActionArguments(req)
- assert.Equal(t, "echo 'Tickling Fred'", out)
- assert.Nil(t, err)
- }
- func TestArgumentNameSnakeCase(t *testing.T) {
- req := newExecRequest()
- req.Binding.Action = &config.Action{
- Title: "Do some tickles",
- Shell: "echo 'Tickling {{ person_name }}'",
- Arguments: []config.ActionArgument{
- {
- Name: "person_name",
- Type: "ascii",
- },
- },
- }
- req.Arguments = map[string]string{
- "person_name": "Fred",
- }
- out, err := parseActionArguments(req)
- assert.Equal(t, "echo 'Tickling Fred'", out)
- assert.Nil(t, err)
- }
- func TestGetLogsEmpty(t *testing.T) {
- e, cfg := testingExecutor()
- assert.Equal(t, int64(10), cfg.LogHistoryPageSize, "Logs page size should be 10")
- logs, paging := e.GetLogTrackingIds(0, 10)
- assert.NotNil(t, logs, "Logs should not be nil")
- assert.Equal(t, 0, len(logs), "No logs yet")
- assert.Equal(t, int64(0), paging.CountRemaining, "There should be no remaining logs")
- }
- func TestGetLogsLessThanPageSize(t *testing.T) {
- e, cfg := testingExecutor()
- cfg.Actions = append(cfg.Actions, &config.Action{
- Title: "blat",
- Shell: "date",
- })
- cfg.Sanitize()
- // Rebuild action map to include newly added action
- e.RebuildActionMap()
- assert.Equal(t, int64(10), cfg.LogHistoryPageSize, "Logs page size should be 10")
- logEntries, paging := e.GetLogTrackingIds(0, 10)
- assert.Equal(t, 0, len(logEntries), "There should be 0 logs")
- assert.Zero(t, paging.CountRemaining, "There should be no remaining logs")
- execNewReqAndWait(e, "blat", cfg)
- execNewReqAndWait(e, "blat", cfg)
- execNewReqAndWait(e, "blat", cfg)
- execNewReqAndWait(e, "blat", cfg)
- execNewReqAndWait(e, "blat", cfg)
- execNewReqAndWait(e, "blat", cfg)
- execNewReqAndWait(e, "blat", cfg)
- logEntries, paging = e.GetLogTrackingIds(0, 10)
- assert.Equal(t, 7, len(logEntries), "There should be 7 logs")
- assert.Zero(t, paging.CountRemaining, "There should be no remaining logs")
- execNewReqAndWait(e, "blat", cfg)
- execNewReqAndWait(e, "blat", cfg)
- execNewReqAndWait(e, "blat", cfg)
- execNewReqAndWait(e, "blat", cfg)
- execNewReqAndWait(e, "blat", cfg)
- logEntries, paging = e.GetLogTrackingIds(0, 10)
- assert.Equal(t, 10, len(logEntries), "There should be 10 logs")
- assert.Equal(t, int64(2), paging.CountRemaining, "There should be 1 remaining logs")
- }
- func execNewReqAndWait(e *Executor, title string, cfg *config.Config) {
- req := &ExecutionRequest{
- // ActionTitle: title,
- Cfg: cfg,
- }
- // Ensure we have a binding for the requested title
- e.RebuildActionMap()
- var action *config.Action
- for _, a := range cfg.Actions {
- if a.Title == title {
- action = a
- break
- }
- }
- if action != nil {
- req.Binding = e.FindBindingWithNoEntity(action)
- }
- wg, _ := e.ExecRequest(req)
- wg.Wait()
- }
- func TestGetPagingIndexes(t *testing.T) {
- assert.Zero(t, getPagingStartIndex(5, 0), "Testing start index from empty list")
- assert.Equal(t, int64(4), getPagingStartIndex(5, 10), "Testing start index from mid point")
- assert.Equal(t, int64(9), getPagingStartIndex(-1, 10), "Testing start index with negative offset")
- assert.Equal(t, int64(0), getPagingStartIndex(15, 10), "Testing start index with large offset")
- assert.Equal(t, int64(9), getPagingStartIndex(0, 10), "Testing start index with zero count")
- }
- func TestUnsetRequiredArgument(t *testing.T) {
- req := newExecRequest()
- req.Binding.Action = &config.Action{
- Title: "Print your name",
- Shell: "echo 'Your name is: {{ name }}'",
- Arguments: []config.ActionArgument{
- {
- Name: "name",
- Type: "ascii",
- },
- },
- }
- req.Arguments = map[string]string{}
- out, err := parseActionArguments(req)
- assert.Equal(t, "", out)
- assert.NotNil(t, err)
- }
- func TestUnusedArgumentStillPassesTypeSafetyCheck(t *testing.T) {
- req := newExecRequest()
- req.Binding.Action = &config.Action{
- Title: "Print your name",
- Shell: "echo 'Your name is: {{ name }}'",
- Arguments: []config.ActionArgument{
- {
- Name: "name",
- Type: "ascii",
- },
- {
- Name: "age",
- Type: "int",
- },
- },
- }
- req.Arguments = map[string]string{
- "name": "Fred",
- "age": "Not an integer",
- }
- out, err := parseActionArguments(req)
- assert.Equal(t, "", out)
- assert.NotNil(t, err)
- }
- // https://github.com/OliveTin/OliveTin/issues/564
- func TestMangleInvalidArgumentValues(t *testing.T) {
- e, cfg := testingExecutor()
- a1 := &config.Action{
- Title: "Validate my date without seconds because I am from an Android phone",
- Shell: "echo 'The date is: {{ date }}'",
- Arguments: []config.ActionArgument{
- {
- Name: "date",
- Type: "datetime",
- },
- },
- }
- cfg.Actions = append(cfg.Actions, a1)
- cfg.Sanitize()
- // Build bindings for newly added action
- e.RebuildActionMap()
- req := ExecutionRequest{
- // Action: a1,
- AuthenticatedUser: auth.UserFromSystem(cfg, "testuser"),
- Cfg: cfg,
- Arguments: map[string]string{
- "date": "1990-01-10T12:00", // Invalid format, should be without seconds
- },
- }
- // Set binding to our appended action
- req.Binding = e.FindBindingWithNoEntity(a1)
- wg, _ := e.ExecRequest(&req)
- wg.Wait()
- assert.NotNil(t, req.logEntry, "Log entry should not be nil")
- assert.Equal(t, req.logEntry.Output, "The date is: 1990-01-10T12:00:00\n", "Date should be mangled to a valid format")
- }
|