azurerm_route / azurerm_route_table - additions

[tombuildsstuff]

• azurerm_route_table

  • switching over to using the enum values in the Azure SDK for Go, rather than our own hard-coded strings
  • refactoring the resource to avoid a potential crash
  • migrating Route over from a Set to a List

• azurerm_route

  • switching over to using the enum values in the Azure SDK for Go, rather than our own hard-coded strings
  • refactoring the resource to avoid a potential crash
  • updating the documentation
  • fixing the sidebar highlight

[tombuildsstuff]

Tests pass:

 $ acctests azurerm TestAccAzureRMRoute_
=== RUN   TestAccAzureRMRoute_importBasic
--- PASS: TestAccAzureRMRoute_importBasic (103.64s)
=== RUN   TestAccAzureRMRoute_basic
--- PASS: TestAccAzureRMRoute_basic (99.84s)
=== RUN   TestAccAzureRMRoute_disappears
--- PASS: TestAccAzureRMRoute_disappears (101.34s)
=== RUN   TestAccAzureRMRoute_multipleRoutes
--- PASS: TestAccAzureRMRoute_multipleRoutes (128.76s)
PASS
ok  	github.com/terraform-providers/terraform-provider-azurerm/azurerm	433.614s

[tombuildsstuff]

Route Table tests:

$ acctests azurerm TestAccAzureRMRouteTable_
=== RUN   TestAccAzureRMRouteTable_importBasic
--- PASS: TestAccAzureRMRouteTable_importBasic (81.93s)
=== RUN   TestAccAzureRMRouteTable_importSingleRoute
--- PASS: TestAccAzureRMRouteTable_importSingleRoute (80.34s)
=== RUN   TestAccAzureRMRouteTable_importMultipleRoutes
--- PASS: TestAccAzureRMRouteTable_importMultipleRoutes (79.26s)
=== RUN   TestAccAzureRMRouteTable_basic
--- PASS: TestAccAzureRMRouteTable_basic (78.18s)
=== RUN   TestAccAzureRMRouteTable_singleRoute
--- PASS: TestAccAzureRMRouteTable_singleRoute (79.23s)
=== RUN   TestAccAzureRMRouteTable_disappears
--- PASS: TestAccAzureRMRouteTable_disappears (78.33s)
=== RUN   TestAccAzureRMRouteTable_withTags
--- PASS: TestAccAzureRMRouteTable_withTags (86.08s)
=== RUN   TestAccAzureRMRouteTable_multipleRoutes
--- PASS: TestAccAzureRMRouteTable_multipleRoutes (80.78s)
PASS
ok  	github.com/terraform-providers/terraform-provider-azurerm/azurerm	644.161s

[mbfrahry]

LGTM!

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!