Add range to terraform_required_version errors

Author: rdimartino

rules/terraform_required_version.go

@@ -78,13 +78,32 @@ func (r *TerraformRequiredVersionRule) Check(runner tflint.Runner) error {
 		exists = exists || ok
 	}
 
-	if !exists {
-		return runner.EmitIssue(
-			r,
-			`terraform "required_version" attribute is required`,
-			hcl.Range{},
-		)
+	if exists {
+		return nil
+	}
+
+	var missingRange hcl.Range
+
+	if len(body.Blocks) > 0 {
+		missingRange = body.Blocks[0].DefRange
+	} else {
+		// If there are no "terraform" blocks, create a hcl.Range for a file
+
+		// Grab a single filename from keys of files map
+		var filename string
+		for k := range files {
+			filename = k
+			break
+		}
+
+		missingRange = hcl.Range{
+			Filename: filename,
+		}
 	}
 
-	return nil
+	return runner.EmitIssue(
+		r,
+		`terraform "required_version" attribute is required`,
+		missingRange,
+	)
 }

rules/terraform_required_version_test.go

@@ -3,6 +3,7 @@ package rules
 import (
 	"testing"
 
+	"github.com/hashicorp/hcl/v2"
 	"github.com/terraform-linters/tflint-plugin-sdk/helper"
 )
 
@@ -25,6 +26,17 @@ terraform {}
 				{
 					Rule:    NewTerraformRequiredVersionRule(),
 					Message: "terraform \"required_version\" attribute is required",
+					Range: hcl.Range{
+						Filename: "module.tf",
+						Start: hcl.Pos{
+							Line:   2,
+							Column: 1,
+						},
+						End: hcl.Pos{
+							Line:   2,
+							Column: 10,
+						},
+					},
 				},
 			},
 		},
@@ -53,6 +65,23 @@ terraform {
 `,
 			Expected: helper.Issues{},
 		},
+		{
+			Name: "no terraform block",
+			Content: `
+locals {
+	foo = "bar"
+}
+`,
+			Expected: helper.Issues{
+				{
+					Rule:    NewTerraformRequiredVersionRule(),
+					Message: "terraform \"required_version\" attribute is required",
+					Range: hcl.Range{
+						Filename: "module.tf",
+					},
+				},
+			},
+		},
 	}
 
 	rule := NewTerraformRequiredVersionRule()