Remove nancy checks

.github/workflows/ci.yml

@@ -1,5 +1,5 @@
 name: CI
-on: pull_request
+on: [push]
 
 jobs:
   golangci-lint:
@@ -27,10 +27,10 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
-      - uses: reviewdog/action-hadolint@v1
+      - uses: hadolint/hadolint-action@v3.1.0
         with:
-          fail_on_error: true
-          reporter: github-pr-review
+          recursive: true
+          dockerfile: Dockerfile
 
   dotenv-linter:
     name: runner / dotenv-linter
@@ -41,39 +41,16 @@ jobs:
         with:
           reporter: github-pr-review
 
-  check-dependencies:
-    name: runner / check-dependencies
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-go@v2
-      - name: WriteGoList
-        run: go list -json -m all > go.list
-      - name: Nancy
-        uses: sonatype-nexus-community/nancy-github-action@main
-        continue-on-error: true
-
   tests:
     name: runner / tests
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
       - uses: actions/setup-go@v2
         with:
-          go-version: 1.17
-
+          go-version: "1.19.4"
       - name: Unit Tests
-        run: "go test \
-          -v \
-          -race \
-          -covermode atomic \
-          -coverprofile=coverage.txt \
-          ./internal/..."
+        run: "make test"
       - name: Upload coverage report
         run: bash <(curl -s https://codecov.io/bash)
 
-      - name: Integration tests
-        run: "docker-compose up \
-          --build \
-          --abort-on-container-exit \
-          --exit-code-from integration"

.gitignore

@@ -20,4 +20,7 @@ bin/
 *.out
 
 # Dependency directories (remove the comment below to include it)
-vendor/
+vendor/
+
+# coverage files
+coverage.txt

Dockerfile

@@ -18,4 +18,4 @@ COPY --from=builder /app/config /config
 COPY --from=builder /app/migrations /migrations
 COPY --from=builder /bin/app /app
 COPY --from=builder /etc/ssl/certs/ca-certificates.crt /etc/ssl/certs/
-CMD ["/app"]
+CMD ["/app"]

Makefile

@@ -16,10 +16,6 @@ compose-up: ### Run docker-compose
 	docker-compose up --build -d postgres rabbitmq && docker-compose logs -f
 .PHONY: compose-up
 
-compose-up-integration-test: ### Run docker-compose with integration test
-	docker-compose up --build --abort-on-container-exit --exit-code-from integration
-.PHONY: compose-up-integration-test
-
 compose-down: ### Down docker-compose
 	docker-compose down --remove-orphans
 .PHONY: compose-down
@@ -42,20 +38,20 @@ linter-golangci: ### check by golangci linter
 .PHONY: linter-golangci
 
 linter-hadolint: ### check by hadolint linter
-	git ls-files --exclude='Dockerfile*' --ignored | xargs hadolint
+	find . -name 'Dockerfile' | xargs hadolint
 .PHONY: linter-hadolint
 
 linter-dotenv: ### check by dotenv linter
 	dotenv-linter
 .PHONY: linter-dotenv
 
-test: ### run test
-	go test -v -cover -race ./internal/...
+test: ### run all tests including slow running system (e.g. system-tests)
+	go test --tags=system -v -cover -covermode atomic -coverprofile=coverage.txt ./internal/... ./pkg/...
 .PHONY: test
 
-integration-test: ### run integration-test
-	go clean -testcache && go test -v ./integration-test/...
-.PHONY: integration-test
+test-fast: ### run fast tests only
+	go test -v -cover ./internal/... ./pkg/...
+.PHONY: test-fast
 
 mock: ### run mockgen
 	mockgen -source ./internal/usecase/interfaces.go -package usecase_test > ./internal/usecase/mocks_test.go