Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix test cases for remove operation #14

Merged
merged 12 commits into from
Jul 13, 2018
Merged

Fix test cases for remove operation #14

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
699dd1c
Separate lint to it's own stage, and dry run as it's own stage
Rickkwa Jul 12, 2018
8047c33
Start containers; idempotency test
Rickkwa Jul 12, 2018
e4272b1
Allow lint failures?
Rickkwa Jul 12, 2018
0779c93
Add second idempotency test; introduce linting error to check allow_f…
Rickkwa Jul 13, 2018
203496a
Add dry run
Rickkwa Jul 13, 2018
93f881c
Add second dry run check for removing users; introduce lint error on …
Rickkwa Jul 13, 2018
a283ffd
try allow failures one last time
Rickkwa Jul 13, 2018
61d0bcc
Fix lint error
Rickkwa Jul 13, 2018
c9a7923
Rename action to operation since action is reversed word in ansible […
Rickkwa Jul 13, 2018
d68bbfe
fix remove operation not running
Rickkwa Jul 13, 2018
17c296e
Use set_fact task instead of play vars
Rickkwa Jul 13, 2018
85f2bd6
Merge branch 'master' into extra_jobs [ci skip]
Rickkwa Jul 13, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Use set_fact task instead of play vars
  • Loading branch information
@Rickkwa
Rickkwa committed Jul 13, 2018
commit 17c296ef33ae7b968bf2bff695bca26c4895134f
56 changes: 31 additions & 25 deletions tests/test-dry-run.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,28 +1,34 @@
---

- hosts: containers
vars:
um_uid_base: 10000
um_gid_base: 10000
um_group_inventory:
- name: customgroup
rel_gid: 1
present: ['all']
absent:
- "{{ 'all' if operation == 'remove' else '' }}"
sudoer: |
%customgroup ALL=(ALL) ALL
um_user_inventory:
- name: foo
rel_uid: 1
present: ['all']
absent:
- "{{ 'all' if operation == 'remove' else '' }}"
groups:
- customgroup
- name: bar
rel_uid: 2
present:
- all
roles:
- user_management
tasks:
- name: Set user/group inventory
set_fact:
um_uid_base: 10000
um_gid_base: 10000
um_group_inventory:
- name: customgroup
rel_gid: 1
present: ['all']
absent:
- "{{ 'all' if operation == 'remove' else '' }}"
sudoer: |
%customgroup ALL=(ALL) ALL
um_user_inventory:
- name: foo
rel_uid: 1
present: ['all']
absent:
- "{{ 'all' if operation == 'remove' else '' }}"
groups:
- customgroup
- name: bar
rel_uid: 2
present:
- all
absent:
- "{{ 'all' if operation == 'remove' else '' }}"

- name: Run role
import_role:
name: user_management
56 changes: 31 additions & 25 deletions tests/test-idempotency.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,28 +1,34 @@
---

- hosts: containers
vars:
um_uid_base: 10000
um_gid_base: 10000
um_group_inventory:
- name: customgroup
rel_gid: 1
present: ['all']
absent:
- "{{ 'all' if operation == 'remove' else '' }}"
sudoer: |
%customgroup ALL=(ALL) ALL
um_user_inventory:
- name: foo
rel_uid: 1
present: ['all']
absent:
- "{{ 'all' if operation == 'remove' else '' }}"
groups:
- customgroup
- name: bar
rel_uid: 2
present:
- all
roles:
- user_management
tasks:
- name: Set user/group inventory
set_fact:
um_uid_base: 10000
um_gid_base: 10000
um_group_inventory:
- name: customgroup
rel_gid: 1
present: ['all']
absent:
- "{{ 'all' if operation == 'remove' else '' }}"
sudoer: |
%customgroup ALL=(ALL) ALL
um_user_inventory:
- name: foo
rel_uid: 1
present: ['all']
absent:
- "{{ 'all' if operation == 'remove' else '' }}"
groups:
- customgroup
- name: bar
rel_uid: 2
present:
- all
absent:
- "{{ 'all' if operation == 'remove' else '' }}"

- name: Run role
import_role:
name: user_management