fix instance tests

ngine-io · Nov 26, 2024 · f0183b2 · f0183b2
1 parent 5413316
commit f0183b2
Show file tree

Hide file tree

Showing 6 changed files with 97 additions and 93 deletions.
diff --git a/plugins/modules/instance.py b/plugins/modules/instance.py
@@ -1093,6 +1093,10 @@ def restore_instance(self):
     def get_result(self, resource):
         super(AnsibleCloudStackInstance, self).get_result(resource)
         if resource:
+            # 4.18 does not return keypairs as list as doc claims
+            if "ssh_keys" in self.result and not isinstance("list", self.result["ssh_keys"]):
+                self.result["ssh_keys"] = [self.result["ssh_keys"]]
+
             self.result["user_data"] = self._get_instance_user_data(resource)
             if "securitygroup" in resource:
                 security_groups = []

diff --git a/tests/integration/targets/instance/tasks/host.yml b/tests/integration/targets/instance/tasks/host.yml
@@ -48,7 +48,7 @@
   assert:
     that:
       - instance is changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.host is not defined
       - instance.state == "Stopped"
 
@@ -63,7 +63,7 @@
   assert:
     that:
       - instance is changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.host == host.name
       - instance.state == "Running"
 
@@ -78,7 +78,7 @@
   assert:
     that:
       - instance is not changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.host == host.name
       - instance.state == "Running"
 
@@ -101,7 +101,7 @@
   assert:
     that:
       - instance is changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.host != host.name
       - instance.state == "Running"
 
@@ -116,7 +116,7 @@
   assert:
     that:
       - instance is changed
-      - instance.name cs_resource_prefix + "-vm-" + instance_number
+      - instance.name cs_resource_prefix + "-vm-" + instance_number|string
       - instance.host == host.name
       - instance.state == "Running"
 
@@ -131,7 +131,7 @@
   assert:
     that:
       - instance is not changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
-      - instance.display_name == cs_resource_prefix + "-display-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
+      - instance.display_name == cs_resource_prefix + "-display-" + instance_number|string
       - instance.host == host.name
       - instance.state == "Running"
diff --git a/tests/integration/targets/instance/tasks/present.yml b/tests/integration/targets/instance/tasks/present.yml
@@ -47,8 +47,8 @@
   assert:
     that:
       - instance is changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
-      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
+      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_1
       - instance.state == "Running"
       - instance.ssh_key == cs_resource_prefix + "-sshkey"
@@ -74,8 +74,8 @@
   assert:
     that:
       - instance is not changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
-      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
+      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_1
       - instance.state == "Running"
       - instance.ssh_key == cs_resource_prefix + "-sshkey"
@@ -96,8 +96,8 @@
   assert:
     that:
       - instance is not changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
-      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
+      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_1
       - instance.state == "Running"
 
@@ -111,8 +111,8 @@
   assert:
     that:
       - instance is not changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
-      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
+      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_1
       - instance.state == "Running"
 
@@ -127,8 +127,8 @@
   assert:
     that:
       - instance is changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
-      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
+      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_1
       - instance.state == "Running"
 
@@ -142,8 +142,8 @@
   assert:
     that:
       - instance is changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
-      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
+      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_1
       - instance.state == "Stopped"
 
@@ -171,8 +171,8 @@
   assert:
     that:
       - instance is changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
-      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
+      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_1
       - instance.state == "Stopped"
 
@@ -187,8 +187,8 @@
   assert:
     that:
       - instance is changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
-      - instance.display_name == cs_resource_prefix + "-display-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
+      - instance.display_name == cs_resource_prefix + "-display-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_2
       - instance.state == "Stopped"
 
@@ -203,7 +203,7 @@
   assert:
     that:
       - instance is not changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.display_name == "{{ cs_resource_prefix }}-display-{{ instance_number }}"
       - instance.service_offering == test_cs_instance_offering_2
       - instance.state == "Stopped"
@@ -218,8 +218,8 @@
   assert:
     that:
       - instance is changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
-      - instance.display_name == cs_resource_prefix + "-display-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
+      - instance.display_name == cs_resource_prefix + "-display-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_2
       - instance.state == "Running"
 
@@ -233,8 +233,8 @@
   assert:
     that:
       - instance is not changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
-      - instance.display_name == cs_resource_prefix + "-display-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
+      - instance.display_name == cs_resource_prefix + "-display-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_2
       - instance.state == "Running"
 
@@ -250,8 +250,8 @@
   assert:
     that:
       - instance is changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
-      - instance.display_name == cs_resource_prefix + "-display-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
+      - instance.display_name == cs_resource_prefix + "-display-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_2
       - instance.state == "Running"
 
@@ -266,8 +266,8 @@
   assert:
     that:
       - instance is changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
-      - instance.display_name == cs_resource_prefix + "-display-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
+      - instance.display_name == cs_resource_prefix + "-display-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_1
       - instance.state == "Running"
 
@@ -282,8 +282,8 @@
   assert:
     that:
       - instance is not changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
-      - instance.display_name == cs_resource_prefix + "-display-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
+      - instance.display_name == cs_resource_prefix + "-display-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_1
       - instance.state == "Running"
 
@@ -299,8 +299,8 @@
   assert:
     that:
       - instance is changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
-      - instance.display_name == cs_resource_prefix + "-display-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
+      - instance.display_name == cs_resource_prefix + "-display-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_1
 
 - name: test restore instance
@@ -314,6 +314,6 @@
   assert:
     that:
       - instance is changed
-      - instance.name == cs_resource_prefix + "-vm-" + instance_number
-      - instance.display_name == cs_resource_prefix + "-display-" + instance_number
+      - instance.name == cs_resource_prefix + "-vm-" + instance_number|string
+      - instance.display_name == cs_resource_prefix + "-display-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_1
diff --git a/tests/integration/targets/instance/tasks/present_display_name.yml b/tests/integration/targets/instance/tasks/present_display_name.yml
@@ -24,7 +24,7 @@
   assert:
     that:
       - instance is changed
-      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_1
       - instance.state == "Running"
       - instance.ssh_key == cs_resource_prefix + "-sshkey"
@@ -45,7 +45,7 @@
   assert:
     that:
       - instance is not changed
-      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_1
       - instance.state == "Running"
       - instance.ssh_key == cs_resource_prefix + "-sshkey"
@@ -61,7 +61,7 @@
   assert:
     that:
       - instance is not changed
-      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_1
       - instance.state == "Running"
 
@@ -75,7 +75,7 @@
   assert:
     that:
       - instance is changed
-      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_1
       - instance.state == "Stopped"
 
@@ -101,7 +101,7 @@
   assert:
     that:
       - instance is changed
-      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.service_offering == "{{ test_cs_instance_offering_2 }}"
       - instance.state == "Stopped"
 
@@ -115,7 +115,7 @@
   assert:
     that:
       - instance is changed
-      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.service_offering == "{{ test_cs_instance_offering_2 }}"
       - instance.state == "Running"
 
@@ -129,7 +129,7 @@
   assert:
     that:
       - instance is not changed
-      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.service_offering == "{{ test_cs_instance_offering_2 }}"
       - instance.state == "Running"
 
@@ -144,7 +144,7 @@
   assert:
     that:
       - instance is changed
-      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_1
       - instance.state == "Running"
 
@@ -159,7 +159,7 @@
   assert:
     that:
       - instance is not changed
-      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_1
       - instance.state == "Running"
 
@@ -174,5 +174,5 @@
   assert:
     that:
       - instance is changed
-      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number
+      - instance.display_name == cs_resource_prefix + "-vm-" + instance_number|string
       - instance.service_offering == test_cs_instance_offering_1