From 6973f009e9d5c31d39f9c6dc3ff282f9c31730bb Mon Sep 17 00:00:00 2001 From: Pior Bastida Date: Fri, 11 Jan 2019 22:54:48 -0500 Subject: [PATCH] TaskAction public API (#263) --- pkg/tasks/task.go | 4 ++-- pkg/tasks/task_action.go | 4 ++-- pkg/tasks/task_runner_test.go | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/pkg/tasks/task.go b/pkg/tasks/task.go index 6ea8deed..f2aff050 100644 --- a/pkg/tasks/task.go +++ b/pkg/tasks/task.go @@ -10,7 +10,7 @@ import ( type Task struct { *TaskDefinition header string - actions []taskAction + actions []TaskAction feature features.FeatureInfo } @@ -22,7 +22,7 @@ func (t *Task) SetFeature(name, param string) { t.feature = features.NewFeatureInfo(name, param) } -func (t *Task) AddAction(action taskAction) { +func (t *Task) AddAction(action TaskAction) { t.actions = append(t.actions, action) } diff --git a/pkg/tasks/task_action.go b/pkg/tasks/task_action.go index 56e6162a..dc1a502c 100644 --- a/pkg/tasks/task_action.go +++ b/pkg/tasks/task_action.go @@ -2,7 +2,7 @@ package tasks import "fmt" -type taskAction interface { +type TaskAction interface { Description() string Needed(*Context) *ActionResult Run(*Context) error @@ -26,7 +26,7 @@ func actionNotNeeded() *ActionResult { return &ActionResult{Needed: false} } -func runAction(ctx *Context, action taskAction) error { +func runAction(ctx *Context, action TaskAction) error { desc := action.Description() result := action.Needed(ctx) diff --git a/pkg/tasks/task_runner_test.go b/pkg/tasks/task_runner_test.go index f1077412..d0811195 100644 --- a/pkg/tasks/task_runner_test.go +++ b/pkg/tasks/task_runner_test.go @@ -135,7 +135,7 @@ func TestTaskRunner(t *testing.T) { task := &Task{ TaskDefinition: &TaskDefinition{Name: "testtask"}, - actions: []taskAction{action1, action2}, + actions: []TaskAction{action1, action2}, } taskRunner := &TaskRunnerImpl{} @@ -158,7 +158,7 @@ func TestTaskRunnerWithError(t *testing.T) { task := &Task{ TaskDefinition: &TaskDefinition{Name: "testtask"}, - actions: []taskAction{action1, action2}, + actions: []TaskAction{action1, action2}, } taskRunner := &TaskRunnerImpl{}