From c9df11e32f6988eb6ded2fdd275d09b72ad30a56 Mon Sep 17 00:00:00 2001
From: Paul Stack <public@paulstack.co.uk>
Date: Fri, 17 Mar 2017 11:06:26 +0000
Subject: [PATCH] provider/pagerduty: Remove legacy pagerduty vendor datasource
 acctest (#12803)

Removes the test for datasource vendor name_prefix as that field has now
been removed
---
 .../data_source_pagerduty_vendor_test.go      | 50 +------------------
 1 file changed, 1 insertion(+), 49 deletions(-)

diff --git a/builtin/providers/pagerduty/data_source_pagerduty_vendor_test.go b/builtin/providers/pagerduty/data_source_pagerduty_vendor_test.go
index c70d54ef1..59b639aa2 100644
--- a/builtin/providers/pagerduty/data_source_pagerduty_vendor_test.go
+++ b/builtin/providers/pagerduty/data_source_pagerduty_vendor_test.go
@@ -14,7 +14,7 @@ func TestAccDataSourcePagerDutyVendor_Basic(t *testing.T) {
 		Providers:    testAccProviders,
 		CheckDestroy: testAccCheckPagerDutyScheduleDestroy,
 		Steps: []resource.TestStep{
-			resource.TestStep{
+			{
 				Config: testAccDataSourcePagerDutyVendorConfig,
 				Check: resource.ComposeTestCheckFunc(
 					testAccDataSourcePagerDutyVendor("data.pagerduty_vendor.foo"),
@@ -24,22 +24,6 @@ func TestAccDataSourcePagerDutyVendor_Basic(t *testing.T) {
 	})
 }
 
-func TestAccDataSourcePagerDutyVendorLegacy_Basic(t *testing.T) {
-	resource.Test(t, resource.TestCase{
-		PreCheck:     func() { testAccPreCheck(t) },
-		Providers:    testAccProviders,
-		CheckDestroy: testAccCheckPagerDutyScheduleDestroy,
-		Steps: []resource.TestStep{
-			resource.TestStep{
-				Config: testAccDataSourcePagerDutyVendorLegacyConfig,
-				Check: resource.ComposeTestCheckFunc(
-					testAccDataSourcePagerDutyVendorLegacy("data.pagerduty_vendor.foo"),
-				),
-			},
-		},
-	})
-}
-
 func testAccDataSourcePagerDutyVendor(n string) resource.TestCheckFunc {
 	return func(s *terraform.State) error {
 
@@ -66,40 +50,8 @@ func testAccDataSourcePagerDutyVendor(n string) resource.TestCheckFunc {
 	}
 }
 
-func testAccDataSourcePagerDutyVendorLegacy(n string) resource.TestCheckFunc {
-	return func(s *terraform.State) error {
-
-		r := s.RootModule().Resources[n]
-		a := r.Primary.Attributes
-
-		if a["id"] == "" {
-			return fmt.Errorf("Expected to get a vendor ID from PagerDuty")
-		}
-
-		if a["id"] != "PAM4FGS" {
-			return fmt.Errorf("Expected the Datadog Vendor ID to be: PAM4FGS, but got: %s", a["id"])
-		}
-
-		if a["name"] != "Datadog" {
-			return fmt.Errorf("Expected the Datadog Vendor Name to be: Datadog, but got: %s", a["name"])
-		}
-
-		if a["type"] != "generic_events_api_inbound_integration" {
-			return fmt.Errorf("Expected the Datadog Vendor Type to be: generic_events_api_inbound_integration, but got: %s", a["type"])
-		}
-
-		return nil
-	}
-}
-
 const testAccDataSourcePagerDutyVendorConfig = `
 data "pagerduty_vendor" "foo" {
   name = "cloudwatch"
 }
 `
-
-const testAccDataSourcePagerDutyVendorLegacyConfig = `
-data "pagerduty_vendor" "foo" {
-  name_regex = "Datadog"
-}
-`