Terraform v0.15 (#1099)

* Bump Terraform to v0.15

* Fix module fixtures for v0.15

* Checkout submodules in GitHub Actions

* Update docs

.github/ISSUE_TEMPLATE/bug_report.md

@@ -30,6 +30,6 @@ $ tflint -v
 TFLint version 0.26.0
 + ruleset.aws (0.3.1-bundled)
 $ terraform -v
-Terraform v0.14.9
+Terraform v0.15.0
 ```
 -->

.github/workflows/build.yml

@@ -20,6 +20,8 @@ jobs:
     steps:
     - name: Checkout
       uses: actions/checkout@v2
+      with:
+        submodules: true
     - name: Set up Go
       uses: actions/[email protected]
       with:

.github/workflows/docker.yml

@@ -8,6 +8,8 @@ jobs:
     steps:
     - name: Checkout
       uses: actions/checkout@v2
+      with:
+        submodules: true
     - name: docker build
       run: docker build -t terraform-linters/tflint .
     - name: docker run

.github/workflows/e2e.yml

@@ -20,6 +20,8 @@ jobs:
     steps:
     - name: Checkout
       uses: actions/checkout@v2
+      with:
+        submodules: true
     - name: Set up Go
       uses: actions/[email protected]
       with:

.github/workflows/generated_code_checks.yml

@@ -8,6 +8,8 @@ jobs:
     steps:
     - name: Checkout
       uses: actions/checkout@v2
+      with:
+        submodules: true
     - name: Set up Go
       uses: actions/[email protected]
       with:

.github/workflows/install_script.yml

@@ -18,6 +18,8 @@ jobs:
     steps:
     - name: Checkout
       uses: actions/checkout@v2
+      with:
+        submodules: true
     - name: Install latest version
       run: |
         bash install_linux.sh

.github/workflows/lint.yml

@@ -14,6 +14,8 @@ jobs:
     steps:
     - name: Checkout
       uses: actions/checkout@v2
+      with:
+        submodules: true
     - name: Set up Go
       uses: actions/[email protected]
       with:

.github/workflows/release.yml

@@ -13,6 +13,8 @@ jobs:
     steps:
     - name: Checkout
       uses: actions/checkout@v2
+      with:
+        submodules: true
     - name: Set up Go
       uses: actions/[email protected]
       with:

.gitmodules

@@ -0,0 +1,3 @@
+[submodule "tflint/test-fixtures/v0.15.0_module/.terraform/modules/consul"]
+	path = tflint/test-fixtures/v0.15.0_module/.terraform/modules/consul
+	url = https://github.com/hashicorp/terraform-aws-consul

docs/user-guide/compatibility.md

@@ -1,6 +1,6 @@
 # Compatibility with Terraform
 
-Since TFLint embeds a specific version of Terraform as a library, some features implicitly assume the behavior of Terraform v0.14.9.
+Since TFLint embeds a specific version of Terraform as a library, some features implicitly assume the behavior of Terraform v0.15.0
 
 Of course, TFLint may work correctly if you run it on other versions. But, false positives/negatives can occur based on this assumption.
 

go.mod

@@ -10,7 +10,7 @@ require (
 	github.com/hashicorp/go-version v1.3.0
 	github.com/hashicorp/hcl/v2 v2.9.1
 	github.com/hashicorp/logutils v1.0.0
-	github.com/hashicorp/terraform v0.14.10
+	github.com/hashicorp/terraform v0.15.0
 	github.com/jessevdk/go-flags v1.5.0
 	github.com/jstemmer/go-junit-report v0.9.1
 	github.com/mattn/go-colorable v0.1.8

tflint/loader_test.go

@@ -14,8 +14,8 @@ import (
 	"github.com/zclconf/go-cty/cty"
 )
 
-func Test_LoadConfig_v0_12_0(t *testing.T) {
-	withinFixtureDir(t, "v0.12.0_module", func() {
+func Test_LoadConfig_v0_15_0(t *testing.T) {
+	withinFixtureDir(t, "v0.15.0_module", func() {
 		loader, err := NewLoader(afero.Afero{Fs: afero.NewOsFs()}, moduleConfig())
 		if err != nil {
 			t.Fatalf("Unexpected error occurred: %s", err)
@@ -25,18 +25,14 @@ func Test_LoadConfig_v0_12_0(t *testing.T) {
 			t.Fatalf("Unexpected error occurred: %s", err)
 		}
 
-		if _, exists := config.Children["ecs_on_spotfleet"]; !exists {
-			t.Fatalf("`ecs_on_spotfleet` module is not loaded: %#v", config.Children)
-		}
-
-		if _, exists := config.Children["ecs_on_spotfleet"].Module.ManagedResources["aws_ecs_cluster.main"]; !exists {
-			t.Fatalf("`ecs_on_spotfleet` module resource `aws_ecs_cluster.main` is not loaded: %#v", config.Children["ecs_on_spotfleet"].Module.ManagedResources)
-		}
-
 		if _, exists := config.Children["instance"]; !exists {
 			t.Fatalf("`instance` module is not loaded: %#v", config.Children)
 		}
 
+		if _, exists := config.Children["instance"].Module.ManagedResources["aws_instance.main"]; !exists {
+			t.Fatalf("`instance` module resource `aws_instance.main` is not loaded: %#v", config.Children["instance"].Module.ManagedResources)
+		}
+
 		if _, exists := config.Children["consul"]; !exists {
 			t.Fatalf("`consul` module is not loaded: %#v", config.Children)
 		}
@@ -137,7 +133,7 @@ func Test_LoadConfig_invalidConfiguration(t *testing.T) {
 }
 
 func Test_Files(t *testing.T) {
-	withinFixtureDir(t, "v0.12.0_module", func() {
+	withinFixtureDir(t, "v0.15.0_module", func() {
 		loader, err := NewLoader(afero.Afero{Fs: afero.NewOsFs()}, EmptyConfig())
 		if err != nil {
 			t.Fatalf("Unexpected error occurred: %s", err)