Merge pull request #55 from maxamillion/issues/37/authorized_key-check_mode-inconsistent-return

authorized_keys - consistent behavior in check_mode

Reviewed-by: https://github.com/apps/ansible-zuul

tests/integration/targets/authorized_key/tasks/main.yml

@@ -410,12 +410,13 @@
 # -------------------------------------------------------------
 # check mode
 
-- name: copy an existing file in place with comments
+- name: CHECK MODE | copy an existing file in place with comments
   copy:
     src: existing_authorized_keys
     dest: "{{ output_dir | expanduser }}/authorized_keys"
 
-- authorized_key:
+- name: CHECK MODE | add key in check mode to validate return codes
+  authorized_key:
     user: root
     key: "{{ multiple_key_different_order_2 }}"
     state: present
@@ -423,13 +424,21 @@
   check_mode: True
   register: result
 
-- name: assert that the file was not changed
+- name: CHECK MODE | assert that authorized_keys return values are consistent
+  assert:
+    that:
+      - 'result.changed == True'
+      - '"user" in result'
+      - '"key" in result'
+
+- name: CHECK MODE | recopy authorized_keys to ensure it was not changed 
   copy:
     src: existing_authorized_keys
     dest: "{{ output_dir | expanduser }}/authorized_keys"
   register: result
 
-- assert:
+- name: CHECK MODE | assert that the authorized_keys file was not changed
+  assert:
     that:
       - 'result.changed == False'