Merge pull request #4999 from svanharmelen/f-rename-missing-provider-node

Rename `graphNodeMissingProvider` to `graphNodeProvider`
This commit is contained in:
Sander van Harmelen 2016-02-08 20:32:03 +01:00
commit f6822dcb7d
2 changed files with 30 additions and 30 deletions

View File

@ -195,8 +195,8 @@ func (t *MissingProviderTransformer) Transform(g *Graph) error {
continue continue
} }
// Add our own missing provider node to the graph // Add the missing provider node to the graph
m[p] = g.Add(&graphNodeMissingProvider{ProviderNameValue: p}) m[p] = g.Add(&graphNodeProvider{ProviderNameValue: p})
} }
} }
@ -394,34 +394,34 @@ func (n *graphNodeCloseProvider) DotNode(name string, opts *GraphDotOpts) *dot.N
}) })
} }
type graphNodeMissingProvider struct { type graphNodeProvider struct {
ProviderNameValue string ProviderNameValue string
} }
func (n *graphNodeMissingProvider) Name() string { func (n *graphNodeProvider) Name() string {
return fmt.Sprintf("provider.%s", n.ProviderNameValue) return fmt.Sprintf("provider.%s", n.ProviderNameValue)
} }
// GraphNodeEvalable impl. // GraphNodeEvalable impl.
func (n *graphNodeMissingProvider) EvalTree() EvalNode { func (n *graphNodeProvider) EvalTree() EvalNode {
return ProviderEvalTree(n.ProviderNameValue, nil) return ProviderEvalTree(n.ProviderNameValue, nil)
} }
// GraphNodeDependable impl. // GraphNodeDependable impl.
func (n *graphNodeMissingProvider) DependableName() []string { func (n *graphNodeProvider) DependableName() []string {
return []string{n.Name()} return []string{n.Name()}
} }
func (n *graphNodeMissingProvider) ProviderName() string { func (n *graphNodeProvider) ProviderName() string {
return n.ProviderNameValue return n.ProviderNameValue
} }
func (n *graphNodeMissingProvider) ProviderConfig() *config.RawConfig { func (n *graphNodeProvider) ProviderConfig() *config.RawConfig {
return nil return nil
} }
// GraphNodeDotter impl. // GraphNodeDotter impl.
func (n *graphNodeMissingProvider) DotNode(name string, opts *GraphDotOpts) *dot.Node { func (n *graphNodeProvider) DotNode(name string, opts *GraphDotOpts) *dot.Node {
return dot.NewNode(name, map[string]string{ return dot.NewNode(name, map[string]string{
"label": n.Name(), "label": n.Name(),
"shape": "diamond", "shape": "diamond",
@ -429,46 +429,46 @@ func (n *graphNodeMissingProvider) DotNode(name string, opts *GraphDotOpts) *dot
} }
// GraphNodeDotterOrigin impl. // GraphNodeDotterOrigin impl.
func (n *graphNodeMissingProvider) DotOrigin() bool { func (n *graphNodeProvider) DotOrigin() bool {
return true return true
} }
// GraphNodeFlattenable impl. // GraphNodeFlattenable impl.
func (n *graphNodeMissingProvider) Flatten(p []string) (dag.Vertex, error) { func (n *graphNodeProvider) Flatten(p []string) (dag.Vertex, error) {
return &graphNodeMissingProviderFlat{ return &graphNodeProviderFlat{
graphNodeMissingProvider: n, graphNodeProvider: n,
PathValue: p, PathValue: p,
}, nil }, nil
} }
// Same as graphNodeMissingProvider, but for flattening // Same as graphNodeMissingProvider, but for flattening
type graphNodeMissingProviderFlat struct { type graphNodeProviderFlat struct {
*graphNodeMissingProvider *graphNodeProvider
PathValue []string PathValue []string
} }
func (n *graphNodeMissingProviderFlat) Name() string { func (n *graphNodeProviderFlat) Name() string {
return fmt.Sprintf( return fmt.Sprintf(
"%s.%s", modulePrefixStr(n.PathValue), n.graphNodeMissingProvider.Name()) "%s.%s", modulePrefixStr(n.PathValue), n.graphNodeProvider.Name())
} }
func (n *graphNodeMissingProviderFlat) Path() []string { func (n *graphNodeProviderFlat) Path() []string {
return n.PathValue return n.PathValue
} }
func (n *graphNodeMissingProviderFlat) ProviderName() string { func (n *graphNodeProviderFlat) ProviderName() string {
return fmt.Sprintf( return fmt.Sprintf(
"%s.%s", modulePrefixStr(n.PathValue), "%s.%s", modulePrefixStr(n.PathValue),
n.graphNodeMissingProvider.ProviderName()) n.graphNodeProvider.ProviderName())
} }
// GraphNodeDependable impl. // GraphNodeDependable impl.
func (n *graphNodeMissingProviderFlat) DependableName() []string { func (n *graphNodeProviderFlat) DependableName() []string {
return []string{n.Name()} return []string{n.Name()}
} }
func (n *graphNodeMissingProviderFlat) DependentOn() []string { func (n *graphNodeProviderFlat) DependentOn() []string {
var result []string var result []string
// If we're in a module, then depend on our parent's provider // If we're in a module, then depend on our parent's provider
@ -480,7 +480,7 @@ func (n *graphNodeMissingProviderFlat) DependentOn() []string {
result = append(result, fmt.Sprintf( result = append(result, fmt.Sprintf(
"%s%s", "%s%s",
prefix, n.graphNodeMissingProvider.Name())) prefix, n.graphNodeProvider.Name()))
} }
return result return result

View File

@ -222,14 +222,14 @@ func TestDisableProviderTransformer_keep(t *testing.T) {
} }
} }
func TestGraphNodeMissingProvider_impl(t *testing.T) { func TestGraphNodeProvider_impl(t *testing.T) {
var _ dag.Vertex = new(graphNodeMissingProvider) var _ dag.Vertex = new(graphNodeProvider)
var _ dag.NamedVertex = new(graphNodeMissingProvider) var _ dag.NamedVertex = new(graphNodeProvider)
var _ GraphNodeProvider = new(graphNodeMissingProvider) var _ GraphNodeProvider = new(graphNodeProvider)
} }
func TestGraphNodeMissingProvider_ProviderName(t *testing.T) { func TestGraphNodeProvider_ProviderName(t *testing.T) {
n := &graphNodeMissingProvider{ProviderNameValue: "foo"} n := &graphNodeProvider{ProviderNameValue: "foo"}
if v := n.ProviderName(); v != "foo" { if v := n.ProviderName(); v != "foo" {
t.Fatalf("bad: %#v", v) t.Fatalf("bad: %#v", v)
} }