improve big query job error messages (GoogleCloudPlatform#12157)

Author: bindermuehle

mmv1/templates/terraform/custom_expand/bigquery_table_ref.go.tmpl

@@ -40,5 +40,19 @@ func expand{{$.GetPrefix}}{{$.TitlelizeProperty}}(v interface{}, d tpgresource.T
 		transformed["tableId"] =  parts[3]
 	}
 
+	configError := "Invalid BigQuery job destination_table configuration. You must either:\n" +
+	  "1. Set all of project_id, dataset_id, and table_id separately, or\n" +
+		"2. Provide table_id in the form '{{`projects/{{project}}/datasets/{{dataset_id}}/tables/{{table_id}}`}}'"
+
+	// Validate required fields
+	if projectId, ok := transformed["projectId"]; !ok || projectId == nil || 
+	  reflect.ValueOf(projectId).IsZero() {
+		return nil, fmt.Errorf("%s\nMissing or empty projectId", configError)
+	}
+
+	if datasetId, ok := transformed["datasetId"]; !ok || datasetId == nil || 
+	  reflect.ValueOf(datasetId).IsZero() {
+	  return nil, fmt.Errorf("%s\nMissing or empty datasetId", configError)
+	}
 	return transformed, nil
 }

mmv1/third_party/terraform/services/bigquery/resource_bigquery_job_test.go

@@ -2,6 +2,7 @@ package bigquery_test
 
 import (
 	"fmt"
+	"regexp"
 	"testing"
 
 	"github.com/hashicorp/terraform-plugin-testing/helper/resource"
@@ -81,3 +82,64 @@ resource "google_bigquery_job" "job" {
 }
 `, context)
 }
+
+func TestAccBigQueryJob_validationErrors(t *testing.T) {
+	t.Parallel()
+	context := map[string]interface{}{
+		"random_suffix": acctest.RandString(t, 10),
+		"project":       envvar.GetTestProjectFromEnv(),
+	}
+
+	acctest.VcrTest(t, resource.TestCase{
+		PreCheck:                 func() { acctest.AccTestPreCheck(t) },
+		ProtoV5ProviderFactories: acctest.ProtoV5ProviderFactories(t),
+		Steps: []resource.TestStep{
+			{
+				Config: testAccBigQueryJob_missingProjectId(context),
+				ExpectError: regexp.MustCompile(
+					`(?s)Invalid BigQuery job destination_table configuration\. You must either:.*Missing or empty projectId`,
+				),
+			},
+			{
+				Config: testAccBigQueryJob_missingDatasetId(context),
+				ExpectError: regexp.MustCompile(
+					`(?s)Invalid BigQuery job destination_table configuration\. You must either:.*Missing or empty datasetId`,
+				),
+			},
+		},
+	})
+}
+
+func testAccBigQueryJob_missingProjectId(context map[string]interface{}) string {
+	return acctest.Nprintf(`
+resource "google_bigquery_job" "job" {
+  job_id     = "tf-test-job-%{random_suffix}"
+
+  query {
+    query = "SELECT state FROM [lookerdata:cdc.project_tycho_reports]"
+    destination_table {
+      dataset_id = "example_dataset"
+      table_id   = "example_table"
+      # project_id intentionally omitted
+    }
+  }
+}
+`, context)
+}
+
+func testAccBigQueryJob_missingDatasetId(context map[string]interface{}) string {
+	return acctest.Nprintf(`
+resource "google_bigquery_job" "job" {
+  job_id     = "tf-test-job-%{random_suffix}"
+
+  query {
+    query = "SELECT state FROM [lookerdata:cdc.project_tycho_reports]"
+    destination_table {
+      project_id = "%{project}"
+      table_id   = "example_table"
+      # dataset_id intentionally omitted
+    }
+  }
+}
+`, context)
+}