// Copyright (c) The OpenTofu Authors // SPDX-License-Identifier: MPL-2.0 // Copyright (c) 2023 HashiCorp, Inc. // SPDX-License-Identifier: MPL-2.0 package arguments import ( "strings" "testing" "github.com/google/go-cmp/cmp" "github.com/hashicorp/hcl/v2" "github.com/opentofu/opentofu/internal/addrs" "github.com/opentofu/opentofu/internal/tfdiags" ) func TestParseRefresh_basicValid(t *testing.T) { testCases := map[string]struct { args []string want *Refresh }{ "defaults": { nil, &Refresh{ InputEnabled: true, ViewType: ViewHuman, }, }, "input=false": { []string{"-input=false"}, &Refresh{ InputEnabled: false, ViewType: ViewHuman, }, }, "JSON view disables input": { []string{"-json"}, &Refresh{ InputEnabled: false, ViewType: ViewJSON, }, }, } for name, tc := range testCases { t.Run(name, func(t *testing.T) { got, diags := ParseRefresh(tc.args) if len(diags) > 0 { t.Fatalf("unexpected diags: %v", diags) } // Ignore the extended arguments for simplicity got.State = nil got.Operation = nil got.Vars = nil if *got != *tc.want { t.Fatalf("unexpected result\n got: %#v\nwant: %#v", got, tc.want) } }) } } func TestParseRefresh_invalid(t *testing.T) { got, diags := ParseRefresh([]string{"-frob"}) if len(diags) == 0 { t.Fatal("expected diags but got none") } if got, want := diags.Err().Error(), "flag provided but not defined"; !strings.Contains(got, want) { t.Fatalf("wrong diags\n got: %s\nwant: %s", got, want) } if got.ViewType != ViewHuman { t.Fatalf("wrong view type, got %#v, want %#v", got.ViewType, ViewHuman) } } func TestParseRefresh_tooManyArguments(t *testing.T) { got, diags := ParseRefresh([]string{"saved.tfplan"}) if len(diags) == 0 { t.Fatal("expected diags but got none") } if got, want := diags.Err().Error(), "Too many command line arguments"; !strings.Contains(got, want) { t.Fatalf("wrong diags\n got: %s\nwant: %s", got, want) } if got.ViewType != ViewHuman { t.Fatalf("wrong view type, got %#v, want %#v", got.ViewType, ViewHuman) } } func TestParseRefresh_targets(t *testing.T) { foobarbaz, _ := addrs.ParseTargetStr("foo_bar.baz") boop, _ := addrs.ParseTargetStr("module.boop") testCases := map[string]struct { args []string want []addrs.Targetable wantErr string }{ "no targets by default": { args: nil, want: nil, }, "one target": { args: []string{"-target=foo_bar.baz"}, want: []addrs.Targetable{foobarbaz.Subject}, }, "two targets": { args: []string{"-target=foo_bar.baz", "-target", "module.boop"}, want: []addrs.Targetable{foobarbaz.Subject, boop.Subject}, }, "invalid traversal": { args: []string{"-target=foo."}, want: nil, wantErr: "Dot must be followed by attribute name", }, "invalid target": { args: []string{"-target=data[0].foo"}, want: nil, wantErr: "A data source name is required", }, } for name, tc := range testCases { t.Run(name, func(t *testing.T) { got, diags := ParseRefresh(tc.args) if tc.wantErr == "" && len(diags) > 0 { t.Fatalf("unexpected diags: %v", diags) } else if tc.wantErr != "" { if len(diags) == 0 { t.Fatalf("expected diags but got none") } else if got := diags.Err().Error(); !strings.Contains(got, tc.wantErr) { t.Fatalf("wrong diags\n got: %s\nwant: %s", got, tc.wantErr) } } if !cmp.Equal(got.Operation.Targets, tc.want) { t.Fatalf("unexpected result\n%s", cmp.Diff(got.Operation.Targets, tc.want)) } }) } } func TestParseRefresh_targetFile(t *testing.T) { foobarbaz, _ := addrs.ParseTargetStr("foo_bar.baz") boop, _ := addrs.ParseTargetStr("module.boop") barbaz, _ := addrs.ParseTargetStr("bar.baz") testCases := map[string]struct { files []mockFile want []addrs.Targetable }{ "target file no targets": { files: []mockFile{}, want: nil, }, "target file valid single target": { files: []mockFile{ {fileContent: "foo_bar.baz"}}, want: []addrs.Targetable{foobarbaz.Subject}, }, "target file valid multiple targets": { files: []mockFile{ {fileContent: "foo_bar.baz\nmodule.boop"}, }, want: []addrs.Targetable{foobarbaz.Subject, boop.Subject}, }, "target file invalid target": { files: []mockFile{ { fileContent: "foo.", diags: hcl.Diagnostics{ &hcl.Diagnostic{ Severity: hcl.DiagError, Summary: "Attribute name required", Detail: "Dot must be followed by attribute name.", Subject: &hcl.Range{ Start: hcl.Pos{Line: 1, Column: 5, Byte: 4}, End: hcl.Pos{Line: 1, Column: 5, Byte: 4}, }, Context: &hcl.Range{ Start: hcl.Pos{Line: 1, Column: 1}, End: hcl.Pos{Line: 1, Column: 5, Byte: 4}, }, }, }, }, }, want: nil, }, "multiple files valid targets": { files: []mockFile{ {fileContent: "foo_bar.baz"}, {fileContent: "module.boop"}, }, want: []addrs.Targetable{foobarbaz.Subject, boop.Subject}, }, "multiple files invalid target": { files: []mockFile{ {fileContent: "foo_bar.baz"}, { fileContent: "modu(le.boop", diags: hcl.Diagnostics{ &hcl.Diagnostic{ Severity: hcl.DiagError, Summary: "Invalid character", Detail: `Expected an attribute access or an index operator.`, Subject: &hcl.Range{ Start: hcl.Pos{Line: 1, Column: 5, Byte: 4}, End: hcl.Pos{Line: 1, Column: 6, Byte: 5}, }, Context: &hcl.Range{ Start: hcl.Pos{Line: 1, Column: 1}, End: hcl.Pos{Line: 1, Column: 6, Byte: 5}, }, }, }, }, }, want: []addrs.Targetable{foobarbaz.Subject}, }, "multiple files multiple invalid targets": { files: []mockFile{ { fileContent: "modu(le.boop", diags: hcl.Diagnostics{ &hcl.Diagnostic{ Severity: hcl.DiagError, Summary: "Invalid character", Detail: "Expected an attribute access or an index operator.", Subject: &hcl.Range{ Start: hcl.Pos{Line: 1, Column: 5, Byte: 4}, End: hcl.Pos{Line: 1, Column: 6, Byte: 5}, }, Context: &hcl.Range{ Start: hcl.Pos{Line: 1, Column: 1}, End: hcl.Pos{Line: 1, Column: 6, Byte: 5}, }, }, }, }, {fileContent: "foo_bar.baz"}, { fileContent: "bar^.baz", diags: []*hcl.Diagnostic{ &hcl.Diagnostic{ Severity: hcl.DiagError, Summary: "Unsupported operator", Detail: `Bitwise operators are not supported.`, Subject: &hcl.Range{ Start: hcl.Pos{Line: 1, Column: 4, Byte: 3}, End: hcl.Pos{Line: 1, Column: 5, Byte: 4}, }, }, &hcl.Diagnostic{ Severity: hcl.DiagError, Summary: "Invalid character", Detail: "Expected an attribute access or an index operator.", Subject: &hcl.Range{ Start: hcl.Pos{Line: 1, Column: 4, Byte: 3}, End: hcl.Pos{Line: 1, Column: 5, Byte: 4}, }, Context: &hcl.Range{ Start: hcl.Pos{Line: 1, Column: 1}, End: hcl.Pos{Line: 1, Column: 5, Byte: 4}, }, }, }, }, }, want: []addrs.Targetable{foobarbaz.Subject}, }, "target file valid comment": { files: []mockFile{ {fileContent: "#foo_bar.baz"}, }, want: nil, }, "target file valid spaces": { files: []mockFile{ {fileContent: " foo_bar.baz"}, }, want: []addrs.Targetable{foobarbaz.Subject}, }, "target file valid tab": { files: []mockFile{ {fileContent: "\tfoo_bar.baz"}, }, want: []addrs.Targetable{foobarbaz.Subject}, }, "target file valid complicated": { files: []mockFile{ {fileContent: "\tmodule.boop\n#foo_bar.baz\nbar.baz"}, }, want: []addrs.Targetable{boop.Subject, barbaz.Subject}, }, "target file invalid bracket with spaces": { files: []mockFile{ { fileContent: ` [boop]`, diags: hcl.Diagnostics{ &hcl.Diagnostic{ Severity: hcl.DiagError, Summary: "Variable name required", Detail: `Must begin with a variable name.`, Subject: &hcl.Range{ Start: hcl.Pos{Line: 1, Column: 5, Byte: 4}, End: hcl.Pos{Line: 1, Column: 6, Byte: 5}, }, }, }, }, }, want: nil, }, } for name, tc := range testCases { t.Run(name, func(t *testing.T) { targetFileArguments := []string{} wantDiags := tfdiags.Diagnostics{} for _, mockFile := range tc.files { mockFile.tempFileWriter(t) targetFileArguments = append(targetFileArguments, "-target-file="+mockFile.filePath) // for setting the correct filePath on each wantDiag if len(mockFile.diags) > 0 { for _, diag := range mockFile.diags { diag.Subject.Filename = mockFile.filePath if diag.Context != nil { diag.Context.Filename = mockFile.filePath } wantDiags = wantDiags.Append(diag) } } } wantDiagsExported := wantDiags.ForRPC() got, gotDiags := ParseRefresh(targetFileArguments) gotDiagsExported := gotDiags.ForRPC() if len(wantDiagsExported) != 0 || len(gotDiags) != 0 { if len(gotDiags) == 0 { t.Fatalf("expected diags but got none") } if len(wantDiagsExported) == 0 { t.Fatalf("got diags but didn't want any: %v", gotDiags.ErrWithWarnings()) } if diff := cmp.Diff(gotDiagsExported, wantDiagsExported); diff != "" { t.Fatalf("diff between want(+) and got(-) diagnostics\n%s", diff) } } if !cmp.Equal(got.Operation.Targets, tc.want) { t.Fatalf("diff between want(+) and got(-) targets\n%s", cmp.Diff(got.Operation.Targets, tc.want)) } }) } } func TestParseRefresh_excludes(t *testing.T) { foobarbaz, _ := addrs.ParseTargetStr("foo_bar.baz") boop, _ := addrs.ParseTargetStr("module.boop") testCases := map[string]struct { args []string want []addrs.Targetable wantErr string }{ "no excludes by default": { args: nil, want: nil, }, "one exclude": { args: []string{"-exclude=foo_bar.baz"}, want: []addrs.Targetable{foobarbaz.Subject}, }, "two excludes": { args: []string{"-exclude=foo_bar.baz", "-exclude", "module.boop"}, want: []addrs.Targetable{foobarbaz.Subject, boop.Subject}, }, "invalid traversal": { args: []string{"-exclude=foo."}, want: nil, wantErr: "Dot must be followed by attribute name", }, "invalid target": { args: []string{"-exclude=data[0].foo"}, want: nil, wantErr: "A data source name is required", }, } for name, tc := range testCases { t.Run(name, func(t *testing.T) { got, diags := ParseRefresh(tc.args) if tc.wantErr == "" && len(diags) > 0 { t.Fatalf("unexpected diags: %v", diags) } else if tc.wantErr != "" { if len(diags) == 0 { t.Fatalf("expected diags but got none") } else if got := diags.Err().Error(); !strings.Contains(got, tc.wantErr) { t.Fatalf("wrong diags\n got: %s\nwant: %s", got, tc.wantErr) } } if !cmp.Equal(got.Operation.Excludes, tc.want) { t.Fatalf("unexpected result\n%s", cmp.Diff(got.Operation.Excludes, tc.want)) } }) } } func TestParseRefresh_excludeAndTarget(t *testing.T) { got, gotDiags := ParseRefresh([]string{"-exclude=foo_bar.baz", "-target=foo_bar.bar"}) if len(gotDiags) == 0 { t.Fatalf("expected error, but there was none") } wantDiags := tfdiags.Diagnostics{ tfdiags.Sourceless( tfdiags.Error, "Invalid combination of arguments", "The target and exclude planning options are mutually-exclusive. Each plan must use either only the target options or only the exclude options.", ), } if diff := cmp.Diff(wantDiags.ForRPC(), gotDiags.ForRPC()); diff != "" { t.Errorf("wrong diagnostics\n%s", diff) } if len(got.Operation.Targets) > 0 { t.Errorf("Did not expect operation to parse targets, but it parsed %d targets", len(got.Operation.Targets)) } if len(got.Operation.Excludes) > 0 { t.Errorf("Did not expect operation to parse excludes, but it parsed %d targets", len(got.Operation.Excludes)) } } func TestParseRefresh_vars(t *testing.T) { testCases := map[string]struct { args []string want []FlagNameValue }{ "no var flags by default": { args: nil, want: nil, }, "one var": { args: []string{"-var", "foo=bar"}, want: []FlagNameValue{ {Name: "-var", Value: "foo=bar"}, }, }, "one var-file": { args: []string{"-var-file", "cool.tfvars"}, want: []FlagNameValue{ {Name: "-var-file", Value: "cool.tfvars"}, }, }, "ordering preserved": { args: []string{ "-var", "foo=bar", "-var-file", "cool.tfvars", "-var", "boop=beep", }, want: []FlagNameValue{ {Name: "-var", Value: "foo=bar"}, {Name: "-var-file", Value: "cool.tfvars"}, {Name: "-var", Value: "boop=beep"}, }, }, } for name, tc := range testCases { t.Run(name, func(t *testing.T) { got, diags := ParseRefresh(tc.args) if len(diags) > 0 { t.Fatalf("unexpected diags: %v", diags) } if vars := got.Vars.All(); !cmp.Equal(vars, tc.want) { t.Fatalf("unexpected result\n%s", cmp.Diff(vars, tc.want)) } if got, want := got.Vars.Empty(), len(tc.want) == 0; got != want { t.Fatalf("expected Empty() to return %t, but was %t", want, got) } }) } }