From 561beca512a300b6d35e9c5a0c789fa4dcaa7609 Mon Sep 17 00:00:00 2001
From: Martin Pitt <martin@piware.de>
Date: Jan 27 2018 14:01:13 +0000
Subject: standard-tests-scripts: Implement required_packages


Replace the hardcoded `rsync` package with `required_packages` from the
test YAML.

---

diff --git a/roles/standard-test-scripts/README.md b/roles/standard-test-scripts/README.md
index d407bcc..4242bd1 100644
--- a/roles/standard-test-scripts/README.md
+++ b/roles/standard-test-scripts/README.md
@@ -14,4 +14,10 @@ You should define the following variables:
 
  * tests: An array of scripts to run
  * files: A list of files or directories needed by the scripts
+ * required_packages: A list of prerequisite packages required by
+     the tests. Please note that for Atomic Host, additional
+     packages will be installed using the rpm-ostree command which
+     is affecting the test subject (it's similar as rebuilding an
+     rpm package to be tested) so this should be used with caution
+     and only when necessary.
 
diff --git a/roles/standard-test-scripts/tasks/main.yml b/roles/standard-test-scripts/tasks/main.yml
index bddd620..16ed44a 100644
--- a/roles/standard-test-scripts/tasks/main.yml
+++ b/roles/standard-test-scripts/tasks/main.yml
@@ -2,7 +2,7 @@
 - name: Install testing requirements
   package: name={{ item }} state=present
   with_items:
-   - rsync
+   - "{{ required_packages }}"
   when: ansible_pkg_mgr != 'unknown'
 
 - name: Copy test scripts to target
diff --git a/roles/standard-test-scripts/vars/main.yml b/roles/standard-test-scripts/vars/main.yml
index 81108ab..85561fd 100644
--- a/roles/standard-test-scripts/vars/main.yml
+++ b/roles/standard-test-scripts/vars/main.yml
@@ -1,3 +1,4 @@
 ---
 tests: []
 files: []
+required_packages: []