[#16] .Update added linter report as well

.github/workflows/go.yml

@@ -24,8 +24,17 @@ jobs:
           golangci-lint --version
           make lint
 
-  test:
-    name: Test
+      - name: Run the unit tests and check for race conditions
+        run: make test.report
+
+      - name: SonarCloud Scan
+        uses: sonarsource/sonarcloud-github-action@master
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
+
+  integration-test:
+    name: Integration Test
     runs-on: ubuntu-latest
     steps:
       - name: Set up Go 1.13
@@ -37,15 +46,6 @@ jobs:
       - name: Check out code into the Go module directory
         uses: actions/checkout@v1
 
-      - name: Run the unit tests and check for race conditions
-        run: make test.report
-
-      - name: SonarCloud Scan
-        uses: sonarsource/sonarcloud-github-action@master
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-          SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
-
       - name: Start the local gremlin as preparation for the integration test and run the integration tests
         run: |
           make infra.up

.gitignore

@@ -12,3 +12,5 @@ local
 coverage.out
 
 test-report.out
+
+lint.out

Makefile

@@ -15,9 +15,6 @@ test: sep ## Runs all unittests and generates a coverage report.
 	@echo "--> Run the unit-tests and checks for race conditions."
 	@go test -timeout 30s -race -run "^Test.*[^IT]$$" -covermode=atomic
 
-test.report: sep ## Runs all unittests and generates a coverage- and a test-report.
-	@echo "--> Run the unit-tests"	
-	@go test -timeout 30s -race -run "^Test.*[^IT]$$" -covermode=atomic -coverprofile=coverage.out -json | tee test-report.out
 
 test.integration: sep ## Runs all integration tests. As precondition a local gremlin-server has to run and listen on port 8182.
 	@echo "--> Run the integration-tests"
@@ -32,6 +29,15 @@ lint: ## Runs the linter to check for coding-style issues
 	@echo "!!!!golangci-lint has to be installed. See: https://github.com/golangci/golangci-lint#install"
 	@golangci-lint run --fast
 
+report.test: sep ## Runs all unittests and generates a coverage- and a test-report.
+	@echo "--> Run the unit-tests"	
+	@go test -timeout 30s -race -run "^Test.*[^IT]$$" -covermode=atomic -coverprofile=coverage.out -json | tee test-report.out
+
+report.lint: ## Runs the linter to check for coding-style issues and generates the report file used in the ci pipeline
+	@echo "--> Lint project + Reporting"
+	@echo "!!!!golangci-lint has to be installed. See: https://github.com/golangci/golangci-lint#install"
+	@golangci-lint run --fast --out-format checkstyle | tee lint.out
+
 gen-mocks: sep ## Generates test doubles (mocks).
 	@echo "--> generate mocks (github.com/golang/mock/gomock is required for this)"
 	@go get github.com/golang/mock/gomock

connection.go

@@ -173,7 +173,7 @@ func (ws *websocket) Read() (messageType int, msg []byte, err error) {
 	if err := ws.conn.SetReadDeadline(time.Now().Add(ws.readingWait)); err != nil {
 		return 0, nil, err
 	}
-
+	fmt.Printf("%d", "hallo")
 	return ws.conn.ReadMessage()
 }
 

sonar-project.properties

@@ -9,4 +9,5 @@ sonar.tests=.
 sonar.test.inclusions=**/*_test.go
 sonar.test.exclusions=**/vendor/**,**/test/mocks/**
 sonar.go.coverage.reportPaths=coverage.out
-sonar.go.tests.reportPaths=test-report.out
+sonar.go.tests.reportPaths=test-report.out
+sonar.go.golangci-lint.reportPaths=lint.out