feedback from review

Author: Pedro Montiel

sumologic/data_source_sumologic_cse_log_mapping_vendor_product.go

@@ -41,7 +41,7 @@ func dataSourceCSELogMappingVendorAndProductRead(d *schema.ResourceData, meta in
 
 	if vendorAndProduct == nil {
 		d.SetId("")
-		return fmt.Errorf("Vendor product not found, removing from state: %v - %v", id, err)
+		return fmt.Errorf("Vendor product not found, removing from state: %v", id)
 	}
 
 	d.SetId(vendorAndProduct.GUID)

sumologic/data_source_sumologic_cse_log_mapping_vendor_product_test.go

@@ -0,0 +1,39 @@
+package sumologic
+
+import (
+	"testing"
+
+	"github.com/hashicorp/terraform-plugin-sdk/helper/resource"
+)
+
+func TestDataSourceCSELogMappingVendorProduct_basic(t *testing.T) {
+	resource.Test(t, resource.TestCase{
+		PreCheck:  func() { testAccPreCheck(t) },
+		Providers: testAccProviders,
+		Steps: []resource.TestStep{
+			{
+				Config: testDataSourceSumologicCSELogMappingVendorProduct,
+				Check: resource.ComposeTestCheckFunc(
+					testDataSourceSumologicCSELogMappingVendorProductCheck("data.sumologic_cse_log_mapping_vendor_product.web_gateway"),
+				),
+			},
+		},
+	})
+}
+
+func testDataSourceSumologicCSELogMappingVendorProductCheck(name string) resource.TestCheckFunc {
+	return resource.ComposeTestCheckFunc(
+		resource.TestCheckResourceAttrSet(name, "product"),
+		resource.TestCheckResourceAttr(name, "product", "Web Gateway"),
+		resource.TestCheckResourceAttrSet(name, "vendor"),
+		resource.TestCheckResourceAttr(name, "vendor", "McAfee"),
+		resource.TestCheckResourceAttrSet(name, "guid"),
+	)
+}
+
+var testDataSourceSumologicCSELogMappingVendorProduct = `
+data "sumologic_cse_log_mapping_vendor_product" "web_gateway" {
+  	product = "Web Gateway"
+	vendor = "McAfee"
+}
+`

sumologic/sumologic_client.go

@@ -28,15 +28,16 @@ type Client struct {
 var ProviderVersion string
 
 var endpoints = map[string]string{
-	"us1": "https://api.sumologic.com/api/",
-	"us2": "https://api.us2.sumologic.com/api/",
-	"fed": "https://api.fed.sumologic.com/api/",
-	"eu":  "https://api.eu.sumologic.com/api/",
-	"au":  "https://api.au.sumologic.com/api/",
-	"de":  "https://api.de.sumologic.com/api/",
-	"jp":  "https://api.jp.sumologic.com/api/",
-	"ca":  "https://api.ca.sumologic.com/api/",
-	"in":  "https://api.in.sumologic.com/api/",
+	"us1":  "https://api.sumologic.com/api/",
+	"us2":  "https://api.us2.sumologic.com/api/",
+	"fed":  "https://api.fed.sumologic.com/api/",
+	"eu":   "https://api.eu.sumologic.com/api/",
+	"au":   "https://api.au.sumologic.com/api/",
+	"de":   "https://api.de.sumologic.com/api/",
+	"jp":   "https://api.jp.sumologic.com/api/",
+	"ca":   "https://api.ca.sumologic.com/api/",
+	"in":   "https://api.in.sumologic.com/api/",
+	"nite": "https://nite-api.sumologic.net/api/",
 }
 
 var rateLimiter = time.NewTicker(time.Minute / 240)