Merge branch 'main' of github.com:matryer/xbar into main

Author: matryer

pkg/plugins/action.go

@@ -42,7 +42,7 @@ func (i *Item) Action() ActionFunc {
 	if i.Params.Shell != "" {
 		actions = append(actions, actionShell(debugf, i, i.Params.Shell, i.Params.ShellParams))
 	}
-	if i.Params.Refresh == true {
+	if i.Params.Refresh {
 		shouldDelayBeforeRefresh := false
 		if len(actions) > 0 {
 			// there are actions other than refresh, so let's introduce a
@@ -157,6 +157,5 @@ func actionRefresh(debugf DebugFunc, refreshFunc func(ctx context.Context)) Acti
 	return func(ctx context.Context) {
 		debugf("action refresh")
 		refreshFunc(ctx)
-		return
 	}
 }

pkg/plugins/install_test.go

@@ -176,7 +176,6 @@ func TestGetInstalledPluginName(t *testing.T) {
 // 		is.True(err != nil)
 // 	})
 // }
-
-const simplePlugin = `#!/bin/bash
-echo "Hello, xbar."
-`
+// const simplePlugin = `#!/bin/bash
+// echo "Hello, xbar."
+// `

pkg/plugins/items.go renamed to pkg/plugins/item_params.go

@@ -39,7 +39,7 @@ func TestParseParams(t *testing.T) {
 	s, params, err = parseParams(`no params`)
 	is.NoErr(err)
 	is.Equal(s, "no params")
-	is.Equal(params.Terminal, false)  // Terminal
+	is.Equal(params.Terminal, "false")  // Terminal
 	is.Equal(params.Refresh, false)   // Refresh
 	is.Equal(params.Dropdown, true)   // Dropdown
 	is.Equal(params.Length, 0)        // Length