From bdb0c35ba3a2643806f62bd29df14fb8bccfe057 Mon Sep 17 00:00:00 2001
From: Casey Lee <cplee@nektos.com>
Date: Mon, 17 Feb 2020 10:30:52 -0800
Subject: [PATCH] code complexity of NewPlanExecutor

Signed-off-by: Casey Lee <cplee@nektos.com>
---
 act/model/planner.go | 14 ++++++++++++++
 act/runner/runner.go | 10 +---------
 2 files changed, 15 insertions(+), 9 deletions(-)

diff --git a/act/model/planner.go b/act/model/planner.go
index 35be015c..0519765a 100644
--- a/act/model/planner.go
+++ b/act/model/planner.go
@@ -137,6 +137,20 @@ func (wp *workflowPlanner) GetEvents() []string {
 	return events
 }
 
+// MaxRunNameLen determines the max name length of all jobs
+func (p *Plan) MaxRunNameLen() int {
+	maxRunNameLen := 0
+	for _, stage := range p.Stages {
+		for _, run := range stage.Runs {
+			runNameLen := len(run.String())
+			if runNameLen > maxRunNameLen {
+				maxRunNameLen = runNameLen
+			}
+		}
+	}
+	return maxRunNameLen
+}
+
 // GetJobIDs will get all the job names in the stage
 func (s *Stage) GetJobIDs() []string {
 	names := make([]string, 0)
diff --git a/act/runner/runner.go b/act/runner/runner.go
index 4230c291..dc774d4e 100644
--- a/act/runner/runner.go
+++ b/act/runner/runner.go
@@ -50,15 +50,7 @@ func New(runnerConfig *Config) (Runner, error) {
 }
 
 func (runner *runnerImpl) NewPlanExecutor(plan *model.Plan) common.Executor {
-	maxJobNameLen := 0
-	for _, stage := range plan.Stages {
-		for _, run := range stage.Runs {
-			jobNameLen := len(run.String())
-			if jobNameLen > maxJobNameLen {
-				maxJobNameLen = jobNameLen
-			}
-		}
-	}
+	maxJobNameLen := plan.MaxRunNameLen()
 
 	pipeline := make([]common.Executor, 0)
 	for _, stage := range plan.Stages {