From 908c5fa44f5b681028f03057862cb5784ca969e0 Mon Sep 17 00:00:00 2001 From: Martin Atkins Date: Thu, 9 Mar 2023 09:08:19 -0800 Subject: [PATCH] addrs: PartialExpandedModule.String correct for entirely-unexpanded This logic was previously incorrect for any case where the number of expanded steps was zero, causing the string representation to begin with a dot when it ought not to. --- internal/addrs/partial_expanded.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/addrs/partial_expanded.go b/internal/addrs/partial_expanded.go index 0908c34f5f..65ad6570bd 100644 --- a/internal/addrs/partial_expanded.go +++ b/internal/addrs/partial_expanded.go @@ -155,7 +155,7 @@ func (pem PartialExpandedModule) String() string { buf.WriteString(pem.expandedPrefix.String()) } for i, callName := range pem.unexpandedSuffix { - if i > 0 || len(pem.unexpandedSuffix) != 0 { + if i > 0 || len(pem.expandedPrefix) != 0 { buf.WriteByte('.') } buf.WriteString("module.")