Fix integration and go-lint tests

src/common/runner/runner_test.go

@@ -223,18 +223,24 @@ func TestRunnerInternals(t *testing.T) {
 	})
 
 	t.Run("Test skip resource by comment", func(t *testing.T) {
-		options := clioptions.TagOptions{
-			Directory: "../../../tests/terraform/skipComment/skipOne.tf",
-			Parsers:   []string{"Terraform"},
-		}
-		runner := Runner{}
-		runner.Init(&options)
-		runner.parsers[0].ParseFile(options.Directory)
-		utils.AppendSkipedByCommentToRunnerSkippedResources(&runner.skippedResources)
-		result := make([]string, 0)
-		result = append(result, "aws_instance.example_instance")
-		assert.Equal(t, result, runner.skippedResources)
-	})
+        options := clioptions.TagOptions{
+            Directory: "../../../tests/terraform/skipComment/skipOne.tf",
+            Parsers:   []string{"Terraform"},
+        }
+        runner := Runner{}
+        err := runner.Init(&options)
+        if err != nil {
+            t.Error(err)
+        }
+        tfBlocks, err := runner.parsers[0].ParseFile(options.Directory)
+        if err != nil {
+            t.Error(err)
+        }
+        assert.Equal(t, 3, len(tfBlocks))
+        utils.AppendSkipedByCommentToRunnerSkippedResources(&runner.skippedResources)
+		result := []string{"aws_instance.example_instance"}
+        assert.Equal(t, result, runner.skippedResources)
+    })
 
 	t.Run("Test skip resource - cloudformation", func(t *testing.T) {
 		runner := Runner{}

src/common/utils/utils.go

@@ -9,6 +9,7 @@ import (
 	"regexp"
 	"strings"
 	"unicode"
+	"sync"
 
 	"github.com/bridgecrewio/yor/src/common"
 	"github.com/bridgecrewio/yor/src/common/logger"
@@ -20,6 +21,10 @@ var RemoveGcpInvalidChars = regexp.MustCompile(`[^\p{Ll}\p{Lo}\p{N}_-]`)
 var SkipResourcesByComment = make([]string, 0)
 
 func AppendSkipedByCommentToRunnerSkippedResources(skippedResources *[]string) {
+    var mutex sync.Mutex
+	mutex.Lock()
+	defer mutex.Unlock()
+
 	*skippedResources = append(*skippedResources, SkipResourcesByComment...)
 	SkipResourcesByComment = SkipResourcesByComment[:0]
 }

src/terraform/structure/terraform_parser.go

@@ -179,7 +179,7 @@ func (p *TerraformParser) ParseFile(filePath string) ([]structure.IBlock, error)
 		line := terraformBlock.GetLines().Start
 		if line > 1 && line <= len(lines) {
 			lineAbove := lines[line-2]
-			if strings.ToUpper(strings.TrimSpace(lineAbove))== "#YOR:SKIPALL" {
+			if strings.ToUpper(strings.TrimSpace(lineAbove)) == "#YOR:SKIPALL" {
 				skipAll = true
 			}