Fixed issues after running int.tests.

hashicorp · Oct 26, 2018 · ed5043f · ed5043f
1 parent d7dc04c
commit ed5043f
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 6 deletions.
diff --git a/azurerm/data_source_api_management_api.go b/azurerm/data_source_api_management_api.go
@@ -76,7 +76,8 @@ func dataSourceApiManagementApi() *schema.Resource {
 			"revision": {
 				Type:     schema.TypeInt,
 				Optional: true,
-				Default:  1,
+				Default:  nil,
+				Computed: true,
 			},
 
 			"api_version": {

diff --git a/azurerm/data_source_api_management_api_test.go b/azurerm/data_source_api_management_api_test.go
@@ -31,8 +31,8 @@ func TestAccDataSourceAzureRMApiManagementApi_basic(t *testing.T) {
 					resource.TestCheckResourceAttr(dataSourceName, "subscription_key_parameter_names.#", "1"),
 					resource.TestCheckResourceAttr(dataSourceName, "subscription_key_parameter_names.0.header", "Ocp-Apim-Subscription-Key"),
 					resource.TestCheckResourceAttr(dataSourceName, "subscription_key_parameter_names.0.query", "subscription-key"),
-					resource.TestCheckResourceAttr(dataSourceName, "version", ""),
-					resource.TestCheckResourceAttr(dataSourceName, "version_set_id", ""),
+					resource.TestCheckResourceAttr(dataSourceName, "api_version", ""),
+					resource.TestCheckResourceAttr(dataSourceName, "api_version_set_id", ""),
 				),
 			},
 		},
@@ -64,6 +64,7 @@ resource "azurerm_api_management_api" "test" {
   name         = "acctestAMA-%d"
   service_name = "${azurerm_api_management.test.name}"
   path         = "api1"
+  protocols    = ["https"]
 
   import {
     content_value  = "${file("testdata/api_management_api_swagger.json")}"

diff --git a/azurerm/resource_arm_api_management_api.go b/azurerm/resource_arm_api_management_api.go
@@ -147,7 +147,8 @@ func resourceArmApiManagementApi() *schema.Resource {
 			"revision": {
 				Type:     schema.TypeInt,
 				Optional: true,
-				Default:  1,
+				Computed: true,
+				Default:  nil,
 			},
 
 			"version": {

diff --git a/azurerm/resource_arm_api_management_api_test.go b/azurerm/resource_arm_api_management_api_test.go
@@ -154,6 +154,7 @@ resource "azurerm_api_management_api" "test" {
   name         = "acctestAMA-%d"
   service_name = "${azurerm_api_management.test.name}"
   path         = "api1"
+  protocols    = ["https"]
 
   import {
     content_value  = "${file("testdata/api_management_api_swagger.json")}"
@@ -189,6 +190,7 @@ resource "azurerm_api_management_api" "test" {
   name         = "acctestAMA-%d"
   service_name = "${azurerm_api_management.test.name}"
   path         = "api1"
+  protocols    = ["http", "https"]
 
   import {
     content_value  = "${file("testdata/api_management_api_wsdl.xml")}"
@@ -207,8 +209,6 @@ resource "azurerm_api_management_api" "test" {
     query  = "test2"
   }
 
-  protocols = ["http", "https"]
-
   resource_group_name = "${azurerm_resource_group.test.name}"
 }
 `, rInt, location, rInt, rInt)