Fix #574 VLAN group returning multiple results (#741)

Author: rodvand

plugins/module_utils/netbox_utils.py

@@ -452,7 +452,7 @@
     "virtual_machine": set(["name", "cluster"]),
     "vm_bridge": set(["name"]),
     "vlan": set(["group", "name", "site", "tenant", "vid", "vlan_group"]),
-    "vlan_group": set(["slug", "site", "scope"]),
+    "vlan_group": set(["name", "slug", "site", "scope"]),
     "vrf": set(["name", "tenant"]),
     "webhook": set(["name"]),
     "wireless_lan": set(["ssid"]),
@@ -853,7 +853,11 @@ def _build_query_params(
                     query_id = self._get_query_param_id(match, child)
                 else:
                     query_id = self._get_query_param_id(match, module_data)
-                query_dict.update({match + "_id": query_id})
+
+                if parent == "vlan_group" and match == "site":
+                    query_dict.update({match: query_id})
+                else:
+                    query_dict.update({match + "_id": query_id})
             else:
                 if child:
                     value = child.get(match)

tests/unit/module_utils/test_data/build_query_params_child/data.json

@@ -176,8 +176,9 @@
             "site": "Test Site"
         },
         "expected": {
+            "name": "Test VLAN Group",
             "slug": "test-vlan-group",
-            "site_id": 1
+            "site": 1
         }
     },
     {