Skip to content

Commit 1636489

Browse files
Copilottobio
andcommitted
Refactor EnrichPolicy models to separate resource and data source types, fix test assertion
Co-authored-by: tobio <[email protected]>
1 parent 3ca7bb2 commit 1636489

File tree

5 files changed

+9
-5
lines changed

5 files changed

+9
-5
lines changed

internal/elasticsearch/enrich/acc_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ func TestAccResourceEnrichPolicyFW(t *testing.T) {
2929
resource.TestCheckResourceAttr("elasticstack_elasticsearch_enrich_policy.policy", "indices.0", name),
3030
resource.TestCheckResourceAttr("elasticstack_elasticsearch_enrich_policy.policy", "enrich_fields.0", "first_name"),
3131
resource.TestCheckResourceAttr("elasticstack_elasticsearch_enrich_policy.policy", "enrich_fields.1", "last_name"),
32-
resource.TestCheckResourceAttr("elasticstack_elasticsearch_enrich_policy.policy", "query", "{\"match_all\":{}}"),
32+
resource.TestCheckResourceAttr("elasticstack_elasticsearch_enrich_policy.policy", "query", "{\"match_all\": {}}\n"),
3333
resource.TestCheckResourceAttr("elasticstack_elasticsearch_enrich_policy.policy", "execute", "true"),
3434
),
3535
},

internal/elasticsearch/enrich/create.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ func (r *enrichPolicyResource) Update(ctx context.Context, req resource.UpdateRe
3131
}
3232

3333
func (r *enrichPolicyResource) upsert(ctx context.Context, plan tfsdk.Plan, state *tfsdk.State) diag.Diagnostics {
34-
var data EnrichPolicyData
34+
var data EnrichPolicyDataWithExecute
3535
var diags diag.Diagnostics
3636
diags.Append(plan.Get(ctx, &data)...)
3737
if diags.HasError() {

internal/elasticsearch/enrich/delete.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ import (
1010
)
1111

1212
func (r *enrichPolicyResource) Delete(ctx context.Context, req resource.DeleteRequest, resp *resource.DeleteResponse) {
13-
var data EnrichPolicyData
13+
var data EnrichPolicyDataWithExecute
1414
resp.Diagnostics.Append(req.State.Get(ctx, &data)...)
1515
if resp.Diagnostics.HasError() {
1616
return

internal/elasticsearch/enrich/models.go

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,11 @@ type EnrichPolicyData struct {
2020
MatchField types.String `tfsdk:"match_field"`
2121
EnrichFields types.List `tfsdk:"enrich_fields"`
2222
Query jsontypes.Normalized `tfsdk:"query"`
23-
Execute types.Bool `tfsdk:"execute"`
23+
}
24+
25+
type EnrichPolicyDataWithExecute struct {
26+
EnrichPolicyData
27+
Execute types.Bool `tfsdk:"execute"`
2428
}
2529

2630
// populateFromPolicy converts models.EnrichPolicy to EnrichPolicyData fields

internal/elasticsearch/enrich/read.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ import (
1212
)
1313

1414
func (r *enrichPolicyResource) Read(ctx context.Context, req resource.ReadRequest, resp *resource.ReadResponse) {
15-
var data EnrichPolicyData
15+
var data EnrichPolicyDataWithExecute
1616
resp.Diagnostics.Append(req.State.Get(ctx, &data)...)
1717
if resp.Diagnostics.HasError() {
1818
return

0 commit comments

Comments
 (0)