terraform test is not applying defaults to variables referenced in assertions

[liamcervante]

Terraform Version

`v1.6.0-beta1`

Terraform Configuration Files

variable "input" {

  type = object({
    required = string
    optional = optional(string)
    default = optional(string, "default")
  })

  default = {
    required = "required"
  }

}

resource "test_resource" "resource" {
  value = var.input.default
}

output "computed" {
  value = test_resource.resource.value
}

# main.tftest.hcl


run "stacked" {
  variables {
    input = {
      required = "required"
      optional = "optional"
      default = "overridden"
    }
  }

  assert {
    condition = output.computed == "overridden"
    error_message = "did not override default value"
  }
}

run "defaults" {
  assert {
    condition = output.computed == "default"
    error_message = "didn't set default value"
  }
}

run "custom_defined_apply_defaults" {
  variables {
    input = {
      required = "required"
    }
  }

  assert {
    condition = output.computed == "default"
    error_message = "didn't set default value"
  }
}

Debug Output

N/A

Expected Behavior

Tests should apply defaults and pass.

Actual Behavior

        main.tftest.hcl... in progress
          run "stacked"... pass
          run "defaults"... pass
          run "custom_defined_apply_defaults"... fail
        ╷
        │ Error: Invalid value for input variable
        │ 
        │ The given value is not suitable for var.input declared at main.tf:2,1-17:
        │ attributes "default" and "optional" are required.
        ╵
        main.tftest.hcl... tearing down
        main.tftest.hcl... fail
        
        Failure! 2 passed, 1 failed.

Steps to Reproduce

terraform init
terraform test

Additional Context

We're missing a call to apply the defaults before we convert the variables here.

I've looked at this again, and I think we're taking the wrong approach. We should have the computed input values from the plan already available - so let's just use that? Will see if possible.

References

First discovered by here.

[github-actions]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.