Files
atlantis/server/events/pre_workflow_hooks_command_runner_test.go
KevinSnyderCodes 1a8344489f fix: add path to WorkingDir methods (#2180)
* fix: add path to WorkingDir methods

`WorkingDirLocker` and `WorkingDir` are closely related -- the former acquires a lock, which ensures that the latter can safely operate on a given file path.

In #2131 we added `path` to the `WorkingDirLocker` lock key, but neglected to add the same to `WorkingDir`.

This commit adds `path` as an argument to certain `WorkingDir` methods, and includes `path` in the directory that we use to clone the repository for a given project.

Since `path` can include certain special characters such as `/`, we encode `path` as base32 when using it as part of a file path. This should ensure no special characters are used in the filesystem, and that the value can be decoded if desired (unlike hashes such as md5).

Additional changes:

- All calls to changed methods have been updated, including unit tests
- Mocks have been regenerated for `WorkingDir` and `WorkingDirLocker`
- `working_dir_test.go`
  - Commands that operate on filesystem paths have been updated to include the base32 encoded `path` value
  - When running `git init`, we include `-b master` as additional arguments, to ensure that `master` is our default branch (expected by the unit tests)

* Try fixing E2E tests

* Fix DefaultPendingPlanFinder

In addition to iterating over `workspaceDirs`, also iterate over `pathDirs`, and use both `workspace` and `path` to build `repoDir`.

* Fix DefaultPendingPlanFinder unit tests

* Fix DefaultProjectCommandBuilder unit tests

Co-authored-by: Kevin Snyder <kevinsnyder@KevinSnydersMBP.lan>
Co-authored-by: Kevin Snyder <kevinsnyder@ip-192-168-1-188.ec2.internal>
2022-04-08 11:55:35 -07:00

223 lines
6.5 KiB
Go

package events_test
import (
"errors"
"testing"
. "github.com/petergtz/pegomock"
"github.com/runatlantis/atlantis/server/core/config/valid"
runtime_mocks "github.com/runatlantis/atlantis/server/core/runtime/mocks"
"github.com/runatlantis/atlantis/server/events"
"github.com/runatlantis/atlantis/server/events/command"
"github.com/runatlantis/atlantis/server/events/mocks"
"github.com/runatlantis/atlantis/server/events/models"
"github.com/runatlantis/atlantis/server/events/models/fixtures"
vcsmocks "github.com/runatlantis/atlantis/server/events/vcs/mocks"
"github.com/runatlantis/atlantis/server/logging"
. "github.com/runatlantis/atlantis/testing"
)
var preWh events.DefaultPreWorkflowHooksCommandRunner
var preWhWorkingDir *mocks.MockWorkingDir
var preWhWorkingDirLocker *mocks.MockWorkingDirLocker
var whPreWorkflowHookRunner *runtime_mocks.MockPreWorkflowHookRunner
func preWorkflowHooksSetup(t *testing.T) {
RegisterMockTestingT(t)
vcsClient := vcsmocks.NewMockClient()
preWhWorkingDir = mocks.NewMockWorkingDir()
preWhWorkingDirLocker = mocks.NewMockWorkingDirLocker()
whPreWorkflowHookRunner = runtime_mocks.NewMockPreWorkflowHookRunner()
preWh = events.DefaultPreWorkflowHooksCommandRunner{
VCSClient: vcsClient,
WorkingDirLocker: preWhWorkingDirLocker,
WorkingDir: preWhWorkingDir,
PreWorkflowHookRunner: whPreWorkflowHookRunner,
}
}
func newBool(b bool) *bool {
return &b
}
func TestRunPreHooks_Clone(t *testing.T) {
log := logging.NewNoopLogger(t)
var newPull = fixtures.Pull
newPull.BaseRepo = fixtures.GithubRepo
ctx := &command.Context{
Pull: newPull,
HeadRepo: fixtures.GithubRepo,
User: fixtures.User,
Log: log,
}
testHook := valid.WorkflowHook{
StepName: "test",
RunCommand: "some command",
}
pCtx := models.WorkflowHookCommandContext{
BaseRepo: fixtures.GithubRepo,
HeadRepo: fixtures.GithubRepo,
Pull: newPull,
Log: log,
User: fixtures.User,
Verbose: false,
}
repoDir := "path/to/repo"
result := "some result"
t.Run("success hooks in cfg", func(t *testing.T) {
preWorkflowHooksSetup(t)
var unlockCalled = newBool(false)
unlockFn := func() {
unlockCalled = newBool(true)
}
globalCfg := valid.GlobalCfg{
Repos: []valid.Repo{
{
ID: fixtures.GithubRepo.ID(),
PreWorkflowHooks: []*valid.WorkflowHook{
&testHook,
},
},
},
}
preWh.GlobalCfg = globalCfg
When(preWhWorkingDirLocker.TryLock(fixtures.GithubRepo.FullName, newPull.Num, events.DefaultWorkspace, events.DefaultRepoRelDir)).ThenReturn(unlockFn, nil)
When(preWhWorkingDir.Clone(log, fixtures.GithubRepo, newPull, events.DefaultWorkspace, events.DefaultRepoRelDir)).ThenReturn(repoDir, false, nil)
When(whPreWorkflowHookRunner.Run(pCtx, testHook.RunCommand, repoDir)).ThenReturn(result, nil)
err := preWh.RunPreHooks(ctx)
Ok(t, err)
whPreWorkflowHookRunner.VerifyWasCalledOnce().Run(pCtx, testHook.RunCommand, repoDir)
Assert(t, *unlockCalled == true, "unlock function called")
})
t.Run("success hooks not in cfg", func(t *testing.T) {
preWorkflowHooksSetup(t)
globalCfg := valid.GlobalCfg{
Repos: []valid.Repo{
// one with hooks but mismatched id
{
ID: "id1",
PreWorkflowHooks: []*valid.WorkflowHook{
&testHook,
},
},
// one with the correct id but no hooks
{
ID: fixtures.GithubRepo.ID(),
PreWorkflowHooks: []*valid.WorkflowHook{},
},
},
}
preWh.GlobalCfg = globalCfg
err := preWh.RunPreHooks(ctx)
Ok(t, err)
whPreWorkflowHookRunner.VerifyWasCalled(Never()).Run(pCtx, testHook.RunCommand, repoDir)
preWhWorkingDirLocker.VerifyWasCalled(Never()).TryLock(fixtures.GithubRepo.FullName, newPull.Num, events.DefaultWorkspace, events.DefaultRepoRelDir)
preWhWorkingDir.VerifyWasCalled(Never()).Clone(log, fixtures.GithubRepo, newPull, events.DefaultWorkspace, events.DefaultRepoRelDir)
})
t.Run("error locking work dir", func(t *testing.T) {
preWorkflowHooksSetup(t)
globalCfg := valid.GlobalCfg{
Repos: []valid.Repo{
{
ID: fixtures.GithubRepo.ID(),
PreWorkflowHooks: []*valid.WorkflowHook{
&testHook,
},
},
},
}
preWh.GlobalCfg = globalCfg
When(preWhWorkingDirLocker.TryLock(fixtures.GithubRepo.FullName, newPull.Num, events.DefaultWorkspace, events.DefaultRepoRelDir)).ThenReturn(func() {}, errors.New("some error"))
err := preWh.RunPreHooks(ctx)
Assert(t, err != nil, "error not nil")
preWhWorkingDir.VerifyWasCalled(Never()).Clone(log, fixtures.GithubRepo, newPull, events.DefaultWorkspace, events.DefaultRepoRelDir)
whPreWorkflowHookRunner.VerifyWasCalled(Never()).Run(pCtx, testHook.RunCommand, repoDir)
})
t.Run("error cloning", func(t *testing.T) {
preWorkflowHooksSetup(t)
var unlockCalled = newBool(false)
unlockFn := func() {
unlockCalled = newBool(true)
}
globalCfg := valid.GlobalCfg{
Repos: []valid.Repo{
{
ID: fixtures.GithubRepo.ID(),
PreWorkflowHooks: []*valid.WorkflowHook{
&testHook,
},
},
},
}
preWh.GlobalCfg = globalCfg
When(preWhWorkingDirLocker.TryLock(fixtures.GithubRepo.FullName, newPull.Num, events.DefaultWorkspace, events.DefaultRepoRelDir)).ThenReturn(unlockFn, nil)
When(preWhWorkingDir.Clone(log, fixtures.GithubRepo, newPull, events.DefaultWorkspace, events.DefaultRepoRelDir)).ThenReturn(repoDir, false, errors.New("some error"))
err := preWh.RunPreHooks(ctx)
Assert(t, err != nil, "error not nil")
whPreWorkflowHookRunner.VerifyWasCalled(Never()).Run(pCtx, testHook.RunCommand, repoDir)
Assert(t, *unlockCalled == true, "unlock function called")
})
t.Run("error running pre hook", func(t *testing.T) {
preWorkflowHooksSetup(t)
var unlockCalled = newBool(false)
unlockFn := func() {
unlockCalled = newBool(true)
}
globalCfg := valid.GlobalCfg{
Repos: []valid.Repo{
{
ID: fixtures.GithubRepo.ID(),
PreWorkflowHooks: []*valid.WorkflowHook{
&testHook,
},
},
},
}
preWh.GlobalCfg = globalCfg
When(preWhWorkingDirLocker.TryLock(fixtures.GithubRepo.FullName, newPull.Num, events.DefaultWorkspace, events.DefaultRepoRelDir)).ThenReturn(unlockFn, nil)
When(preWhWorkingDir.Clone(log, fixtures.GithubRepo, newPull, events.DefaultWorkspace, events.DefaultRepoRelDir)).ThenReturn(repoDir, false, nil)
When(whPreWorkflowHookRunner.Run(pCtx, testHook.RunCommand, repoDir)).ThenReturn(result, errors.New("some error"))
err := preWh.RunPreHooks(ctx)
Assert(t, err != nil, "error not nil")
Assert(t, *unlockCalled == true, "unlock function called")
})
}