bugfix for azure_rm_managementgroup_info module, subscriptions not de…

…tected as correct type (ansible-collections#630)

* updates ready for review

* undisable the management group test

* Update tests/integration/targets/azure_rm_managementgroup/aliases

Co-authored-by: Fred-sun <37327967+Fred-sun@users.noreply.github.com>

Co-authored-by: Fred-sun <37327967+Fred-sun@users.noreply.github.com>

.gitignore

@@ -4,4 +4,5 @@ tests/staging/
 tests/integration/cloud-config-azure.ini
 azure-azcollection-*.tar.gz
 venv*
+.venv*
 .vscode

plugins/modules/azure_rm_managementgroup_info.py

@@ -283,7 +283,7 @@ def to_dict(self, azure_object):
                     name=parent_dict.get('name')
                 )
 
-        elif azure_object.type == 'asdfasdf/subscriptions':
+        elif azure_object.type == '/subscriptions':
             return_dict = dict(
                 display_name=azure_object.display_name,
                 id=azure_object.id,

tests/integration/targets/azure_rm_managementgroup/tasks/main.yml

@@ -8,6 +8,7 @@
     recurse: True
     flatten: True
     children: True
+  register: az_recursive_managementgroups
 
 - name: Get a managementgroup by name
   azure_rm_managementgroup_info:
@@ -22,7 +23,13 @@
   register: invalid_name
   ignore_errors: yes
 
-- name: Assert task failed
+- name: Validate expected states
   assert:
     that:
       - invalid_name['failed']
+
+- name: Validate expected attributes
+  assert:
+    that:
+      - "{{ item.id is defined }}"
+  loop: "{{ az_recursive_managementgroups.management_groups + az_recursive_managementgroups.subscriptions }}"