test: improve load_balancer_service integration using new framework (#546)

SUMMARY

Use the new testing framework for the load_balancer_service integration
tests.

Depends on #545

.ansible-lint

@@ -8,7 +8,6 @@ exclude_paths:
   - examples/
   - tests/integration/targets/certificate
   - tests/integration/targets/floating_ip
-  - tests/integration/targets/load_balancer_service
   - tests/integration/targets/load_balancer_target
   - tests/integration/targets/placement_group
   - tests/integration/targets/primary_ip

tests/integration/targets/load_balancer_service/tasks/cleanup.yml

@@ -0,0 +1,5 @@
+---
+- name: Cleanup test_load_balancer
+  hetzner.hcloud.load_balancer:
+    name: "{{ hcloud_load_balancer_name }}"
+    state: absent

tests/integration/targets/load_balancer_service/tasks/prepare.yml

@@ -0,0 +1,7 @@
+---
+- name: Create test_load_balancer
+  hetzner.hcloud.load_balancer:
+    name: "{{ hcloud_load_balancer_name }}"
+    load_balancer_type: lb11
+    location: "{{ hcloud_location_name }}"
+  register: test_load_balancer

tests/integration/targets/load_balancer_service/tasks/test.yml

@@ -1,126 +1,155 @@
 # Copyright: (c) 2019, Hetzner Cloud GmbH <info@hetzner-cloud.de>
 # GNU General Public License v3.0+ (see COPYING or https://www.gnu.org/licenses/gpl-3.0.txt)
 ---
-- name: setup load_balancer
+- name: Test missing required parameters
-  hetzner.hcloud.load_balancer:
-    name: "{{hcloud_load_balancer_name}}"
-    load_balancer_type: lb11
-    state: present
-    location: "{{ hcloud_location_name }}"
-  register: load_balancer
-- name: verify setup load_balancer
-  assert:
-    that:
-      - load_balancer is success
-
-- name: test fail load balancer does not exist
   hetzner.hcloud.load_balancer_service:
-    load_balancer: does-not-exist
-    protocol: http
-    listen_port: 80
     state: present
-  register: result
   ignore_errors: true
-- name: verify test fail load_balancer does not exist
+  register: result
-  assert:
+- name: Verify missing required parameters
+  ansible.builtin.assert:
     that:
       - result is failed
-      - "result.msg == 'resource (load_balancer) does not exist: does-not-exist'"
+      - 'result.msg == "missing required arguments: listen_port, load_balancer"'
 
-- name: test create load_balancer service with checkmode
+- name: Test create with checkmode
   hetzner.hcloud.load_balancer_service:
-    load_balancer: "{{hcloud_load_balancer_name}}"
+    load_balancer: "{{ hcloud_load_balancer_name }}"
-    protocol: "http"
     listen_port: 80
+    protocol: http
     state: present
-  register: result
   check_mode: true
-- name: verify test create load_balancer service with checkmode
+  register: result
-  assert:
+- name: Verify create with checkmode
+  ansible.builtin.assert:
     that:
       - result is changed
 
-- name: test create load_balancer service
+- name: Test create
   hetzner.hcloud.load_balancer_service:
-    load_balancer: "{{hcloud_load_balancer_name}}"
+    load_balancer: "{{ hcloud_load_balancer_name }}"
-    protocol: "http"
     listen_port: 80
+    destination_port: 80
+    protocol: tcp
     state: present
-  register: load_balancer_service
+  register: result
-- name: verify create load_balancer service
+- name: Verify create
-  assert:
+  ansible.builtin.assert:
     that:
-      - load_balancer_service is changed
+      - result is changed
-      - load_balancer_service.hcloud_load_balancer_service.protocol == "http"
+      - result.hcloud_load_balancer_service.load_balancer == hcloud_load_balancer_name
-      - load_balancer_service.hcloud_load_balancer_service.listen_port == 80
+      - result.hcloud_load_balancer_service.protocol == "tcp"
-      - load_balancer_service.hcloud_load_balancer_service.destination_port == 80
+      - result.hcloud_load_balancer_service.listen_port == 80
-      - load_balancer_service.hcloud_load_balancer_service.proxyprotocol is sameas false
+      - result.hcloud_load_balancer_service.destination_port == 80
+      - result.hcloud_load_balancer_service.proxyprotocol is false
+      - result.hcloud_load_balancer_service.health_check.protocol == "tcp"
+      - result.hcloud_load_balancer_service.health_check.port == 80
+      - result.hcloud_load_balancer_service.health_check.interval == 15
+      - result.hcloud_load_balancer_service.health_check.retries == 3
+      - result.hcloud_load_balancer_service.health_check.timeout == 10
+      - result.hcloud_load_balancer_service.http == None
 
-- name: test create load_balancer service idempotency
+- name: Test create idempotency
   hetzner.hcloud.load_balancer_service:
-    load_balancer: "{{hcloud_load_balancer_name}}"
+    load_balancer: "{{ hcloud_load_balancer_name }}"
-    protocol: "http"
     listen_port: 80
+    destination_port: 80
+    protocol: tcp
     state: present
-  register: load_balancer_service
+  register: result
-- name: verify create load_balancer service idempotency
+- name: Verify create idempotency
-  assert:
+  ansible.builtin.assert:
     that:
-      - load_balancer_service is not changed
+      - result is not changed
 
-- name: test update load_balancer service
+- name: Test create with not existing load_balancer
   hetzner.hcloud.load_balancer_service:
-    load_balancer: "{{hcloud_load_balancer_name}}"
+    load_balancer: not-existing
-    protocol: "tcp"
     listen_port: 80
+    protocol: http
     state: present
-  register: load_balancer_service
+  ignore_errors: true
-- name: verify create load_balancer service
+  register: result
-  assert:
+- name: Verify create with not existing load_balancer
+  ansible.builtin.assert:
     that:
-      - load_balancer_service is changed
+      - result is failed
-      - load_balancer_service.hcloud_load_balancer_service.protocol == "tcp"
+      - 'result.msg == "resource (load_balancer) does not exist: not-existing"'
-      - load_balancer_service.hcloud_load_balancer_service.listen_port == 80
-      - load_balancer_service.hcloud_load_balancer_service.destination_port == 80
-      - load_balancer_service.hcloud_load_balancer_service.proxyprotocol is sameas false
 
-- name: test absent load_balancer service
+- name: Test update
   hetzner.hcloud.load_balancer_service:
-    load_balancer: "{{hcloud_load_balancer_name}}"
+    load_balancer: "{{ hcloud_load_balancer_name }}"
-    protocol: "http"
+    listen_port: 80
+    protocol: http
+    state: present
+  register: result
+- name: Verify update
+  ansible.builtin.assert:
+    that:
+      - result is changed
+      - result.hcloud_load_balancer_service.load_balancer == hcloud_load_balancer_name
+      - result.hcloud_load_balancer_service.protocol == "http"
+      - result.hcloud_load_balancer_service.listen_port == 80
+      - result.hcloud_load_balancer_service.destination_port == 80
+      - result.hcloud_load_balancer_service.proxyprotocol is false
+      - result.hcloud_load_balancer_service.health_check.protocol == "tcp"
+      - result.hcloud_load_balancer_service.health_check.port == 80
+      - result.hcloud_load_balancer_service.health_check.interval == 15
+      - result.hcloud_load_balancer_service.health_check.retries == 3
+      - result.hcloud_load_balancer_service.health_check.timeout == 10
+      - result.hcloud_load_balancer_service.http != None
+
+- name: Test delete
+  hetzner.hcloud.load_balancer_service:
+    load_balancer: "{{ hcloud_load_balancer_name }}"
     listen_port: 80
     state: absent
   register: result
-- name: verify test absent load_balancer service
+- name: Verify delete
-  assert:
+  ansible.builtin.assert:
     that:
       - result is changed
 
-- name: test create load_balancer service with http
+- name: Test create with http
   hetzner.hcloud.load_balancer_service:
-    load_balancer: "{{hcloud_load_balancer_name}}"
+    load_balancer: "{{ hcloud_load_balancer_name }}"
-    protocol: "http"
+    protocol: http
     listen_port: 80
     http:
-      cookie_name: "Test"
+      cookie_name: keks
       sticky_sessions: true
     state: present
-  register: load_balancer_service
+  register: result
-- name: verify create load_balancer service
+- name: Verify create with http
-  assert:
+  ansible.builtin.assert:
     that:
-      - load_balancer_service is changed
+      - result is changed
-      - load_balancer_service.hcloud_load_balancer_service.protocol == "http"
+      - result.hcloud_load_balancer_service.load_balancer == hcloud_load_balancer_name
-      - load_balancer_service.hcloud_load_balancer_service.listen_port == 80
+      - result.hcloud_load_balancer_service.listen_port == 80
-      - load_balancer_service.hcloud_load_balancer_service.destination_port == 80
+      - result.hcloud_load_balancer_service.destination_port == 80
-      - load_balancer_service.hcloud_load_balancer_service.proxyprotocol is sameas false
+      - result.hcloud_load_balancer_service.protocol == "http"
+      - result.hcloud_load_balancer_service.proxyprotocol is false
+      - result.hcloud_load_balancer_service.http.certificates == []
+      - result.hcloud_load_balancer_service.http.cookie_name == "keks"
+      - result.hcloud_load_balancer_service.http.cookie_lifetime == 300
+      - result.hcloud_load_balancer_service.http.redirect_http == false
+      - result.hcloud_load_balancer_service.http.sticky_sessions == true
+      - result.hcloud_load_balancer_service.health_check.protocol == "http"
+      - result.hcloud_load_balancer_service.health_check.port == 80
+      - result.hcloud_load_balancer_service.health_check.interval == 15
+      - result.hcloud_load_balancer_service.health_check.retries == 3
+      - result.hcloud_load_balancer_service.health_check.timeout == 10
+      - result.hcloud_load_balancer_service.health_check.http.domain == ""
+      - result.hcloud_load_balancer_service.health_check.http.path == "/"
+      - result.hcloud_load_balancer_service.health_check.http.status_codes == ["2??", "3??"]
 
-- name: cleanup load_balancer
+- name: Test delete with http
-  hetzner.hcloud.load_balancer:
+  hetzner.hcloud.load_balancer_service:
-    name: "{{ hcloud_load_balancer_name }}"
+    load_balancer: "{{ hcloud_load_balancer_name }}"
+    listen_port: 80
     state: absent
   register: result
-- name: verify cleanup load_balancer
+- name: Verify delete with http
-  assert:
+  ansible.builtin.assert:
     that:
-      - result is success
+      - result is changed