Skip to content

Commit 96c2daa

Browse files
committed
Apply Pino's review
1 parent 3ad822a commit 96c2daa

7 files changed

Lines changed: 19 additions & 30 deletions

tests/tasks/remove_candlepin_container.yml

Lines changed: 0 additions & 23 deletions
This file was deleted.

tests/tasks/setup_candlepin.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,9 @@
33
- name: Setup the test data
44
import_tasks: setup_test_data.yml
55

6+
- name: Remove Candlepin container
7+
include_tasks: teardown_candlepin.yml
8+
69
- name: Deploy Candlepin
710
when: lsr_rhc_test_data_file | length == 0
811
block:
@@ -36,9 +39,6 @@
3639
use: "{{ __rhc_is_ostree |
3740
ternary('ansible.posix.rhel_rpm_ostree', omit) }}"
3841

39-
- name: Remove Candlepin container
40-
include_tasks: remove_candlepin_container.yml
41-
4242
- name: Start Candlepin container
4343
vars:
4444
podman_run_cli:

tests/tasks/teardown_candlepin.yml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
# SPDX-License-Identifier: MIT
2+
---
3+
- name: Teardown Candlepin container
4+
when: lsr_rhc_test_data_file | length == 0
5+
block:
6+
- name: Stop and remove Candlepin container
7+
command:
8+
argv:
9+
- podman
10+
- stop
11+
- candlepin
12+
changed_when: true

tests/tests_environments.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -109,4 +109,4 @@
109109
rhc_state: absent
110110
always:
111111
- name: Clean up Candlepin container
112-
include_tasks: tasks/remove_candlepin_container.yml
112+
include_tasks: tasks/teardown_candlepin.yml

tests/tests_proxy.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -357,7 +357,7 @@
357357
rhc_state: absent
358358

359359
- name: Clean up Candlepin container
360-
include_tasks: tasks/remove_candlepin_container.yml
360+
include_tasks: tasks/teardown_candlepin.yml
361361

362362
- name: Remove SELinux policy for proxy ports
363363
include_role:

tests/tests_register_unregister.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -190,4 +190,4 @@
190190
rhc_state: absent
191191

192192
- name: Clean up Candlepin container
193-
include_tasks: tasks/remove_candlepin_container.yml
193+
include_tasks: tasks/teardown_candlepin.yml

tests/tests_repositories.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,4 +102,4 @@
102102
rhc_state: absent
103103

104104
- name: Clean up Candlepin container
105-
include_tasks: tasks/remove_candlepin_container.yml
105+
include_tasks: tasks/teardown_candlepin.yml

0 commit comments

Comments
 (0)