Skip to content
Permalink
Browse files

Terraform: Fix unbound local error (#57044)

* Added Unit tests
* Fixed regression

Fixes: #56934

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
  • Loading branch information...
Akasurde committed Jun 17, 2019
1 parent 9e225fb commit cf623fa62fd78910f86617cfbd7e8c9564feee28
Showing with 21 additions and 0 deletions.
  1. +1 −0 lib/ansible/modules/cloud/misc/terraform.py
  2. +20 −0 test/units/modules/cloud/misc/test_terraform.py
@@ -359,6 +359,7 @@ def main():
module.params.get('targets'), state, plan_file)
command.append(plan_file)

out, err = '', ''
if needs_application and not module.check_mode and not state == 'planned':
rc, out, err = module.run_command(command, cwd=project_path)
# checks out to decide if changes were made during execution
@@ -0,0 +1,20 @@
# Copyright: (c) 2019, Ansible Project
# GNU General Public License v3.0+ (see COPYING or https://www.gnu.org/licenses/gpl-3.0.txt)

import json

import pytest

from ansible.modules.cloud.misc import terraform
from units.modules.utils import set_module_args


def test_terraform_without_argument(capfd):
set_module_args({})
with pytest.raises(SystemExit) as results:
terraform.main()

out, err = capfd.readouterr()
assert not err
assert json.loads(out)['failed']
assert 'project_path' in json.loads(out)['msg']

0 comments on commit cf623fa

Please sign in to comment.
You can’t perform that action at this time.