refactor(worker): rename provider.setLogFile to provider.prepareLogFile

worker/cmd_provider.go

@@ -62,7 +62,7 @@ func (p *cmdProvider) Start() error {
 		env[k] = v
 	}
 	p.cmd = newCmdJob(p.command, p.WorkingDir(), env)
-	if err := p.setLogFile(); err != nil {
+	if err := p.prepareLogFile(); err != nil {
 		return err
 	}
 

worker/config.go

@@ -15,21 +15,6 @@ const (
 	ProvCommand
 )
 
-func (p ProviderEnum) MarshalText() ([]byte, error) {
-
-	switch p {
-	case ProvCommand:
-		return []byte("command"), nil
-	case ProvRsync:
-		return []byte("rsync"), nil
-	case ProvTwoStageRsync:
-		return []byte("two-stage-rsync"), nil
-	default:
-		return []byte{}, errors.New("Invalid ProviderEnum value")
-	}
-
-}
-
 func (p *ProviderEnum) UnmarshalText(text []byte) error {
 	s := string(text)
 	switch s {

worker/provider.go

@@ -122,7 +122,7 @@ func (p *baseProvider) Hooks() []jobHook {
 	return p.hooks
 }
 
-func (p *baseProvider) setLogFile() error {
+func (p *baseProvider) prepareLogFile() error {
 	if p.LogFile() == "/dev/null" {
 		p.cmd.SetLogFile(nil)
 		return nil

worker/rsync_provider.go

@@ -81,7 +81,7 @@ func (p *rsyncProvider) Start() error {
 	command = append(command, p.upstreamURL, p.WorkingDir())
 
 	p.cmd = newCmdJob(command, p.WorkingDir(), env)
-	if err := p.setLogFile(); err != nil {
+	if err := p.prepareLogFile(); err != nil {
 		return err
 	}
 

worker/two_stage_rsync_provider.go

@@ -117,7 +117,7 @@ func (p *twoStageRsyncProvider) Run() error {
 		command = append(command, p.upstreamURL, p.WorkingDir())
 
 		p.cmd = newCmdJob(command, p.WorkingDir(), env)
-		if err := p.setLogFile(); err != nil {
+		if err := p.prepareLogFile(); err != nil {
 			return err
 		}