Skip to content

Commit

Permalink
No empty strings on upstream patterns
Browse files Browse the repository at this point in the history
  • Loading branch information
Brunhil committed Feb 12, 2021
1 parent 6b62c54 commit 9c0946f
Showing 1 changed file with 8 additions and 5 deletions.
13 changes: 8 additions & 5 deletions azurerm/internal/services/signalr/signalr_service_resource.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ import (
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/helpers/tf"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/signalr/parse"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/signalr/validate"
signalrValidate "github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/signalr/validate"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/tags"
azSchema "github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/tf/schema"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/timeouts"
Expand Down Expand Up @@ -111,30 +111,33 @@ func resourceArmSignalRService() *schema.Resource {
Type: schema.TypeList,
Required: true,
Elem: &schema.Schema{
Type: schema.TypeString,
Type: schema.TypeString,
ValidateFunc: validation.StringIsNotEmpty,
},
},

"event_pattern": {
Type: schema.TypeList,
Required: true,
Elem: &schema.Schema{
Type: schema.TypeString,
Type: schema.TypeString,
ValidateFunc: validation.StringIsNotEmpty,
},
},

"hub_pattern": {
Type: schema.TypeList,
Required: true,
Elem: &schema.Schema{
Type: schema.TypeString,
Type: schema.TypeString,
ValidateFunc: validation.StringIsNotEmpty,
},
},

"url_template": {
Type: schema.TypeString,
Required: true,
ValidateFunc: validate.UrlTemplate,
ValidateFunc: signalrValidate.UrlTemplate,
},
},
},
Expand Down

0 comments on commit 9c0946f

Please sign in to comment.