Merge pull request #265 from SumoLogic/sumo_269118

Fixed testsuite

Author: akhil-sumologic

aws-observability-terraform/test/appmodule/README.md

@@ -13,8 +13,11 @@
 1. Clone git repository
 2. Set variables at following file
    - aws-observability-terraform/examples/appmodule/main.auto.tfvars
-3. Now Go back to folder aws-observability-terraform and run following command to execute all Test functions, with function name pattern as Test*
-```go test -v -timeout 50m ./test/appmodule```
+3. Now Go back to folder aws-observability-terraform and 
+   - Run the following command to execute all Test functions, with function name pattern as Test*
+   ```go test -v -timeout 120m ./test/appmodule```
+   For single Terratest case
+   ```go test -v -timeout 30m ./test/appmodule -run ^TestAppModule1$```
 4. Go test command runs all functions starting with name Test*. So if you want to run a particular test case according to your use case, comment the rest of the Testing functions. It’ll take less time to run than running all test cases.
 5. In case of failure - you can determine the cause of failure, fix the issue and re-run the failed test case.
 6. For example if you only run testcase 3 you can expect successful test execution as follows:

aws-observability-terraform/test/appmodule/terraform.go

@@ -2,10 +2,11 @@ package test
 
 import (
 	"fmt"
-	"testing"
-
 	"github.com/gruntwork-io/terratest/modules/terraform"
 	test_structure "github.com/gruntwork-io/terratest/modules/test-structure"
+	"os"
+	"path/filepath"
+	"testing"
 )
 
 // Deploy the resources using Terraform
@@ -42,4 +43,12 @@ func destroyTerraform(t *testing.T, workingDir string) {
 		terraform.DestroyE(t, terraformOptions)
 	}
 	fmt.Println(out)
+	fmt.Println("####################################################################")
+	fmt.Println("Clean up local state files")
+	fmt.Println("####################################################################")
+	// Clean up local state files
+	os.RemoveAll(filepath.Join(workingDir, ".terraform"))
+	os.Remove(filepath.Join(workingDir, ".test-data"))
+	os.Remove(filepath.Join(workingDir, "terraform.tfstate"))
+	os.Remove(filepath.Join(workingDir, "terraform.tfstate.backup"))
 }

aws-observability-terraform/test/sourcemodule/README.md

@@ -16,8 +16,11 @@
    - aws-observability-terraform/examples/sourcemodule/overrideSources/main.auto.tfvars
 3. Provide Existing collectorID and S3 bucket at function TestSourceModule4 at
    - aws-observability-terraform/test/sourcemodule/source_test.go
-4. Go to folder aws-observability-terraform and run following command to execute all Test functions, with function name pattern as Test* 
-```go test -v -timeout 50m ./test/sourcemodule```
+4. Go to folder aws-observability-terraform and 
+   - Run following command to execute all Test functions, with function name pattern as Test* 
+   ```go test -v -timeout 100m ./test/sourcemodule```
+   For single Terratest case
+   ```go test -v -timeout 20m ./test/sourcemodule -run ^TestSourceModule1$```
 5. Go test command runs all functions starting with name Test*. So if you want to run a particular test case according to your use case, comment the rest of the Testing functions. It’ll take less time to run than running all test cases.
 6. For example if you only run testcase 3 you can expect successful test execution as follows:
 

aws-observability-terraform/test/sourcemodule/source_test.go

@@ -179,7 +179,7 @@ func TestSourceModule2(t *testing.T) {
 	})
 }
 
-Testing scenerio 3 - Collect CW Logs via Lambda Log forwarder, CW Metrics via CW metric source
+// Testing scenario 3 - Collect CW Logs via Lambda Log forwarder, CW Metrics via CW metric source
 func TestSourceModule3(t *testing.T) {
 	// t.Parallel()