separate exec plugin Action from Spec

Following commits will add the ability to reference an exec plugin
Action from a nested Spec `on.fail` field, so we simply separate out the
Action bit from the Spec in the exec plugin here.

Signed-off-by: Jay Pipes <[email protected]>

Author: jaypipes

plugin/exec/action.go

@@ -0,0 +1,21 @@
+// Use and distribution licensed under the Apache license version 2.
+//
+// See the COPYING file in the root project directory for full text.
+
+package exec
+
+// Action describes a single execution of one or more commands via the
+// operating system's `exec` family of functions.
+type Action struct {
+	// Exec is the exact command to execute.
+	//
+	// You may execute more than one command but must include the `shell` field
+	// to indicate that the command should be run in a shell. It is best
+	// practice, however, to simply use multiple `exec` specs instead of
+	// executing multiple commands in a single shell call.
+	Exec string `yaml:"exec"`
+	// Shell is the specific shell to use in executing the command. If empty
+	// (the default), no shell is used to execute the command and instead the
+	// operating system's `exec` family of calls is used.
+	Shell string `yaml:"shell,omitempty"`
+}

plugin/exec/parse_test.go

@@ -57,7 +57,9 @@ func TestSimpleCommand(t *testing.T) {
 				Index:    0,
 				Defaults: &gdttypes.Defaults{},
 			},
-			Exec: "ls",
+			Action: gdtexec.Action{
+				Exec: "ls",
+			},
 		},
 	}
 	assert.Equal(expTests, s.Tests)

plugin/exec/spec.go

@@ -12,17 +12,7 @@ import (
 // operating system's `exec` family of functions.
 type Spec struct {
 	gdttypes.Spec
-	// Exec is the exact command to execute.
-	//
-	// You may execute more than one command but must include the `shell` field
-	// to indicate that the command should be run in a shell. It is best
-	// practice, however, to simply use multiple `exec` specs instead of
-	// executing multiple commands in a single shell call.
-	Exec string `yaml:"exec"`
-	// Shell is the specific shell to use in executing the command. If empty
-	// (the default), no shell is used to execute the command and instead the
-	// operating system's `exec` family of calls is used.
-	Shell string `yaml:"shell,omitempty"`
+	Action
 	// Assert is an object containing the conditions that the Spec will assert.
 	Assert *Expect `yaml:"assert,omitempty"`
 }