add cmd for terraform plan/show

Author: Emyrk

cli/root.go

@@ -96,6 +96,7 @@ func (r *RootCmd) Root() *serpent.Command {
 			return nil
 		},
 	}
+	cmd.AddSubcommands(r.TerraformPlan())
 	return cmd
 }
 

preview.go

@@ -69,6 +69,7 @@ func Preview(ctx context.Context, input Input, dir fs.FS) (*Output, hcl.Diagnost
 			},
 		}
 	}
+
 	var _ = outputs
 
 	diags := make(hcl.Diagnostics, 0)

preview_test.go

@@ -184,6 +184,16 @@ func Test_Extract(t *testing.T) {
 			expUnknowns: []string{},
 			params:      map[string]func(t *testing.T, parameter types.Parameter){},
 		},
+		{
+			name:    "dupemodparams",
+			dir:     "dupemodparams",
+			expTags: map[string]string{},
+			input: preview.Input{
+				ParameterValues: map[string]types.ParameterValue{},
+			},
+			expUnknowns: []string{},
+			params:      map[string]func(t *testing.T, parameter types.Parameter){},
+		},
 		{
 			name:    "test",
 			dir:     "test",

previewe2e_test.go

@@ -20,6 +20,9 @@ import (
 // Test_VerifyE2E will fully evaluate with `terraform apply`
 // and verify the output of `preview` against the tfstate. This
 // is the e2e test for the preview package.
+// It uses the `terraform plan` output, meaning this mimics the
+// "Workspace Create" form of preview. Which is arguably the most
+// important part of the preview lifecycle.
 //
 //  1. Terraform versions listed from 'verify.TerraformTestVersions' are
 //     installed into a temp directory. If multiple versions exist, a e2e test
@@ -44,6 +47,7 @@ import (
 // The goal of the test is to compare `tfstate` with the output of `preview`.
 // If `preview`'s implementation of terraform is incorrect, the test will fail.
 // TODO: Adding varied parameter inputs would be a good idea.
+// TODO: Add workspace tag comparisions.
 func Test_VerifyE2E(t *testing.T) {
 	t.Parallel()
 

testdata/.gitignore

@@ -1,4 +1,5 @@
 */.terraform*
 */terraform.tfstate
 */terraform.tfstate.backup
-*/out.plan
+*/out.plan
+*/out.json

testdata/dupemodparams/dupe/main.tf

@@ -0,0 +1,18 @@
+terraform {
+  required_providers {
+    coder = {
+      source = "coder/coder"
+    }
+  }
+}
+
+module "dupe" {
+  source = "./nesteddupe"
+}
+
+data "coder_parameter" "dupe" {
+  name        = "Dupe Question"
+  description = "A question that will be duplicated"
+  type        = "string"
+  default     = "dupe"
+}

testdata/dupemodparams/dupe/nesteddupe/main.tf

@@ -0,0 +1,14 @@
+terraform {
+  required_providers {
+    coder = {
+      source = "coder/coder"
+    }
+  }
+}
+
+data "coder_parameter" "dupe" {
+  name        = "Dupe Question"
+  description = "A question that will be duplicated"
+  type        = "string"
+  default     = "dupe"
+}

testdata/dupemodparams/main.tf

@@ -0,0 +1,18 @@
+terraform {
+  required_providers {
+    coder = {
+      source = "coder/coder"
+    }
+  }
+}
+
+module "dupe" {
+  source = "./dupe"
+}
+
+data "coder_parameter" "dupe" {
+  name        = "Dupe Question"
+  description = "A question that will be duplicated"
+  type        = "string"
+  default     = "dupe"
+}

testdata/dupeparams/main.tf

@@ -0,0 +1,21 @@
+terraform {
+  required_providers {
+    coder = {
+      source = "coder/coder"
+    }
+  }
+}
+
+data "coder_parameter" "dupe" {
+  name        = "Dupe Question"
+  description = "A question that will be duplicated"
+  type        = "string"
+  default     = "dupe"
+}
+
+data "coder_parameter" "dupe" {
+  name        = "Dupe Question"
+  description = "A question that will be duplicated"
+  type        = "string"
+  default     = "dupe"
+}