Revert "fixing a variable assertion bug; they were not working as wri…

…tten" This reverts commit a49d7d9.
nginxinc · Sep 22, 2021 · a63daab · a63daab
1 parent a49d7d9
commit a63daab
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,7 +6,3 @@ ENHANCEMENTS:
 
 *   Implement GitHub Actions CI/CD workflow for repo.
 *   Add Molecule tests for role.
-
-BUG FIXES:
-
-*   The variable assertions were not working as implemented.
diff --git a/molecule/default/converge.yml b/molecule/default/converge.yml
@@ -3,6 +3,9 @@
   hosts: controllers
 
   vars:
+    nginx_controller_api_key: ""
+    nginx_controller_user_email: ""
+    nginx_controller_user_password: ""
     nginx_controller_validate_certs: false
     nginx_controller_fqdn: mock-server.molecule-test:1080
     nginx_controller_auth_token: "ABC123"

diff --git a/tasks/main.yml b/tasks/main.yml
@@ -1,7 +1,7 @@
 ---
 - name: Fail if required variables are not defined
   assert:
-    that: item is in vars and vars[item] | length > 0
+    that: ("{{ item }} is defined") and ("{{ item }} | length > 0")
   loop:
     - nginx_controller_fqdn
     - nginx_controller_auth_token