mirror of
https://github.com/opentofu/opentofu.git
synced 2024-12-28 01:41:48 -06:00
5661ab5991
Previously we just ported over the simple "string", "list", and "map" type hint keywords from the old loader, which exist primarily as hints to the CLI for whether to treat -var=... arguments and environment variables as literal strings or as HCL expressions. However, we've been requested before to allow more specific constraints here because it's generally better UX for a type error to be detected within an expression in a calling "module" block rather than at some point deep inside a third-party module. To allow for more specific constraints, here we use the type constraint expression syntax defined as an extension within HCL, which uses the variable and function call syntaxes to represent types rather than values, like this: - string - number - bool - list(string) - list(any) - list(map(string)) - object({id=string,name=string}) In native HCL syntax this looks like: variable "foo" { type = map(string) } In JSON, this looks like: { "variable": { "foo": { "type": "map(string)" } } } The selection of literal processing or HCL parsing of CLI-set values is now explicit in the model and separate from the type, though it's still derived from the type constraint and thus not directly controllable in configuration. Since this syntax is more complex than the keywords that replaced it, for now the simpler keywords are still supported and "list" and "map" are interpreted as list(any) and map(any) respectively, mimicking how they were interpreted by Terraform 0.11 and earlier. For the time being our documentation should continue to recommend these shorthand versions until we gain more experience with the more-specific type constraints; most users should just make use of the additional primitive type constraints this enables: bool and number. As a result of these more-complete type constraints, we can now type-check the default value at config load time, which has the nice side-effect of allowing us to produce a tailored error message if an override file produces an invalid situation; previously the result was rather confusing because the error message referred to the original definition of the variable and not the overridden parts.
157 lines
3.9 KiB
Go
157 lines
3.9 KiB
Go
package configs
|
|
|
|
import (
|
|
"io/ioutil"
|
|
"path/filepath"
|
|
"testing"
|
|
|
|
"github.com/hashicorp/hcl2/hcl"
|
|
)
|
|
|
|
// TestParseLoadConfigFileSuccess is a simple test that just verifies that
|
|
// a number of test configuration files (in test-fixtures/valid-files) can
|
|
// be parsed without raising any diagnostics.
|
|
//
|
|
// This test does not verify that reading these files produces the correct
|
|
// file element contents. More detailed assertions may be made on some subset
|
|
// of these configuration files in other tests.
|
|
func TestParserLoadConfigFileSuccess(t *testing.T) {
|
|
files, err := ioutil.ReadDir("test-fixtures/valid-files")
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
for _, info := range files {
|
|
name := info.Name()
|
|
t.Run(name, func(t *testing.T) {
|
|
src, err := ioutil.ReadFile(filepath.Join("test-fixtures/valid-files", name))
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
parser := testParser(map[string]string{
|
|
name: string(src),
|
|
})
|
|
|
|
_, diags := parser.LoadConfigFile(name)
|
|
if diags.HasErrors() {
|
|
t.Errorf("unexpected error diagnostics")
|
|
for _, diag := range diags {
|
|
t.Logf("- %s", diag)
|
|
}
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
// TestParseLoadConfigFileFailure is a simple test that just verifies that
|
|
// a number of test configuration files (in test-fixtures/invalid-files)
|
|
// produce errors as expected.
|
|
//
|
|
// This test does not verify specific error messages, so more detailed
|
|
// assertions should be made on some subset of these configuration files in
|
|
// other tests.
|
|
func TestParserLoadConfigFileFailure(t *testing.T) {
|
|
files, err := ioutil.ReadDir("test-fixtures/invalid-files")
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
for _, info := range files {
|
|
name := info.Name()
|
|
t.Run(name, func(t *testing.T) {
|
|
src, err := ioutil.ReadFile(filepath.Join("test-fixtures/invalid-files", name))
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
parser := testParser(map[string]string{
|
|
name: string(src),
|
|
})
|
|
|
|
_, diags := parser.LoadConfigFile(name)
|
|
if !diags.HasErrors() {
|
|
t.Errorf("LoadConfigFile succeeded; want errors")
|
|
}
|
|
for _, diag := range diags {
|
|
t.Logf("- %s", diag)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
// This test uses a subset of the same fixture files as
|
|
// TestParserLoadConfigFileFailure, but additionally verifies that each
|
|
// file produces the expected diagnostic summary.
|
|
func TestParserLoadConfigFileFailureMessages(t *testing.T) {
|
|
tests := []struct {
|
|
Filename string
|
|
WantSeverity hcl.DiagnosticSeverity
|
|
WantDiag string
|
|
}{
|
|
{
|
|
"invalid-files/data-resource-lifecycle.tf",
|
|
hcl.DiagError,
|
|
"Unsupported lifecycle block",
|
|
},
|
|
{
|
|
"invalid-files/variable-type-unknown.tf",
|
|
hcl.DiagError,
|
|
"Invalid type specification",
|
|
},
|
|
{
|
|
"invalid-files/unexpected-attr.tf",
|
|
hcl.DiagError,
|
|
"Unsupported attribute",
|
|
},
|
|
{
|
|
"invalid-files/unexpected-block.tf",
|
|
hcl.DiagError,
|
|
"Unsupported block type",
|
|
},
|
|
{
|
|
"invalid-files/resource-lifecycle-badbool.tf",
|
|
hcl.DiagError,
|
|
"Unsuitable value type",
|
|
},
|
|
{
|
|
"valid-files/resources-ignorechanges-all-legacy.tf",
|
|
hcl.DiagWarning,
|
|
"Deprecated ignore_changes wildcard",
|
|
},
|
|
{
|
|
"valid-files/resources-ignorechanges-all-legacy.tf.json",
|
|
hcl.DiagWarning,
|
|
"Deprecated ignore_changes wildcard",
|
|
},
|
|
}
|
|
|
|
for _, test := range tests {
|
|
t.Run(test.Filename, func(t *testing.T) {
|
|
src, err := ioutil.ReadFile(filepath.Join("test-fixtures", test.Filename))
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
parser := testParser(map[string]string{
|
|
test.Filename: string(src),
|
|
})
|
|
|
|
_, diags := parser.LoadConfigFile(test.Filename)
|
|
if len(diags) != 1 {
|
|
t.Errorf("Wrong number of diagnostics %d; want 1", len(diags))
|
|
for _, diag := range diags {
|
|
t.Logf("- %s", diag)
|
|
}
|
|
return
|
|
}
|
|
if diags[0].Severity != test.WantSeverity {
|
|
t.Errorf("Wrong diagnostic severity %#v; want %#v", diags[0].Severity, test.WantSeverity)
|
|
}
|
|
if diags[0].Summary != test.WantDiag {
|
|
t.Errorf("Wrong diagnostic summary\ngot: %s\nwant: %s", diags[0].Summary, test.WantDiag)
|
|
}
|
|
})
|
|
}
|
|
}
|