remove "prepare state" from expanders

That name tag was left in only to reduce the diff when during
implementation. Fix the naming now for these nodes so it is correct, and
prevent any possible name collision between types.
This commit is contained in:
James Bardin 2020-05-12 10:28:33 -04:00
parent 862dc36884
commit a2d2ce35dc
7 changed files with 16 additions and 4 deletions

View File

@ -22,6 +22,10 @@ var (
_ GraphNodeAttachResourceConfig = (*nodeExpandRefreshableDataResource)(nil) _ GraphNodeAttachResourceConfig = (*nodeExpandRefreshableDataResource)(nil)
) )
func (n *nodeExpandRefreshableDataResource) Name() string {
return n.NodeAbstractResource.Name() + " (expand)"
}
func (n *nodeExpandRefreshableDataResource) References() []*addrs.Reference { func (n *nodeExpandRefreshableDataResource) References() []*addrs.Reference {
return (&NodeRefreshableManagedResource{NodeAbstractResource: n.NodeAbstractResource}).References() return (&NodeRefreshableManagedResource{NodeAbstractResource: n.NodeAbstractResource}).References()
} }

View File

@ -36,7 +36,7 @@ func (n *NodePlannableLocal) Name() string {
if path != "" { if path != "" {
return path + "." + addr return path + "." + addr
} }
return addr return addr + " (expand)"
} }
// GraphNodeModulePath // GraphNodeModulePath

View File

@ -33,7 +33,7 @@ var (
) )
func (n *nodeExpandModule) Name() string { func (n *nodeExpandModule) Name() string {
return n.Addr.String() return n.Addr.String() + " (expand)"
} }
// GraphNodeModulePath implementation // GraphNodeModulePath implementation

View File

@ -49,7 +49,7 @@ func (n *nodeExpandModuleVariable) DynamicExpand(ctx EvalContext) (*Graph, error
} }
func (n *nodeExpandModuleVariable) Name() string { func (n *nodeExpandModuleVariable) Name() string {
return fmt.Sprintf("%s.%s", n.Module, n.Addr.String()) return fmt.Sprintf("%s.%s (expand)", n.Module, n.Addr.String())
} }
// GraphNodeModulePath // GraphNodeModulePath

View File

@ -30,7 +30,7 @@ func (n *nodeExpandApplyableResource) References() []*addrs.Reference {
} }
func (n *nodeExpandApplyableResource) Name() string { func (n *nodeExpandApplyableResource) Name() string {
return n.NodeAbstractResource.Name() + " (prepare state)" return n.NodeAbstractResource.Name() + " (expand)"
} }
func (n *nodeExpandApplyableResource) DynamicExpand(ctx EvalContext) (*Graph, error) { func (n *nodeExpandApplyableResource) DynamicExpand(ctx EvalContext) (*Graph, error) {

View File

@ -31,6 +31,10 @@ var (
_ GraphNodeAttachResourceConfig = (*nodeExpandPlannableResource)(nil) _ GraphNodeAttachResourceConfig = (*nodeExpandPlannableResource)(nil)
) )
func (n *nodeExpandPlannableResource) Name() string {
return n.NodeAbstractResource.Name() + " (expand)"
}
// GraphNodeDestroyerCBD // GraphNodeDestroyerCBD
func (n *nodeExpandPlannableResource) CreateBeforeDestroy() bool { func (n *nodeExpandPlannableResource) CreateBeforeDestroy() bool {
if n.ForceCreateBeforeDestroy != nil { if n.ForceCreateBeforeDestroy != nil {

View File

@ -36,6 +36,10 @@ var (
_ GraphNodeAttachDependencies = (*nodeExpandRefreshableManagedResource)(nil) _ GraphNodeAttachDependencies = (*nodeExpandRefreshableManagedResource)(nil)
) )
func (n *nodeExpandRefreshableManagedResource) Name() string {
return n.NodeAbstractResource.Name() + " (expand)"
}
// GraphNodeAttachDependencies // GraphNodeAttachDependencies
func (n *nodeExpandRefreshableManagedResource) AttachDependencies(deps []addrs.ConfigResource) { func (n *nodeExpandRefreshableManagedResource) AttachDependencies(deps []addrs.ConfigResource) {
n.Dependencies = deps n.Dependencies = deps