diff --git a/internal/configs/mock_provider.go b/internal/configs/mock_provider.go index f6d05bfcdcba..81548eec9fdc 100644 --- a/internal/configs/mock_provider.go +++ b/internal/configs/mock_provider.go @@ -98,13 +98,9 @@ type MockData struct { MockDataSources map[string]*MockResource Overrides addrs.Map[addrs.Targetable, *Override] - useForPlan *bool // If true, the mock data can be used for planning. -} - -// UseForPlan returns true if the provider-level setting for overrideComputed -// is true, meaning that computed values can be overridden with the mocked values. -func (data *MockData) UseForPlan() bool { - return data.useForPlan != nil && *data.useForPlan + // UseForPlan returns true if the provider-level setting for overrideComputed + // is true, meaning that computed values can be overridden with the mocked values during planning. + UseForPlan bool } // Merge will merge the target MockData object into the current MockData. @@ -242,7 +238,7 @@ func decodeMockDataBody(body hcl.Body, source OverrideSource) (*MockData, hcl.Di MockResources: make(map[string]*MockResource), MockDataSources: make(map[string]*MockResource), Overrides: addrs.MakeMap[addrs.Targetable, *Override](), - useForPlan: &useForPlan, + UseForPlan: useForPlan, } for _, block := range content.Blocks { diff --git a/internal/providers/mock.go b/internal/providers/mock.go index f65882d40fa3..2b6ecf097870 100644 --- a/internal/providers/mock.go +++ b/internal/providers/mock.go @@ -187,7 +187,7 @@ func (m *Mock) PlanResourceChange(request PlanResourceChangeRequest) PlanResourc ComputedAsUnknown: true, } // if we are allowed to use the mock defaults for plan, we can populate the computed fields with the mock defaults. - if mockedResource, exists := m.Data.MockResources[request.TypeName]; exists && m.Data.UseForPlan() { + if mockedResource, exists := m.Data.MockResources[request.TypeName]; exists && m.Data.UseForPlan { replacement.Value = mockedResource.Defaults replacement.Range = mockedResource.DefaultsRange replacement.ComputedAsUnknown = false