unarchive: handle content and size differences (#83454)

Consider content differs and size differs while unarchiving
the same tar.gz file

Fixes: #29610

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>

7 files changed, 110 insertions, 0 deletions

diff --git a/test/integration/targets/unarchive/files/content_differs.tar.gz b/test/integration/targets/unarchive/files/content_differs.tar.gz
new file mode 100644
index 0000000000..86d0891be5
--- /dev/null
+++ b/test/integration/targets/unarchive/files/content_differs.tar.gz
diff --git a/test/integration/targets/unarchive/files/content_differs_2.tar.gz b/test/integration/targets/unarchive/files/content_differs_2.tar.gz
new file mode 100644
index 0000000000..d048e50f8c
--- /dev/null
+++ b/test/integration/targets/unarchive/files/content_differs_2.tar.gz
diff --git a/test/integration/targets/unarchive/files/size_differs.tar.gz b/test/integration/targets/unarchive/files/size_differs.tar.gz
new file mode 100644
index 0000000000..7f76808dc7
--- /dev/null
+++ b/test/integration/targets/unarchive/files/size_differs.tar.gz
diff --git a/test/integration/targets/unarchive/files/size_differs_2.tar.gz b/test/integration/targets/unarchive/files/size_differs_2.tar.gz
new file mode 100644
index 0000000000..0d53561b84
--- /dev/null
+++ b/test/integration/targets/unarchive/files/size_differs_2.tar.gz
diff --git a/test/integration/targets/unarchive/tasks/main.yml b/test/integration/targets/unarchive/tasks/main.yml
index b07c2fe7fb..278642a106 100644
--- a/test/integration/targets/unarchive/tasks/main.yml
+++ b/test/integration/targets/unarchive/tasks/main.yml
@@ -5,6 +5,8 @@
 - import_tasks: test_tar_gz_creates.yml
 - import_tasks: test_tar_gz_owner_group.yml
 - import_tasks: test_tar_gz_keep_newer.yml
+- import_tasks: test_tar_gz_size_differs.yml
+- import_tasks: test_tar_gz_content_differs.yml
 - import_tasks: test_tar_zst.yml
 - import_tasks: test_zip.yml
 - import_tasks: test_exclude.yml
diff --git a/test/integration/targets/unarchive/tasks/test_tar_gz_content_differs.yml b/test/integration/targets/unarchive/tasks/test_tar_gz_content_differs.yml
new file mode 100644
index 0000000000..79a0be2398
--- /dev/null
+++ b/test/integration/targets/unarchive/tasks/test_tar_gz_content_differs.yml
@@ -0,0 +1,56 @@
+- set_fact:
+    dest: '{{remote_tmp_dir}}/test-unarchive-tar-gz-content-differs'
+
+- name: create our tar.gz unarchive destination
+  file:
+    path: "{{ dest }}"
+    state: directory
+
+- name: unarchive a tar.gz file
+  unarchive:
+    src: 'content_differs.tar.gz'
+    dest: '{{ dest }}'
+    remote_src: no
+  register: unarchive_content_differs_01
+
+- name: verify that the task was marked as changed
+  assert:
+    that:
+      - unarchive_content_differs_01.changed
+
+- name: checksum the file after first unarchive
+  stat:
+    path: '{{ dest }}/hello.txt'
+    checksum_algorithm: sha256
+    get_checksum: yes
+  register: first_unarchive_content
+
+- name: unarchive a tar.gz file with same name, size but different content
+  unarchive:
+    src: 'content_differs_2.tar.gz'
+    dest: '{{ dest }}'
+    remote_src: no
+  register: unarchive_content_differs_01
+
+- name: verify that the task was marked as changed
+  assert:
+    that:
+      - unarchive_content_differs_01.changed
+
+- name: checksum the file after second unarchive
+  stat:
+    path: '{{ dest }}/hello.txt'
+    checksum_algorithm: sha256
+    get_checksum: yes
+  register: second_unarchive_content
+
+- name: verify that unarchive extracted file with new content
+  assert:
+    that:
+      - first_unarchive_content.stat.checksum != second_unarchive_content.stat.checksum
+      - first_unarchive_content.stat.size == second_unarchive_content.stat.size
+
+- name: remove our tar.gz unarchive destination
+  file:
+    path: '{{ dest }}'
+    state: absent
diff --git a/test/integration/targets/unarchive/tasks/test_tar_gz_size_differs.yml b/test/integration/targets/unarchive/tasks/test_tar_gz_size_differs.yml
new file mode 100644
index 0000000000..c4a0d2af80
--- /dev/null
+++ b/test/integration/targets/unarchive/tasks/test_tar_gz_size_differs.yml
@@ -0,0 +1,52 @@
+- set_fact:
+    dest: '{{remote_tmp_dir}}/test-unarchive-tar-gz-size-differs'
+
+- name: create our tar.gz unarchive destination
+  file:
+    path: "{{ dest }}"
+    state: directory
+
+- name: unarchive a tar.gz file
+  unarchive:
+    src: 'size_differs.tar.gz'
+    dest: '{{ dest }}'
+    remote_src: no
+  register: unarchive_size_differs_01
+
+- name: verify that the task was marked as changed
+  assert:
+    that:
+      - unarchive_size_differs_01.changed
+
+- name: Check size after first unarchive
+  stat:
+    path: '{{ dest }}/hello.txt'
+  register: first_unarchive
+
+- name: unarchive a tar.gz file with same name but different size
+  unarchive:
+    src: 'size_differs_2.tar.gz'
+    dest: '{{ dest }}'
+    remote_src: no
+  register: unarchive_size_differs_02
+
+- name: verify that the task was marked as changed
+  assert:
+    that:
+      - unarchive_size_differs_02.changed
+
+- name: Check size after unarchive
+  stat:
+    path: '{{ dest }}/hello.txt'
+  register: second_unarchive
+
+- name: verify that unarchive extracted new sized file
+  assert:
+    that:
+      - first_unarchive.stat.size != second_unarchive.stat.size
+      - first_unarchive.stat.size < second_unarchive.stat.size
+
+- name: remove our tar.gz unarchive destination
+  file:
+    path: '{{ dest }}'
+    state: absent