salt-formula-testing icon indicating copy to clipboard operation
salt-formula-testing copied to clipboard

Fixing LocalCommand scoping issue.

Open kwakwaversal opened this issue 7 years ago • 1 comments

It appears testinfra has changed since you first worked on this repo. As such, testinfra -v didn't work as expected. This was due to scoping issues with LocalCommand, and testinfra.backend.docker no longer exists as a directly accessible attribute (you need to use get_backend_class instead).

I've updated things so that they should run for most people off the bat.

Fixes #1

References: https://docs.saltstack.com/en/latest/topics/installation/rhel.html (CentOS) https://github.com/infOpen/pytest-ansible-docker/blob/master/pytest_ansible_docker.py#L16 https://github.com/splksusqa/tsplk-formula/blob/1145717f414ca78c936313f95db77cb9b9c0e4c0/test/conftest.py#L82

kwakwaversal avatar Mar 15 '17 12:03 kwakwaversal

hello @amolenaar any news regarding this PR ?

marco-m avatar Jul 20 '17 18:07 marco-m