Fix panic when generating json config for tf query (#37596)

Author: dsa0x

internal/command/query_test.go

@@ -0,0 +1,24 @@
+terraform {
+  required_providers {
+    test = {
+      source = "hashicorp/test"
+    }
+  }
+}
+
+variable "target_ami" {
+  description = "The AMI to search for"
+  type        = string
+  default     = "ami-12345"
+}
+
+variable "instance_name" {
+  description = "The instance name to search for"
+  type        = string
+}
+
+provider "test" {}
+
+resource "test_instance" "example" {
+  ami = "ami-12345"
+}

internal/command/testdata/query/duplicate-variables/main.tf

@@ -0,0 +1,15 @@
+// already defined in main.tf
+variable "instance_name" {
+  description = "The instance name to search for"
+  type        = string
+}
+
+
+list "test_instance" "example" {
+  provider = test
+
+  config {
+    ami = var.target_ami
+    foo = var.instance_name
+  }
+}

internal/command/testdata/query/duplicate-variables/query.tfquery.hcl

@@ -11,5 +11,7 @@ list "test_instance" "example2" {
 
   config {
     ami = "ami-nonexistent"
+    // to force deterministic ordering in the result
+    foo = list.test_instance.example.data[0].state.id
   }
 }

internal/command/testdata/query/empty-result/query.tfquery.hcl

@@ -0,0 +1,13 @@
+terraform {
+  required_providers {
+    test = {
+      source = "hashicorp/test"
+    }
+  }
+}
+
+provider "test" {}
+
+resource "test_instance" "example" {
+  ami = "ami-12345"
+}

internal/command/testdata/query/unknown-config/main.tf

@@ -0,0 +1,7 @@
+list "test_instance" "example" {
+  provider = test
+
+  config {
+    ami = uuid() // forces the config to be unknown at plan time
+  }
+}

internal/command/testdata/query/unknown-config/query.tfquery.hcl

@@ -0,0 +1,13 @@
+terraform {
+  required_providers {
+    test = {
+      source = "hashicorp/test"
+    }
+  }
+}
+
+provider "test" {}
+
+resource "test_instance" "example" {
+  ami = "ami-12345"
+}

internal/command/testdata/query/with-invalid-variables/main.tf

@@ -0,0 +1,17 @@
+variable "target_ami" {
+  description = "The AMI to search for"
+  type        = string
+  validation {
+    condition     = length(var.target_ami) > 10
+    error_message = "AMI ID must be longer than 10 characters."
+  }
+}
+
+list "test_instance" "example" {
+  provider = test
+
+  config {
+    ami = var.target_ami
+    foo = "invalid-instance"
+  }
+}

internal/command/testdata/query/with-invalid-variables/query.tfquery.hcl

@@ -0,0 +1,3 @@
+target_ami     = "ami-67890"
+environment    = "staging"
+instance_count = 3

internal/command/testdata/query/with-variables-file/custom.tfvars

@@ -0,0 +1,13 @@
+terraform {
+  required_providers {
+    test = {
+      source = "hashicorp/test"
+    }
+  }
+}
+
+provider "test" {}
+
+resource "test_instance" "example" {
+  ami = "ami-12345"
+}