[dataSourceNetboxVlans] - Add netbox_vlan.id in netbox_vlans.vlans DataSources (#662)

* update(dataSourceNetboxVlans): Add netbox_vlan.id info in return info of netbox_vlans.vlans

* update(dataSourceNetboxVlans): Add documentation

* feature(test): Add missing test

Author: rypti

docs/data-sources/vlans.md

@@ -41,6 +41,7 @@ Read-Only:
 
 - `description` (String)
 - `group_id` (Number)
+- `id` (Number)
 - `name` (String)
 - `role` (Number)
 - `site` (Number)

netbox/data_source_netbox_vlans.go

@@ -43,6 +43,10 @@ func dataSourceNetboxVlans() *schema.Resource {
 				Computed: true,
 				Elem: &schema.Resource{
 					Schema: map[string]*schema.Schema{
+						"id": {
+							Type:     schema.TypeInt,
+							Computed: true,
+						},
 						"vid": {
 							Type:     schema.TypeInt,
 							Computed: true,
@@ -200,6 +204,7 @@ func dataSourceNetboxVlansRead(d *schema.ResourceData, m interface{}) error {
 	for _, v := range filteredVlans {
 		var mapping = make(map[string]interface{})
 
+		mapping["id"] = v.ID
 		mapping["vid"] = v.Vid
 		mapping["name"] = v.Name
 		mapping["description"] = v.Description

netbox/data_source_netbox_vlans_test.go

@@ -72,6 +72,13 @@ func TestAccNetboxVlansDataSource_basic(t *testing.T) {
 					resource.TestCheckResourceAttr("netbox_vlan.test_1", "vid", "1234"),
 				),
 			},
+			{
+				Config: setUp + testAccNetboxVlansByVid(),
+				Check: resource.ComposeTestCheckFunc(
+					resource.TestCheckResourceAttr("data.netbox_vlans.test", "vlans.#", "1"),
+					resource.TestCheckResourceAttrPair("data.netbox_vlans.test", "vlans.0.id", "netbox_vlan.test_1", "id"),
+				),
+			},
 			{
 				Config: setUp + testAccNetboxVlansByVid(),
 				Check: resource.ComposeTestCheckFunc(