Added validation for threshold_type

Author: kumar-avijit

sumologic/resource_sumologic_monitors_library_monitor.go

@@ -5,6 +5,7 @@ import (
 	"strings"
 
 	"github.com/hashicorp/terraform-plugin-sdk/helper/schema"
+	"github.com/hashicorp/terraform-plugin-sdk/helper/validation"
 )
 
 func resourceSumologicMonitorsLibraryMonitor() *schema.Resource {
@@ -96,8 +97,9 @@ func resourceSumologicMonitorsLibraryMonitor() *schema.Resource {
 							Optional: true,
 						},
 						"threshold_type": {
-							Type:     schema.TypeString,
-							Optional: true,
+							Type:         schema.TypeString,
+							Optional:     true,
+							ValidateFunc: validation.StringInSlice([]string{"LessThan", "LessThanOrEqual", "GreaterThan", "GreaterThanOrEqual"}, false),
 						},
 						"time_range": {
 							Type:     schema.TypeString,

sumologic/resource_sumologic_monitors_library_monitor_test.go

@@ -2,6 +2,7 @@ package sumologic
 
 import (
 	"fmt"
+	"regexp"
 	"strconv"
 	"strings"
 	"testing"
@@ -11,6 +12,31 @@ import (
 	"github.com/hashicorp/terraform-plugin-sdk/terraform"
 )
 
+func TestAccSumologicMonitorsLibraryMonitor_schemaValidations(t *testing.T) {
+	var monitorsLibraryMonitor MonitorsLibraryMonitor
+	config := `
+       resource "sumologic_monitor" "test" { 
+         name = "test"
+         type = "MonitorsLibraryMonitor"
+         monitor_type = "Logs"
+         triggers {
+           threshold_type = "foo"
+         }
+       }`
+	expectedError := regexp.MustCompile(".*expected triggers.0.threshold_type to be one of \\[LessThan LessThanOrEqual GreaterThan GreaterThanOrEqual\\], got foo.*")
+	resource.Test(t, resource.TestCase{
+		Providers: testAccProviders,
+		CheckDestroy: testAccCheckMonitorsLibraryMonitorDestroy(monitorsLibraryMonitor),
+		Steps: []resource.TestStep{
+			{
+				Config:      config,
+				PlanOnly:    true,
+				ExpectError: expectedError,
+			},
+		},
+	})
+}
+
 func TestAccSumologicMonitorsLibraryMonitor_basic(t *testing.T) {
 	var monitorsLibraryMonitor MonitorsLibraryMonitor
 	testNameSuffix := acctest.RandString(16)