Merge pull request #5113 from terraform-providers/b/fixing-tests

provider: ensuring the providers slice isn't a nil pointer

azurerm/provider_test.go

@@ -14,16 +14,19 @@ var testAccProvider *schema.Provider
 
 func init() {
 	azureProvider := Provider().(*schema.Provider)
-	supportedProviders := map[string]terraform.ResourceProvider{
-		"azurerm": testAccProvider,
-		"azuread": azuread.Provider().(*schema.Provider),
-	}
 
-	// TODO: these can be de-duped once this is relocated
 	testAccProvider = azureProvider
 	acceptance.AzureProvider = azureProvider
-	testAccProviders = supportedProviders
-	acceptance.SupportedProviders = supportedProviders
+
+	// NOTE: these /cannot/ be simplified into a single shared variable (tried, it causes a nil-slice)
+	testAccProviders = map[string]terraform.ResourceProvider{
+		"azurerm": testAccProvider,
+		"azuread": azuread.Provider().(*schema.Provider),
+	}
+	acceptance.SupportedProviders = map[string]terraform.ResourceProvider{
+		"azurerm": testAccProvider,
+		"azuread": azuread.Provider().(*schema.Provider),
+	}
 }
 
 func TestProvider(t *testing.T) {