Similar DELETE test for all new SQLA resources

Author: pietern

sqlanalytics/resource_query_test.go

@@ -383,7 +383,5 @@ func TestQueryDelete(t *testing.T) {
 	}.Apply(t)
 
 	assert.NoError(t, err, err)
-
-	// Delete doesn't touch schema.ResourceData, so the ID should survive.
-	assert.Equal(t, "foo", d.Id())
+	assert.Equal(t, "foo", d.Id(), "Resource ID should not be empty")
 }

sqlanalytics/resource_visualization_test.go

@@ -238,7 +238,5 @@ func TestVisualizationDelete(t *testing.T) {
 	}.Apply(t)
 
 	assert.NoError(t, err, err)
-
-	// Delete doesn't touch schema.ResourceData, so the ID should survive.
-	assert.Equal(t, "12345", d.Id())
+	assert.Equal(t, "12345", d.Id(), "Resource ID should not be empty")
 }

sqlanalytics/resource_widget_test.go

@@ -152,3 +152,20 @@ func TestWidgetCreateWithParamValue(t *testing.T) {
 	assert.Equal(t, "12345", d.Id(), "Resource ID should not be empty")
 	assert.Equal(t, "678", d.Get("visualization_id"))
 }
+
+func TestWidgetDelete(t *testing.T) {
+	d, err := qa.ResourceFixture{
+		Fixtures: []qa.HTTPFixture{
+			{
+				Method:   "DELETE",
+				Resource: "/api/2.0/preview/sql/widgets/12345",
+			},
+		},
+		Resource: ResourceWidget(),
+		Delete:   true,
+		ID:       "12345",
+	}.Apply(t)
+
+	assert.NoError(t, err, err)
+	assert.Equal(t, "12345", d.Id(), "Resource ID should not be empty")
+}