feature/no_dot_var - Replace "." in columns name to "_"

changelogs/fragments/no_dot_var.yml

@@ -0,0 +1,3 @@
+---
+minor_changes:
+  - replace "." in reference field colums to "_"

plugins/inventory/now.py

@@ -443,7 +443,7 @@ def set_hostvars(self, host, record, columns):
             )
 
         for k in columns:
-            self.inventory.set_variable(host, k, record[k])
+            self.inventory.set_variable(host, k.replace('.','_'), record[k])
 
     def fill_constructed(
         self,

tests/integration/targets/inventory/files/no_dot_var.now.yml

@@ -0,0 +1,17 @@
+---
+plugin: servicenow.itsm.now
+table: cmdb_ci_server
+strict: false
+
+instance:
+  timeout: 1200
+
+columns:
+  - location.name
+  - support_group.name
+
+compose:
+  ansible_host: ip_address
+
+sysparm_query: nameSTARTSWITHmy-vm-
+...

tests/integration/targets/inventory/playbooks/no_dot_var.yml

@@ -0,0 +1,44 @@
+---
+- name: Test sysparm_limit option
+  hosts: localhost
+  gather_facts: false
+
+  tasks:
+    - block:
+        - name: Create imaginary VMs
+          servicenow.itsm.configuration_item:
+            name: my-vm-{{ item }}
+            sys_class_name: cmdb_ci_server
+            ip_address: 10.1.0.{{ item }}
+            other:
+              state: "{{ (item % 4 == 0) | ansible.builtin.ternary('on', 'off') }}"
+              guest_os_fullname: "{{ (item < 105) | ansible.builtin.ternary('OS0', 'OS1') }}"
+          loop: "{{ range(100, 110) }}"
+          register: vms
+
+        - name: Reload inventory
+          ansible.builtin.meta: refresh_inventory
+
+        - ansible.builtin.debug:
+            var: groups
+
+        - ansible.builtin.debug:
+            var: hostvars["my-vm-100"]
+
+        - name: Verify the replaced variable names exist
+          ansible.builtin.assert:
+            that:
+              - groups["all"] | length == 10
+              - hostvars["my-vm-100"].location_name is defined
+              - hostvars["my-vm-100"].support_group_name is defined
+              - hostvars["my-vm-104"].location_name is defined
+              - hostvars["my-vm-104"].support_group_name is defined
+
+      always:
+        - name: Delete VMs
+          servicenow.itsm.configuration_item:
+            state: absent
+            sys_id: "{{ item.record.sys_id }}"
+          loop: "{{ vms.results }}"
+          loop_control:
+            label: "{{ item.record.name }}"