Skip to content
This repository has been archived by the owner on Nov 14, 2024. It is now read-only.

Commit

Permalink
Ansible integration test fixes - pt 2 (#147)
Browse files Browse the repository at this point in the history
<!-- This change is generated by MagicModules. -->
/cc @rambleraptor
  • Loading branch information
modular-magician authored and rambleraptor committed Sep 5, 2019
1 parent f288ffc commit 3496923
Show file tree
Hide file tree
Showing 12 changed files with 112 additions and 100 deletions.
4 changes: 2 additions & 2 deletions plugins/modules/gcp_compute_vpn_tunnel.py
Original file line number Diff line number Diff line change
Expand Up @@ -161,8 +161,8 @@
state: present
register: gateway
- name: create a vpn-tunnel
gcp_compute_vpn-tunnel:
- name: create a vpn tunnel
gcp_compute_vpn_tunnel:
name: "test_object"
region: us-west1
target_vpn_gateway: "{{ gateway }}"
Expand Down
4 changes: 3 additions & 1 deletion plugins/modules/gcp_iam_service_account.py
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,9 @@
EXAMPLES = '''
- name: create a service account
gcp_iam_service_account:
name: test-ansible@graphite-playground.google.com.iam.gserviceaccount.com
name: '"{{resource_name}}@{{gcp_project}}.google.com.iam.gserviceaccount.com"
'
display_name: My Ansible test key
project: "test_project"
auth_kind: "serviceaccount"
Expand Down
2 changes: 1 addition & 1 deletion plugins/modules/gcp_sql_database.py
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@
EXAMPLES = '''
- name: create a instance
gcp_sql_instance:
name: "{{ resource_name }}"
name: "{{resource_name}}-3"
settings:
ip_configuration:
authorized_networks:
Expand Down
2 changes: 1 addition & 1 deletion plugins/modules/gcp_sql_instance.py
Original file line number Diff line number Diff line change
Expand Up @@ -272,7 +272,7 @@
EXAMPLES = '''
- name: create a instance
gcp_sql_instance:
name: "test_object"
name: "{{resource_name}}-2"
settings:
ip_configuration:
authorized_networks:
Expand Down
2 changes: 1 addition & 1 deletion plugins/modules/gcp_sql_user.py
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@
EXAMPLES = '''
- name: create a instance
gcp_sql_instance:
name: "{{ resource_name }}"
name: "{{resource_name}}-1"
settings:
ip_configuration:
authorized_networks:
Expand Down
10 changes: 5 additions & 5 deletions plugins/modules/gcp_storage_bucket_access_control.py
Original file line number Diff line number Diff line change
Expand Up @@ -109,11 +109,11 @@
EXAMPLES = '''
- name: create a bucket
gcp_storage_bucket:
name: "{{ resource_name }}"
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
service_account_file: "{{ gcp_cred_file }}"
state: present
name: "{{ resource_name }}"
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
service_account_file: "{{ gcp_cred_file }}"
state: present
register: bucket
- name: create a bucket access control
Expand Down
20 changes: 10 additions & 10 deletions tests/integration/gcp_compute_vpn_tunnel/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,8 @@
service_account_file: "{{ gcp_cred_file }}"
state: present
register: gateway
- name: delete a vpn-tunnel
gcp_compute_vpn-tunnel:
- name: delete a vpn tunnel
gcp_compute_vpn_tunnel:
name: "{{ resource_name }}"
region: us-west1
target_vpn_gateway: "{{ gateway }}"
Expand All @@ -61,8 +61,8 @@
service_account_file: "{{ gcp_cred_file }}"
state: absent
#----------------------------------------------------------
- name: create a vpn-tunnel
gcp_compute_vpn-tunnel:
- name: create a vpn tunnel
gcp_compute_vpn_tunnel:
name: "{{ resource_name }}"
region: us-west1
target_vpn_gateway: "{{ gateway }}"
Expand Down Expand Up @@ -94,8 +94,8 @@
that:
- results['resources'] | length == 1
# ----------------------------------------------------------------------------
- name: create a vpn-tunnel that already exists
gcp_compute_vpn-tunnel:
- name: create a vpn tunnel that already exists
gcp_compute_vpn_tunnel:
name: "{{ resource_name }}"
region: us-west1
target_vpn_gateway: "{{ gateway }}"
Expand All @@ -112,8 +112,8 @@
- result.changed == false
- "result.kind == 'compute#vpnTunnel'"
#----------------------------------------------------------
- name: delete a vpn-tunnel
gcp_compute_vpn-tunnel:
- name: delete a vpn tunnel
gcp_compute_vpn_tunnel:
name: "{{ resource_name }}"
region: us-west1
target_vpn_gateway: "{{ gateway }}"
Expand Down Expand Up @@ -145,8 +145,8 @@
that:
- results['resources'] | length == 0
# ----------------------------------------------------------------------------
- name: delete a vpn-tunnel that does not exist
gcp_compute_vpn-tunnel:
- name: delete a vpn tunnel that does not exist
gcp_compute_vpn_tunnel:
name: "{{ resource_name }}"
region: us-west1
target_vpn_gateway: "{{ gateway }}"
Expand Down
20 changes: 15 additions & 5 deletions tests/integration/gcp_iam_service_account/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,9 @@
# Pre-test setup
- name: delete a service account
gcp_iam_service_account:
name: test-ansible@graphite-playground.google.com.iam.gserviceaccount.com
name: '"{{resource_name}}@{{gcp_project}}.google.com.iam.gserviceaccount.com"
'
display_name: My Ansible test key
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
Expand All @@ -24,7 +26,9 @@
#----------------------------------------------------------
- name: create a service account
gcp_iam_service_account:
name: test-ansible@graphite-playground.google.com.iam.gserviceaccount.com
name: '"{{resource_name}}@{{gcp_project}}.google.com.iam.gserviceaccount.com"
'
display_name: My Ansible test key
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
Expand All @@ -50,7 +54,9 @@
# ----------------------------------------------------------------------------
- name: create a service account that already exists
gcp_iam_service_account:
name: test-ansible@graphite-playground.google.com.iam.gserviceaccount.com
name: '"{{resource_name}}@{{gcp_project}}.google.com.iam.gserviceaccount.com"
'
display_name: My Ansible test key
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
Expand All @@ -64,7 +70,9 @@
#----------------------------------------------------------
- name: delete a service account
gcp_iam_service_account:
name: test-ansible@graphite-playground.google.com.iam.gserviceaccount.com
name: '"{{resource_name}}@{{gcp_project}}.google.com.iam.gserviceaccount.com"
'
display_name: My Ansible test key
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
Expand All @@ -90,7 +98,9 @@
# ----------------------------------------------------------------------------
- name: delete a service account that does not exist
gcp_iam_service_account:
name: test-ansible@graphite-playground.google.com.iam.gserviceaccount.com
name: '"{{resource_name}}@{{gcp_project}}.google.com.iam.gserviceaccount.com"
'
display_name: My Ansible test key
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
Expand Down
4 changes: 2 additions & 2 deletions tests/integration/gcp_sql_database/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
# Pre-test setup
- name: create a instance
gcp_sql_instance:
name: "{{ resource_name }}"
name: "{{resource_name}}-3"
settings:
ip_configuration:
authorized_networks:
Expand Down Expand Up @@ -132,7 +132,7 @@
# If errors happen, don't crash the playbook!
- name: delete a instance
gcp_sql_instance:
name: "{{ resource_name }}"
name: "{{resource_name}}-3"
settings:
ip_configuration:
authorized_networks:
Expand Down
120 changes: 60 additions & 60 deletions tests/integration/gcp_sql_instance/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,33 +15,33 @@
# Pre-test setup
- name: delete a instance
gcp_sql_instance:
name: "{{resource_name}}-2"
settings:
ip_configuration:
authorized_networks:
- name: google dns server
value: 8.8.8.8/32
tier: db-n1-standard-1
region: us-central1
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
service_account_file: "{{ gcp_cred_file }}"
state: absent
name: "{{resource_name}}-2"
settings:
ip_configuration:
authorized_networks:
- name: google dns server
value: 8.8.8.8/32
tier: db-n1-standard-1
region: us-central1
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
service_account_file: "{{ gcp_cred_file }}"
state: absent
#----------------------------------------------------------
- name: create a instance
gcp_sql_instance:
name: "{{resource_name}}-2"
settings:
ip_configuration:
authorized_networks:
- name: google dns server
value: 8.8.8.8/32
tier: db-n1-standard-1
region: us-central1
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
service_account_file: "{{ gcp_cred_file }}"
state: present
name: "{{resource_name}}-2"
settings:
ip_configuration:
authorized_networks:
- name: google dns server
value: 8.8.8.8/32
tier: db-n1-standard-1
region: us-central1
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
service_account_file: "{{ gcp_cred_file }}"
state: present
register: result
- name: assert changed is true
assert:
Expand All @@ -63,18 +63,18 @@
# ----------------------------------------------------------------------------
- name: create a instance that already exists
gcp_sql_instance:
name: "{{resource_name}}-2"
settings:
ip_configuration:
authorized_networks:
- name: google dns server
value: 8.8.8.8/32
tier: db-n1-standard-1
region: us-central1
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
service_account_file: "{{ gcp_cred_file }}"
state: present
name: "{{resource_name}}-2"
settings:
ip_configuration:
authorized_networks:
- name: google dns server
value: 8.8.8.8/32
tier: db-n1-standard-1
region: us-central1
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
service_account_file: "{{ gcp_cred_file }}"
state: present
register: result
- name: assert changed is false
assert:
Expand All @@ -84,18 +84,18 @@
#----------------------------------------------------------
- name: delete a instance
gcp_sql_instance:
name: "{{resource_name}}-2"
settings:
ip_configuration:
authorized_networks:
- name: google dns server
value: 8.8.8.8/32
tier: db-n1-standard-1
region: us-central1
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
service_account_file: "{{ gcp_cred_file }}"
state: absent
name: "{{resource_name}}-2"
settings:
ip_configuration:
authorized_networks:
- name: google dns server
value: 8.8.8.8/32
tier: db-n1-standard-1
region: us-central1
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
service_account_file: "{{ gcp_cred_file }}"
state: absent
register: result
- name: assert changed is true
assert:
Expand All @@ -117,18 +117,18 @@
# ----------------------------------------------------------------------------
- name: delete a instance that does not exist
gcp_sql_instance:
name: "{{resource_name}}-2"
settings:
ip_configuration:
authorized_networks:
- name: google dns server
value: 8.8.8.8/32
tier: db-n1-standard-1
region: us-central1
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
service_account_file: "{{ gcp_cred_file }}"
state: absent
name: "{{resource_name}}-2"
settings:
ip_configuration:
authorized_networks:
- name: google dns server
value: 8.8.8.8/32
tier: db-n1-standard-1
region: us-central1
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
service_account_file: "{{ gcp_cred_file }}"
state: absent
register: result
- name: assert changed is false
assert:
Expand Down
4 changes: 2 additions & 2 deletions tests/integration/gcp_sql_user/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
# Pre-test setup
- name: create a instance
gcp_sql_instance:
name: "{{ resource_name }}"
name: "{{resource_name}}-1"
settings:
ip_configuration:
authorized_networks:
Expand Down Expand Up @@ -137,7 +137,7 @@
# If errors happen, don't crash the playbook!
- name: delete a instance
gcp_sql_instance:
name: "{{ resource_name }}"
name: "{{resource_name}}-1"
settings:
ip_configuration:
authorized_networks:
Expand Down
20 changes: 10 additions & 10 deletions tests/integration/gcp_storage_bucket_access_control/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,11 @@
# Pre-test setup
- name: create a bucket
gcp_storage_bucket:
name: "{{ resource_name }}"
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
service_account_file: "{{ gcp_cred_file }}"
state: present
name: "{{ resource_name }}"
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
service_account_file: "{{ gcp_cred_file }}"
state: present
register: bucket
- name: delete a bucket access control
gcp_storage_bucket_access_control:
Expand Down Expand Up @@ -99,10 +99,10 @@
# If errors happen, don't crash the playbook!
- name: delete a bucket
gcp_storage_bucket:
name: "{{ resource_name }}"
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
service_account_file: "{{ gcp_cred_file }}"
state: absent
name: "{{ resource_name }}"
project: "{{ gcp_project }}"
auth_kind: "{{ gcp_cred_kind }}"
service_account_file: "{{ gcp_cred_file }}"
state: absent
register: bucket
ignore_errors: true

0 comments on commit 3496923

Please sign in to comment.