Remove duplicate copy of truncateLongMessage function from rules/prov…

…ider.go
hattan · Aug 26, 2020 · 92becc8 · 92becc8
1 parent db1ebfc
commit 92becc8
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 17 deletions.
diff --git a/rules/azurerm_storage_account_invalid_name.go b/rules/azurerm_storage_account_invalid_name.go
@@ -55,7 +55,7 @@ func (r *AzurermStorageAccountInvalidNameRule) Check(runner tflint.Runner) error
 			if !r.pattern.MatchString(val) {
 				runner.EmitIssueOnExpr(
 					r,
-					fmt.Sprintf(`"%s" does not match valid pattern %s`, truncateLongMessage(val), `^[a-z0-9]{3,24}$`),
+					fmt.Sprintf(`"%s" does not match valid pattern %s`, val, `^[a-z0-9]{3,24}$`),
 					attribute.Expr,
 				)
 			}

diff --git a/rules/provider.go b/rules/provider.go
@@ -1,8 +1,6 @@
 package rules
 
 import (
-	"strings"
-
 	"github.com/terraform-linters/tflint-plugin-sdk/tflint"
 	"github.com/terraform-linters/tflint-ruleset-azurerm/rules/apispec"
 )
@@ -16,17 +14,3 @@ var Rules = append([]tflint.Rule{
 	NewAzurermWindowsVirtualMachineInvalidSizeRule(),
 	NewAzurermWindowsVirtualMachineScaleSetInvalidSkuRule(),
 }, apispec.Rules...)
-
-func truncateLongMessage(str string) string {
-	limit := 80
-
-	str = strings.Replace(str, "\r\n", "\n", -1)
-	str = strings.Replace(str, "\n", "\\n", -1)
-
-	r := []rune(str)
-	if len(r) > limit {
-		return string(r[0:limit]) + "..."
-	}
-
-	return str
-}