From 74cc99fa35ef5b9aefc24b86bea072045dfcf5dc Mon Sep 17 00:00:00 2001
From: Matt Davis <nitzmahone@users.noreply.github.com>
Date: Wed, 12 Jul 2017 18:38:34 -0700
Subject: [PATCH] enable minimal test for win_chocolatey in shippable (#26731)

* until we can have a robust internal source for chocolatey (and/or PSScriptAnalyzer sanity tests), we want to at least make sure that the module is syntactially valid.
---
 .../targets/win_chocolatey/aliases            |  1 +
 .../targets/win_chocolatey/tasks/main.yml     | 77 ++++++++++---------
 2 files changed, 43 insertions(+), 35 deletions(-)

diff --git a/test/integration/targets/win_chocolatey/aliases b/test/integration/targets/win_chocolatey/aliases
index e69de29bb2d..c6d61981670 100644
--- a/test/integration/targets/win_chocolatey/aliases
+++ b/test/integration/targets/win_chocolatey/aliases
@@ -0,0 +1 @@
+windows/ci/group3
diff --git a/test/integration/targets/win_chocolatey/tasks/main.yml b/test/integration/targets/win_chocolatey/tasks/main.yml
index 6e461b4abc9..538dfd95541 100644
--- a/test/integration/targets/win_chocolatey/tasks/main.yml
+++ b/test/integration/targets/win_chocolatey/tasks/main.yml
@@ -16,46 +16,53 @@
 # You should have received a copy of the GNU General Public License
 # along with Ansible.  If not, see <http://www.gnu.org/licenses/>.
 
-- name: install sysinternals
+- name: simple failure smoke test # NB: this is the only test that runs under shippable until others can use non-internet endpoints
   win_chocolatey:
-    name: sysinternals
-    state: present
-  register: install_sysinternals
+  register: choco_fail
+  failed_when: "not choco_fail.msg | regex_search('Missing required argument: name')"
 
-- name: verify install sysinternals
-  assert:
-    that:
-    - 'install_sysinternals.changed == true'
+- when: lookup('env', 'ANSIBLE_TEST_CI') != 'shippable'
+  block:
+  - name: install sysinternals
+    win_chocolatey:
+      name: sysinternals
+      state: present
+    register: install_sysinternals
 
-- name: install sysinternals again
-  win_chocolatey:
-    name: sysinternals
-    state: present
-  register: install_sysinternals_again
+  - name: verify install sysinternals
+    assert:
+      that:
+      - 'install_sysinternals.changed == true'
 
-- name: verify install sysinternals again
-  assert:
-    that:
-    - 'install_sysinternals_again.changed == false'
+  - name: install sysinternals again
+    win_chocolatey:
+      name: sysinternals
+      state: present
+    register: install_sysinternals_again
 
-- name: remove sysinternals
-  win_chocolatey:
-    name: sysinternals
-    state: absent
-  register: remove_sysinternals
+  - name: verify install sysinternals again
+    assert:
+      that:
+      - 'install_sysinternals_again.changed == false'
 
-- name: verify remove sysinternals
-  assert:
-    that:
-    - 'remove_sysinternals.changed == true'
+  - name: remove sysinternals
+    win_chocolatey:
+      name: sysinternals
+      state: absent
+    register: remove_sysinternals
 
-- name: remove sysinternals again
-  win_chocolatey:
-    name: sysinternals
-    state: absent
-  register: remove_sysinternals_again
+  - name: verify remove sysinternals
+    assert:
+      that:
+      - 'remove_sysinternals.changed == true'
 
-- name: verify remove sysinternals again
-  assert:
-    that:
-    - 'remove_sysinternals_again.changed == false'
+  - name: remove sysinternals again
+    win_chocolatey:
+      name: sysinternals
+      state: absent
+    register: remove_sysinternals_again
+
+  - name: verify remove sysinternals again
+    assert:
+      that:
+      - 'remove_sysinternals_again.changed == false'