[feat] add an option for queryStr separator

[ryotafuwa-dev]

WHAT

(Write the change being made with this pull request)
title

WHY

(Write the motivation why you submit this pull request)
Terraform allows comma-separated attributes (e.g. aliased providers like google.common inside a module). In order to modify attributes with comma in its name, we need a way to specify the queryStr split separator charactor.

Behavior

given:

module "A" {
  providers = {
    google.aliasA = google.aliasA
  }
}

where:

hcledit.Create("module.A.providers.google.aliasB", RawVal("google.aliasB"))

then:

module "A" {
  providers = {
    google.aliasA = google.aliasA
  }
}

Because queryStr considers all the commas as the query separator.

target:

module "A" {
  providers = {
    google.aliasA = google.aliasA
    google.aliasB = google.aliasB
  }
}

[ryan-ph]

Can you include a functional test plan in the PR description (i.e. an input HCL config, the current result of running the query, and the new result of running the query)?

I'm not sure I quite understand where exactly these non-. separators are actually used in TF. The unit test uses /, but as far as I understand, this is not an accepted separator in HCL and I don't understand the benefit of adding this.

[ryotafuwa-dev]

As I stated in the PR description, this PR tries to add a way to specify a charactor other than "." as a query separator in HCLEditor due to this.

Terraform allows comma-separated attributes (e.g. aliased providers like google.common inside a module).

Can you include a functional test plan in the PR description (i.e. an input HCL config, the current result of running the query, and the new result of running the query)?

I think the unit tests I added are enough for this change. Are there more tests you'd like to see for this change?

[ryan-ph]

I think the unit tests I added are enough for this change. Are there more tests you'd like to see for this change?

I'm trying to understand where a non-. separator is actually used because I have never seen it before. Neither the unit test nor the PR description include a , separator being used in the HCL config that's being read / modified.

[ryan-ph]

So this is meant to support provider aliasing¹ left hand side operands as part of the query.

Footnotes

1. https://developer.hashicorp.com/terraform/language/providers/configuration#alias-multiple-provider-configurations ↩