conftest pull git repository fails if folder already exists

[elisiariocouto]

Hello everyone. We are using conftest to automate some OPA checks.

We have a repository with our OPA policies and we run conftest pull git::<my repo> before executing conftest test and it fails if the default folder policy already exists.

You can easily reproduce this by running conftest pull git::<your repo> twice, the first time it pulls the policies, the second time it fails with a git error (which is hiding the real issue):

Error: download policies: client get: error downloading 'ssh://git@<myrepo>': /opt/homebrew/bin/git exited with 128: fatal: empty string is not a valid pathspec. please use . instead if you meant to match all paths

Is this intended?

Thank you

[jalseth]

Marking this as an enhancement for now as there is a straightforward work-around available by deleting the folder first.

This would be a good candidate for an action that prompts the user by default, but the confirmation can be skipped with a flag so it works reliably in scripts etc.

PRs are welcomed.