✨ add terraform.plan.variables

[afiune]

This change adds terraform.plan.variables to our terraform provider for a plan connection.

Using the test data added, when running cnquery shell, we now get:

➜  cnquery git:(afiune/terraform/plan-variables) ✗ ./cnquery shell terraform plan providers/terraform/connection/testdata/dynamic_block/tfplan.json -c terraform.plan.variables
! detected Schrödinger's path, cannot detect if it is usable path=/Library/Mondoo/providers/.providers-unpack2980851121/.providers-unpack2980851121.json
! using builtin provider for terraform
→ user requested to ignore .terraform
! using builtin provider for terraform
→ loaded configuration from /Users/afiune/.config/mondoo/mondoo.yml using source default
! using builtin provider for terraform
→ connected to Terraform Plan
  ___ _ __   __ _ _   _  ___ _ __ _   _
 / __| '_ \ / _` | | | |/ _ \ '__| | | |
| (__| | | | (_| | |_| |  __/ |  | |_| |
 \___|_| |_|\__, |\__,_|\___|_|   \__, |
  mondoo™      |_|                |___/  interactive shell

terraform.plan.variables: [
  0: terraform.plan.variable name="environment" value="dev"
]
cnquery>

This allows us to write queries like:

cnquery> terraform.plan.variables.where(value == "dev") {*}
terraform.plan.variables.where: [
  0: {
    value: "dev"
    name: "environment"
  }
]

[github-actions]

Test Results

3 209 tests  +1   3 205 ✅ +1   1m 40s ⏱️ -18s
  379 suites ±0       4 💤 ±0 
   29 files   ±0       0 ❌ ±0 

Results for commit 242727b. ± Comparison against base commit 35e8551.

[misterpantz]

Suggested change

	// Indicates if the plan is applyable or not
	// Whether `apply` is valid for the plan

As much fun as it can be to make up words, how's this alternative? 😁

[misterpantz]

Suggested change

	// Indicates if the plan errored or not
	// Whether the plan errored

[afiune]

Thank you!! I was following patterns in other places within this file, maybe we need a massive update in a few other providers 😅

[misterpantz]

It's a work of art. I have just a couple language suggestions.

[jaym]

maybe canApply?

[afiune]

Discussed and agreed that applyable is coming from Terraform plan output itself, so we will adopt the same name for consistency. cc @misterpantz