Do not run rules on non-Teraform directories (#13)

Author: wata727

rules/terraform_required_version.go

@@ -48,6 +48,15 @@ func (r *TerraformRequiredVersionRule) Check(runner tflint.Runner) error {
 		return nil
 	}
 
+	files, err := runner.GetFiles()
+	if err != nil {
+		return err
+	}
+	if len(files) == 0 {
+		// This rule does not run on non-Terraform directory.
+		return nil
+	}
+
 	body, err := runner.GetModuleContent(&hclext.BodySchema{
 		Blocks: []hclext.BlockSchema{
 			{

rules/terraform_required_version_test.go

@@ -12,6 +12,10 @@ func Test_TerraformRequiredVersionRule(t *testing.T) {
 		Content  string
 		Expected helper.Issues
 	}{
+		{
+			Name:     "empty module",
+			Expected: helper.Issues{},
+		},
 		{
 			Name: "unset",
 			Content: `
@@ -55,7 +59,11 @@ terraform {
 
 	for _, tc := range cases {
 		t.Run(tc.Name, func(t *testing.T) {
-			runner := helper.TestRunner(t, map[string]string{"module.tf": tc.Content})
+			files := map[string]string{}
+			if tc.Content != "" {
+				files = map[string]string{"module.tf": tc.Content}
+			}
+			runner := helper.TestRunner(t, files)
 
 			if err := rule.Check(runner); err != nil {
 				t.Fatal(err)

rules/terraform_standard_module_structure.go

@@ -57,6 +57,15 @@ func (r *TerraformStandardModuleStructureRule) Check(runner tflint.Runner) error
 		return nil
 	}
 
+	files, err := runner.GetFiles()
+	if err != nil {
+		return err
+	}
+	if len(files) == 0 {
+		// This rule does not run on non-Terraform directory.
+		return nil
+	}
+
 	body, err := runner.GetModuleContent(&hclext.BodySchema{
 		Blocks: []hclext.BlockSchema{
 			{

rules/terraform_standard_module_structure_test.go

@@ -15,8 +15,15 @@ func Test_TerraformStandardModuleStructureRule(t *testing.T) {
 		Expected helper.Issues
 	}{
 		{
-			Name:    "empty module",
-			Content: map[string]string{},
+			Name:     "empty module",
+			Content:  map[string]string{},
+			Expected: helper.Issues{},
+		},
+		{
+			Name: "non-standard module",
+			Content: map[string]string{
+				"foo.tf": "",
+			},
 			Expected: helper.Issues{
 				{
 					Rule:    NewTerraformStandardModuleStructureRule(),