fix: add test case to validate empty provider (#206)

kylecarbs · web-flow · commit 8fa0fbf8655a · 2024-05-13T11:06:48.000-04:00
diff --git a/provider/provider_test.go b/provider/provider_test.go
@@ -3,6 +3,9 @@ package provider_test
 import (
 	"testing"
 
+	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/resource"
+	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
+	"github.com/hashicorp/terraform-plugin-sdk/v2/terraform"
 	"github.com/stretchr/testify/require"
 
 	"github.com/coder/terraform-provider-coder/provider"
@@ -14,3 +17,34 @@ func TestProvider(t *testing.T) {
 	err := tfProvider.InternalValidate()
 	require.NoError(t, err)
 }
+
+// TestProviderEmpty ensures that the provider can be configured without
+// any actual input data. This is important for adding new fields
+// with backwards compatibility guarantees.
+func TestProviderEmpty(t *testing.T) {
+	t.Parallel()
+	resource.Test(t, resource.TestCase{
+		Providers: map[string]*schema.Provider{
+			"coder": provider.New(),
+		},
+		IsUnitTest: true,
+		Steps: []resource.TestStep{{
+			Config: `
+			provider "coder" {}
+			data "coder_provisioner" "me" {}
+			data "coder_workspace" "me" {}
+			data "coder_external_auth" "git" {
+				id = "git"
+			}
+			data "coder_git_auth" "git" {
+				id = "git"
+			}
+			data "coder_parameter" "param" {
+				name = "hey"
+			}`,
+			Check: func(state *terraform.State) error {
+				return nil
+			},
+		}},
+	})
+}
diff --git a/provider/provisioner_test.go b/provider/provisioner_test.go
@@ -12,6 +12,7 @@ import (
 )
 
 func TestProvisioner(t *testing.T) {
+	t.Parallel()
 	resource.Test(t, resource.TestCase{
 		Providers: map[string]*schema.Provider{
 			"coder": provider.New(),

Original file line number	Diff line number	Diff line change
`@@ -12,6 +12,7 @@ import (`
`12`	`12`	`)`
`13`	`13`
`14`	`14`	`func TestProvisioner(t *testing.T) {`
	`15`	`+ t.Parallel()`
`15`	`16`	`resource.Test(t, resource.TestCase{`
`16`	`17`	`Providers: map[string]*schema.Provider{`
`17`	`18`	`"coder": provider.New(),`